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

helm chart v1.22.4 breaks upgrades due to additional matchLabels added #679

Open
3 tasks done
joemiller opened this issue Dec 19, 2024 · 0 comments · May be fixed by #680
Open
3 tasks done

helm chart v1.22.4 breaks upgrades due to additional matchLabels added #679

joemiller opened this issue Dec 19, 2024 · 0 comments · May be fixed by #680
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@joemiller
Copy link

joemiller commented Dec 19, 2024

Preflight Checklist

  • I have searched the issue tracker for an issue that matches the one I want to file, without success.
  • I am not looking for support or already pursued the available support channels without success.
  • I agree to follow the Code of Conduct.

Operator Version

1.22.4

Installation Type

Official Helm chart

Bank-Vaults Version

No response

Kubernetes Version

1.30.0

Kubernetes Distribution/Provisioner

GKE, EKS

Expected Behavior

Upgrading the helm chart should succeed.

Actual Behavior

Using ArgoCD, when attempting to upgrade the helm chart from 1.22.3 -> 1.22.4, the deployment update is rejected because a new label was added and hardcoded into the deployment.yaml template:

Failed sync attempt to 57acc1b15547b561f72d2ba9a46061f4f889f331: one or more objects failed to apply, reason: error when patching "/dev/shm/300371924": Deployment.apps "vault-operator" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app.kubernetes.io/instance":"vault-operator", "app.kubernetes.io/name":"vault-operator"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable (retried 5 times).

app.kubernetes.io/instance added in 2bd7c7d#diff-9740ffbfce1675e80ab604e95ffa60ef0c21f28db348b2dd2e0f38da341b4a02R20

Steps To Reproduce

  1. upgrade an vault-operator instance installed via the official helm chart from 1.22.3 to 1.22.4

Configuration

n/a

Logs

n/a

Additional Information

No response

@joemiller joemiller added the kind/bug Categorizes issue or PR as related to a bug. label Dec 19, 2024
@joemiller joemiller changed the title helm chart v1.22.4 breaks upgrades with additional matchLabels added helm chart v1.22.4 breaks upgrades due to additional matchLabels added Dec 19, 2024
@csatib02 csatib02 linked a pull request Dec 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant