Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Additional attempts for check_http_traffic, and update skip test message #178

Merged
merged 1 commit into from
Oct 28, 2023
Merged

Additional attempts for check_http_traffic, and update skip test message #178

merged 1 commit into from
Oct 28, 2023

Conversation

sawsa307
Copy link
Contributor

@sawsa307 sawsa307 commented Oct 27, 2023 •
edited
Loading

  • Extend check traffic wait time from 60x5s=5min to 180x5=15min since existing tests are failing due to insufficient wait time.
  • Update skip test message in recipe_test.go.

@sawsa307
Additional attempts for check_http_traffic, and update skip test message
9682ae9 
* Extend check traffic wait time from 60*5s=5min to 180*5=15min since
  existing tests are failing due to insufficient wait time.
* Update skip test message in recipe_test.go.
@sawsa307 sawsa307 force-pushed the update-test-framework branch from 1e5d814 to 9682ae9 Compare October 27, 2023 22:02
@sawsa307
Copy link
Contributor Author

/assign @bowei

@bowei bowei merged commit 2e1fc43 into GoogleCloudPlatform:main Oct 28, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bowei bowei bowei approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sawsa307 @bowei