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.

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

e2e: Switch to using default timed context everywhere #1910

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

marun
Copy link
Contributor

@marun marun commented Aug 24, 2023

Why this should be merged

This change simplifies the test code and the long default timeout should avoid unnecessary flaking.

TODO

@marun marun added the testing This primarily focuses on testing label Aug 24, 2023
@marun marun force-pushed the e2e-default-context branch from 4360544 to 27c6045 Compare August 24, 2023 18:07
@marun marun self-assigned this Aug 24, 2023
@marun marun force-pushed the e2e-default-context branch 3 times, most recently from 970b84d to f7aefe5 Compare August 28, 2023 00:46
@marun marun force-pushed the e2e-default-context branch 3 times, most recently from cf568f6 to e9cd992 Compare September 7, 2023 01:37
@marun marun marked this pull request as ready for review September 7, 2023 02:10
@marun marun force-pushed the e2e-default-context branch from 1315380 to ea04182 Compare September 21, 2023 07:04
@marun
Copy link
Contributor Author

marun commented Sep 21, 2023

Rebased

@marun marun force-pushed the e2e-default-context branch from ea04182 to 11adc70 Compare September 21, 2023 07:43
@StephenButtolph StephenButtolph added this to the v1.10.11 milestone Sep 21, 2023
@StephenButtolph StephenButtolph merged commit a39717b into dev Sep 21, 2023
@StephenButtolph StephenButtolph deleted the e2e-default-context branch September 21, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing This primarily focuses on testing
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants