Skip to content

Fix bug where CanaryWeight is reset to 0 during CanaryPhasePromoting#1851

Merged
aryan9600 merged 1 commit intofluxcd:mainfrom
sayap:fix-wrong-weight-when-promoting
Mar 8, 2026
Merged

Fix bug where CanaryWeight is reset to 0 during CanaryPhasePromoting#1851
aryan9600 merged 1 commit intofluxcd:mainfrom
sayap:fix-wrong-weight-when-promoting

Conversation

@sayap
Copy link
Copy Markdown
Contributor

@sayap sayap commented Oct 27, 2025

This is just a display issue in the status, that can cause some confusion.

@sayap sayap force-pushed the fix-wrong-weight-when-promoting branch from 549d7c7 to 095a31d Compare October 27, 2025 08:50
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 30.00%. Comparing base (12ee6cb) to head (ceab3ec).
⚠️ Report is 83 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1851      +/-   ##
==========================================
- Coverage   39.44%   30.00%   -9.44%     
==========================================
  Files         287      287              
  Lines       22706    18467    -4239     
==========================================
- Hits         8956     5541    -3415     
+ Misses      12777    12195     -582     
+ Partials      973      731     -242     

☔ View full report in Codecov by Sentry.
📢 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.

Copy link
Copy Markdown
Member

@aryan9600 aryan9600 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for raising this! 🙇

@aryan9600 aryan9600 force-pushed the fix-wrong-weight-when-promoting branch from 095a31d to ceab3ec Compare March 7, 2026 09:32
@aryan9600 aryan9600 merged commit c9579da into fluxcd:main Mar 8, 2026
30 of 31 checks passed
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