Skip to content

feat(CI): add service healthcheck and better retry policy#3497

Merged
Leiyks merged 4 commits intomasterfrom
leiyks/fix-ci-stability
Nov 28, 2025
Merged

feat(CI): add service healthcheck and better retry policy#3497
Leiyks merged 4 commits intomasterfrom
leiyks/fix-ci-stability

Conversation

@Leiyks
Copy link
Copy Markdown
Contributor

@Leiyks Leiyks commented Nov 24, 2025

Description

This PR add some stability to the CI by:

  • Adding proper healthcheck when starting external services and test agent
  • Improving retry policy on failing jobs
  • Fix some timeout test that were successful but waiting for background command to finish
  • Make jobs uninterruptible on master branch
  • Fix System test missing dependencies

Jiras: LANGPLAT-643

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.69%. Comparing base (0c35caf) to head (1c501d3).
⚠️ Report is 339 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3497      +/-   ##
==========================================
- Coverage   61.83%   61.69%   -0.14%     
==========================================
  Files         142      142              
  Lines       12923    12923              
  Branches     1695     1695              
==========================================
- Hits         7991     7973      -18     
- Misses       4178     4196      +18     
  Partials      754      754              

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c35caf...1c501d3. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Leiyks Leiyks force-pushed the leiyks/fix-ci-stability branch 6 times, most recently from bc7711f to 42fe909 Compare November 26, 2025 10:44
@Leiyks Leiyks force-pushed the leiyks/fix-ci-stability branch 3 times, most recently from 1e9d619 to c60b418 Compare November 26, 2025 15:07
@Leiyks Leiyks marked this pull request as ready for review November 26, 2025 15:07
@Leiyks Leiyks requested a review from a team as a code owner November 26, 2025 15:07
@Leiyks Leiyks force-pushed the leiyks/fix-ci-stability branch from c60b418 to ab1f9e0 Compare November 27, 2025 15:33
@Leiyks Leiyks changed the title feat: add service healthcheck and better retry policy feat(CI): add service healthcheck and better retry policy Nov 27, 2025
@Leiyks Leiyks force-pushed the leiyks/fix-ci-stability branch 2 times, most recently from 9b55558 to c5306d7 Compare November 27, 2025 17:03
Copy link
Copy Markdown
Member

@realFlowControl realFlowControl left a comment

Choose a reason for hiding this comment

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

Looks good, I left some questions to consider before merging

Comment thread .gitlab/generate-common.php
Comment thread .gitlab/generate-tracer.php Outdated
Comment thread .gitlab/generate-tracer.php
Comment thread .gitlab/wait-for-service-ready.sh Outdated
@Leiyks Leiyks force-pushed the leiyks/fix-ci-stability branch from c5306d7 to 1c501d3 Compare November 28, 2025 12:34
@Leiyks Leiyks merged commit 9468244 into master Nov 28, 2025
2005 of 2007 checks passed
@Leiyks Leiyks deleted the leiyks/fix-ci-stability branch November 28, 2025 14:33
@github-actions github-actions Bot added this to the 1.15.0 milestone Nov 28, 2025
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