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

Attempting to fix flaky tests #180

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

MbolotSuse
Copy link
Contributor

Some tests which relied on timeouts were a bit flaky in CI. This PR raises the timeouts, which seemed to help with the flakiness on local testing.

@MbolotSuse MbolotSuse requested a review from tomleb April 3, 2024 13:32
@MbolotSuse MbolotSuse requested a review from a team as a code owner April 3, 2024 13:32
@JonCrowther
Copy link
Contributor

Addresses: rancher/rancher#44873

@MbolotSuse
Copy link
Contributor Author

Note: @tomleb suggested some improvements, putting this on hold until they are done.

Copy link
Contributor

@tomleb tomleb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some typos but otherwise looks good. Would be great to test the CI a couple of times to give a best effort verification on whether it fixes the issue or not.

You could for example create a new dummy PR with the same codechange but and give -count parameter to the go test command.

pkg/debounce/refresher_test.go Outdated Show resolved Hide resolved
pkg/schema/definitions/refresh_test.go Outdated Show resolved Hide resolved
Some tests which relied on timeouts were a bit flaky in CI. This PR
refactors a few of them to work on a more reliable method of receiving
from a channel and raises the timeout of another test.
@MbolotSuse MbolotSuse merged commit bcba9d5 into rancher:master Apr 4, 2024
1 check passed
@MbolotSuse MbolotSuse deleted the flaky-test branch April 4, 2024 13:23
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.

3 participants