Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 632: Upgrade of pravega cluster is failing intermittently #633

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

anishakj
Copy link
Contributor

Signed-off-by: anisha.kj anisha.kj@dell.com

Change log description

In some cases, after updating stateful set, updatedReplicas field in sts is not updating correctly. Due to this, the check

if sts.Status.UpdatedReplicas == sts.Status.Replicas &&
		sts.Status.UpdatedReplicas == sts.Status.ReadyReplicas 

is succeeding and make upgrade as completed. And segment store pods are not coming up with new image

Purpose of the change

Fixes #632

What the code does

Added additional check to ensure that all the pods are updated with new version

How to verify it

Verified that upgrade is working fine

Signed-off-by: anisha.kj <anisha.kj@dell.com>
@anishakj anishakj requested a review from nishant-yt July 20, 2022 05:08
@codecov-commenter
Copy link

Codecov Report

Merging #633 (8be60e5) into master (323a35a) will increase coverage by 0.08%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
+ Coverage   82.83%   82.92%   +0.08%     
==========================================
  Files          17       17              
  Lines        4073     4075       +2     
==========================================
+ Hits         3374     3379       +5     
+ Misses        565      562       -3     
  Partials      134      134              
Impacted Files Coverage Δ
controllers/upgrade.go 72.81% <60.00%> (-0.17%) ⬇️
controllers/pravegacluster_controller.go 55.69% <100.00%> (+0.71%) ⬆️

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 323a35a...8be60e5. Read the comment docs.

Copy link
Collaborator

@nishant-yt nishant-yt left a comment

Choose a reason for hiding this comment

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

Good to Go

@anishakj anishakj merged commit 01093bb into master Jul 20, 2022
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.

Upgrade of pravega cluster is failing intermittently
3 participants