Skip to content

tests: disable a couple of flaky tests#17209

Merged
porridge merged 1 commit intomasterfrom
porridge/skip
Oct 9, 2025
Merged

tests: disable a couple of flaky tests#17209
porridge merged 1 commit intomasterfrom
porridge/skip

Conversation

@porridge
Copy link
Contributor

@porridge porridge commented Oct 9, 2025

Description

These have a high flake rate and no human activity on the related tickets:

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • modified existing tests

How I validated my change

CI

@porridge porridge requested review from a team and janisz October 9, 2025 05:56
@rhacs-bot
Copy link
Contributor

Images are ready for the commit at cab7af9.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.9.x-1026-gcab7af93bb.

@codecov
Copy link

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.84%. Comparing base (ba4b793) to head (cab7af9).
⚠️ Report is 1067 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #17209      +/-   ##
==========================================
- Coverage   48.84%   48.84%   -0.01%     
==========================================
  Files        2716     2716              
  Lines      203237   203237              
==========================================
- Hits        99266    99264       -2     
- Misses      96153    96155       +2     
  Partials     7818     7818              
Flag Coverage Δ
go-unit-tests 48.84% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@porridge porridge changed the title tests: disable a couple flaky tests tests: disable a couple of flaky tests Oct 9, 2025
@porridge porridge merged commit c3a9e96 into master Oct 9, 2025
91 checks passed
@porridge porridge deleted the porridge/skip branch October 9, 2025 11:15
@janisz
Copy link
Contributor

janisz commented Oct 9, 2025

@janisz janisz added backport release-4.8 backport release-4.9 https://spaces.redhat.com/spaces/StackRox/pages/558727298 labels Feb 10, 2026
@rhacs-bot
Copy link
Contributor

The backport to release-4.9 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-4.9 release-4.9
# Navigate to the new working tree
cd .worktrees/backport-release-4.9
# Create a new branch
git switch --create backport-17209-to-release-4.9
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 c3a9e96f1cb9001d7ffe534d07dac2f5011e7e61
# Push it to GitHub
git push --set-upstream origin backport-17209-to-release-4.9
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-4.9

Then, create a pull request where the base branch is release-4.9 and the compare/head branch is backport-17209-to-release-4.9.

rhacs-bot pushed a commit that referenced this pull request Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release-4.8 backport release-4.9 https://spaces.redhat.com/spaces/StackRox/pages/558727298

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants