Skip to content

Tune OS test timeouts for e2e runs#7801

Merged
ncopa merged 3 commits into
k0sproject:mainfrom
twz123:conformance-tests
Jun 16, 2026
Merged

Tune OS test timeouts for e2e runs#7801
ncopa merged 3 commits into
k0sproject:mainfrom
twz123:conformance-tests

Conversation

@twz123

@twz123 twz123 commented Jun 16, 2026

Copy link
Copy Markdown
Member

Description

This mostly applies to full conformance test runs that are called from the release workflow. In order to obtain the "Certified Kubernetes" badge, these cannot be run in parallel. They currently take around two hours to complete, close to the Ginkgo test timeout. Funnily enough, the 64-bit ARM VMs are slightly faster than the 64-bit x86_64 VMs, resulting in mostly successful arm64 runs and very unreliable amd64 runs. Moreover, Ginkgo/sonobuoy simply report any overall test timeouts as failures of the pending tests, with no indication that the tests did not even complete. This makes it very difficult to realize that this is a timeout issue rather than a flaky test.

For an all-serial workflow step, use the following timeouts:

  • three hours for Ginkgo,
  • three and a half hours for Sonobuoy
  • and four hours for the workflow step.

When the serial step follows a parallel step, use tighter timeouts because only the remaining serial tests should run:

  • thirty minutes for Ginkgo,
  • forty-five minutes for Sonobuoy,
  • and one hur for the workflow step.

See:

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

  • Manual test
  • Auto test added

Checklist

  • My code follows the style guidelines of this project
  • My commit messages are signed-off
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

twz123 added 3 commits June 16, 2026 12:22
This is useful for the release workflow, where we're only interested in
successful runs and a quicker feedback loop.

Signed-off-by: Tom Wieczorek <twieczorek@mirantis.com>
This mostly applies to full conformance test runs that are called from
the release workflow. In order to obtain the "Certified Kubernetes"
badge, these cannot be run in parallel. They currently take around two
hours to complete, close to the Ginkgo test timeout. Funnily enough,
the 64-bit ARM VMs are slightly faster than the 64-bit x86_64 VMs,
resulting in mostly successful arm64 runs and very unreliable amd64
runs. Moreover, Ginkgo/sonobuoy simply report any overall test timeouts
as failures of the pending tests, with no indication that the tests did
not even complete. This makes it very difficult to realize that this is
a timeout issue rather than a flaky test.

For an all-serial workflow step, use the following timeouts:

* three hours for Ginkgo,
* three and a half hours for Sonobuoy
* and four hours for the workflow step.

When the serial step follows a parallel step, use tighter timeouts
because only the remaining serial tests should run:

* thirty minutes for Ginkgo,
* forty-five minutes for Sonobuoy,
* and one hur for the workflow step.

See: 71b3209 ("Use OS tests in release workflow")
Signed-off-by: Tom Wieczorek <twieczorek@mirantis.com>
Keep the format, but instead of writing it into a console group, write
it into a step summary.

Signed-off-by: Tom Wieczorek <twieczorek@mirantis.com>
@twz123 twz123 added bug Something isn't working area/ci backport/release-1.36 PR that needs to be backported/cherrypicked to the release-1.36 branch labels Jun 16, 2026
@twz123 twz123 marked this pull request as ready for review June 16, 2026 10:37
@twz123 twz123 requested review from a team as code owners June 16, 2026 10:37
@twz123 twz123 requested review from jnummelin and kke June 16, 2026 10:37
@ncopa ncopa merged commit 04c5a84 into k0sproject:main Jun 16, 2026
12 checks passed
@k0s-bot

k0s-bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release-1.36:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci backport/release-1.36 PR that needs to be backported/cherrypicked to the release-1.36 branch bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants