Skip to content

Commit

Permalink
Make the timeout a param
Browse files Browse the repository at this point in the history
  • Loading branch information
houshengbo committed Feb 27, 2024
1 parent 2cf8548 commit 3e4928f
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 7 deletions.
4 changes: 4 additions & 0 deletions test/e2e-common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ export TEST_RESOURCE="knative"
export TEST_EVENTING_MONITORING_NAMESPACE="knative-monitoring"
export KO_FLAGS="${KO_FLAGS:-}"
export INGRESS_CLASS=${INGRESS_CLASS:-istio.ingress.networking.knative.dev}
export TIMEOUT_CI=30m

# GKE cluster version
readonly K8S_CLUSTER_VERSION=1.28

# Boolean used to indicate whether to generate serving YAML based on the latest code in the branch KNATIVE_SERVING_REPO_BRANCH.
GENERATE_SERVING_YAML=0
Expand Down
4 changes: 2 additions & 2 deletions test/e2e-eventing-upgrade-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -83,9 +83,9 @@ function test_setup() {
}

# Skip installing istio as an add-on.
initialize --cluster-version=1.28 "$@" --skip-istio-addon
initialize --cluster-version=${K8S_CLUSTER_VERSION} "$@" --skip-istio-addon

TIMEOUT=30m
TIMEOUT=${TIMEOUT_CI}

header "Running upgrade tests"

Expand Down
4 changes: 2 additions & 2 deletions test/e2e-serving-upgrade-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -74,9 +74,9 @@ function create_test_namespace_serving() {
}

# Skip installing istio as an add-on.
initialize --cluster-version=1.28 "$@" --skip-istio-addon
initialize --cluster-version=${K8S_CLUSTER_VERSION} "$@" --skip-istio-addon

TIMEOUT=30m
TIMEOUT=${TIMEOUT_CI}

header "Running upgrade tests"

Expand Down
2 changes: 1 addition & 1 deletion test/e2e-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ function knative_setup() {
}

# Skip installing istio as an add-on
initialize --cluster-version=1.28 $@ --skip-istio-addon
initialize --cluster-version=${K8S_CLUSTER_VERSION} $@ --skip-istio-addon

Check failure on line 37 in test/e2e-tests.sh

View workflow job for this annotation

GitHub Actions / style / suggester / shell

[shellcheck (suggestion)] reported by reviewdog 🐶 Raw Output: test/e2e-tests.sh:37:-initialize --cluster-version=${K8S_CLUSTER_VERSION} $@ --skip-istio-addon test/e2e-tests.sh:37:+initialize --cluster-version="${K8S_CLUSTER_VERSION}" $@ --skip-istio-addon

# If we got this far, the operator installed Knative Serving
header "Running tests for Knative Operator"
Expand Down
4 changes: 2 additions & 2 deletions test/e2e-upgrade-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ export GO111MODULE=auto
source "$(dirname "${BASH_SOURCE[0]}")/e2e-common.sh"

# Skip installing istio as an add-on.
initialize --cluster-version=1.28 "$@" --skip-istio-addon
initialize --cluster-version=${K8S_CLUSTER_VERSION} "$@" --skip-istio-addon

TIMEOUT=30m
TIMEOUT=${TIMEOUT_CI}

header "Running upgrade tests"

Expand Down

0 comments on commit 3e4928f

Please sign in to comment.