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: increase timeout on testCPUSecurityMitigationsEnable #19613

Merged

Conversation

tomasmatus
Copy link
Member

This test barely makes it within the default 10 minutes timeout. From what I see most of the time is spent by waiting for multiple reboots of the machine.
Locally this took almost 7 minutes to run so for CI we can bump this timeout to 20 minutes.

This test barely makes it within the default 10 minutes timeout.
From what I see most of the time is spent by waiting for multiple
reboots of the machine.
Locally this took almost 7 minutes to run so for CI we can bump this
timeout to 20 minutes.
@martinpitt martinpitt added the flake unstable test label Nov 15, 2023
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

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

Dekuji!

@martinpitt
Copy link
Member

Meh, lots of new flakes here from 3x affected. The testCryptoPolicies timeouts are harder, as a successful run takes less than 5 mins.

So if you are looking for something else, please create a new PR which amplifies that test, and let it run with verbosity, so that we know where exactly it hangs. (Ping me on chat for details)

In the meantime, one round of retries, as this is really trivial and separate.

@martinpitt martinpitt merged commit d2ccfc0 into cockpit-project:main Nov 15, 2023
90 of 93 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flake unstable test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants