Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Fix ako statefulset upgrade issue #2566

Merged

Conversation

XudongLiuHarold
Copy link
Member

Signed-off-by: Xudong Liu [email protected]

What this PR does / why we need it

Currently, when users upgrade load-balancer-and-ingress package, they may meet following error:

 USEFUL-ERROR-MESSAGE: kapp: Error: Applying update statefulset/ako (apps/v1) namespace: avi-system:
 Updating resource statefulset/ako (apps/v1) namespace: avi-system:
 StatefulSet.apps "ako" is invalid: spec:
 Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden (reason: Invalid)

if they changed anything in those fields.

To avoid this problem, we add annotations to ignore default kapp-controller rules to ensure upgrade successfully.

Details for the Release Notes (PLEASE PROVIDE)

Users can change statefuleset's template, replica and etc field now.

Which issue(s) this PR fixes

Fixes: #

Describe testing done for PR

Special notes for your reviewer

@github-actions github-actions bot added the owner/packages Work executed by a package's maintainer label Nov 17, 2021
@XudongLiuHarold XudongLiuHarold requested a review from a team November 17, 2021 19:25
Copy link
Contributor

@HanFa HanFa left a comment

Choose a reason for hiding this comment

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

LGTM

@XudongLiuHarold XudongLiuHarold merged commit d1426f4 into main Nov 22, 2021
@davidvonthenen davidvonthenen deleted the topic/xudongl/fix-ako-statefulset-upgrade-issue branch November 29, 2021 18:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/packages Work executed by a package's maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants