Skip to content

Fix canary rollback behaviour#1171

Merged
stefanprodan merged 1 commit intofluxcd:mainfrom
aryan9600:fix-primary-restart
Apr 15, 2022
Merged

Fix canary rollback behaviour#1171
stefanprodan merged 1 commit intofluxcd:mainfrom
aryan9600:fix-primary-restart

Conversation

@aryan9600
Copy link
Member

Prevents the canary from getting triggered, when a canary deploy is
updated to match the primary deploy after an analysis fails.

Fixes: #1163

Signed-off-by: Sanskar Jaiswal [email protected]

@aryan9600 aryan9600 requested a review from stefanprodan as a code owner April 15, 2022 08:06
@codecov-commenter
Copy link

Codecov Report

Merging #1171 (1635732) into main (67cc965) will decrease coverage by 0.03%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##             main    #1171      +/-   ##
==========================================
- Coverage   56.81%   56.78%   -0.04%     
==========================================
  Files          79       79              
  Lines        6616     6622       +6     
==========================================
+ Hits         3759     3760       +1     
- Misses       2301     2304       +3     
- Partials      556      558       +2     
Impacted Files Coverage Δ
pkg/canary/status.go 47.74% <0.00%> (-0.63%) ⬇️
pkg/controller/scheduler.go 46.58% <25.00%> (-0.19%) ⬇️

Continue to review full report at Codecov.

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

Prevents the canary from getting triggered, when a canary deploy is
updated to match the primary deploy after an analysis fails.

Signed-off-by: Sanskar Jaiswal <[email protected]>
@aryan9600 aryan9600 force-pushed the fix-primary-restart branch from 1635732 to c7c0c76 Compare April 15, 2022 08:19
Copy link
Member

@stefanprodan stefanprodan 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 @aryan9600 🏅

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.

Primary pods are unnecessary re-created when syncing canary and primary Deployments after a failed canary analysis

3 participants