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

tests: Add wait for function and packagerevision readiness #3751

Merged
merged 1 commit into from
Jan 26, 2023

Conversation

justinsb
Copy link
Contributor

The agggregated apiserver can sometimes be slow to respond, for
example if it is polling git synchronously. After creating a
Repository in our tests, wait for the aggregated apiserver to start
responding before returning.

This should clean up some test flakes.

@justinsb
Copy link
Contributor Author

/retest

Looks like we have more than one class of flake :-)

@justinsb justinsb force-pushed the add_wait_to_tests branch 4 times, most recently from 020354a to c3c771a Compare January 25, 2023 02:02
The agggregated apiserver can sometimes be slow to respond, for
example if it is polling git synchronously.  After creating a
Repository in our tests, wait for the aggregated apiserver to start
responding before returning.

This should clean up some test flakes.
@justinsb justinsb merged commit 0632148 into kptdev:main Jan 26, 2023
@natasha41575 natasha41575 mentioned this pull request Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants