Skip to content

ci(workflows): integrate flake detector into ci workflow and fix control client test flake - #5053

Open
kislaykishore wants to merge 2 commits into
masterfrom
flake-test
Open

kislaykishore wants to merge 2 commits into
masterfrom
flake-test

Conversation

@kislaykishore

@kislaykishore kislaykishore commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Description

CI Parallelization & Consolidation:

  • Integrates the flake detector tests directly into .github/workflows/ci.yml as a matrix-based concurrent job (flake-detector) and removes .github/workflows/flake-detector.yml.
  • Runs flake-detector (unit) and flake-detector (race) concurrently across separate virtual machines alongside linux-tests, format-test, and lint, reducing total CI turnaround time while maintaining -p 1 for intra-test stability.

Link to the issue in case of a bug fix.

b//552815191

Testing details

  1. Manual - Verified with make build and checked CI workflow syntax.
  2. Unit tests - N/A
  3. Integration tests - N/A

Any backward incompatible change? If so, please explain.

N/A

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.77%. Comparing base (879d375) to head (93a5d75).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5053   +/-   ##
=======================================
  Coverage   86.77%   86.77%           
=======================================
  Files         176      176           
  Lines       18861    18863    +2     
=======================================
+ Hits        16367    16369    +2     
  Misses       2493     2493           
  Partials        1        1           
Flag Coverage Δ
unittests 86.77% <ø> (+<0.01%) ⬆️

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kislaykishore kislaykishore changed the title ci(workflows): integrate flake detector into ci workflow ci(workflows): integrate flake detector into ci workflow and fix control client test flake Aug 26, 2026
@kislaykishore
kislaykishore marked this pull request as ready for review August 26, 2026 13:56
@kislaykishore
kislaykishore requested a review from a team as a code owner August 26, 2026 13:56
@kislaykishore
kislaykishore requested a review from meet2mky August 26, 2026 13:56
@github-actions github-actions Bot added the remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR. label Aug 26, 2026
@kislaykishore
kislaykishore enabled auto-merge (squash) August 26, 2026 14:32
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@github-actions

Copy link
Copy Markdown

Hi @charith87, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

…ipping

- Restore tools/build_gcsfuse packaging and build verification in build job
- Ensure linux-tests aggregator properly fails on upstream matrix/build errors
- Fix -race flag and remove CGO_ENABLED=0 in flake-detector race suite
- Filter integration test packages from flake-detector unit suite
- Restrict flake-detector to master and workflow_dispatch to avoid PR runner bloat
- Return ^$ fallback regex in skip_tests when empty to prevent argument absorption
@github-actions

Copy link
Copy Markdown

Hi @charith87, @anushka567, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

3 similar comments
@github-actions

Copy link
Copy Markdown

Hi @charith87, @anushka567, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

@github-actions

Copy link
Copy Markdown

Hi @charith87, @anushka567, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

@github-actions

Copy link
Copy Markdown

Hi @charith87, @anushka567, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

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

Labels

remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants