Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Commit

Permalink
ci: use consistent wait times across test scenarios (#1545)
Browse files Browse the repository at this point in the history
A few CI runs are failing due to the CI timing out just
before the test passes. This change makes the CI timeout
consistent for tests and increasing the timeout by a bit
to give slower CI runs a chance to pass.
  • Loading branch information
shashankram authored Aug 17, 2020
1 parent 2c3f8bb commit c708123
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,10 @@ on:
- "**.md"
env:
KUBECONFIG: ".kube/config"
CI_WAIT_FOR_OK_SECONDS: 60
CI_WAIT_FOR_OK_SECONDS: 90
CI_MAX_ITERATIONS_THRESHOLD: 90
CI_CLIENT_CONCURRENT_CONNECTIONS: 1
CI_MAX_WAIT_FOR_POD_TIME_SECONDS: 60
CI_MAX_WAIT_FOR_POD_TIME_SECONDS: 90
CI_MIN_SUCCESS_THRESHOLD: 1
OSM_HUMAN_DEBUG_LOG: true

Expand Down Expand Up @@ -172,7 +172,6 @@ jobs:
BOOKTHIEF_EXPECTED_RESPONSE_CODE: "404"
ENABLE_EGRESS: "false"
EGRESS_EXPECTED_RESPONSE_CODE: "404" # egress is disabled
CI_WAIT_FOR_OK_SECONDS: 75
DEPLOY_TRAFFIC_SPLIT: "true"
run: |
touch .env
Expand Down Expand Up @@ -262,7 +261,6 @@ jobs:
BOOKTHIEF_EXPECTED_RESPONSE_CODE: "404"
ENABLE_EGRESS: "false"
EGRESS_EXPECTED_RESPONSE_CODE: "404" # egress is disabled
CI_WAIT_FOR_OK_SECONDS: 75
DEPLOY_TRAFFIC_SPLIT: "false"
run: |
touch .env
Expand Down

0 comments on commit c708123

Please sign in to comment.