Skip to content

Conversation

@VaishnaviHire
Copy link
Collaborator

@VaishnaviHire VaishnaviHire commented Aug 1, 2025

During upgrades, if we do server-side apply without preserving the selectors field, we get 'field is immutable' error which blocks deployment updates and also causes continuous operator reconciliation. Preserving existing selector values fixes the issue.

Copy link
Collaborator

@rhdedgar rhdedgar left a comment

Choose a reason for hiding this comment

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

Looks good to me. Approved!

Copy link
Collaborator

@mfleader mfleader left a comment

Choose a reason for hiding this comment

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

Do you have time to add a regression test since our test suite didn't catch this? If not I can write one as a follow up.

During upgrades, if we do server-side apply without preserving the selectors field, we get 'field is immutable'
error which blocks deployment updates and also causes continuous operator reconciliation.
Preserving exists selector values fixes the issue.

Signed-off-by: Vaishnavi Hire <vhire@redhat.com>
@mergify mergify bot merged commit e2cc886 into llamastack:main Aug 1, 2025
6 checks passed
VaishnaviHire added a commit to VaishnaviHire/llama-stack-k8s-operator that referenced this pull request Aug 19, 2025
During upgrades, if we do server-side apply without preserving the selectors field, we get 'field is immutable' error which blocks deployment updates and also causes continuous operator reconciliation. Preserving existing selector values fixes the issue.

Approved-by: rhdedgar

Approved-by: mfleader
(cherry picked from commit e2cc886)
VaishnaviHire added a commit to VaishnaviHire/llama-stack-k8s-operator that referenced this pull request Sep 18, 2025
During upgrades, if we do server-side apply without preserving the selectors field, we get 'field is immutable' error which blocks deployment updates and also causes continuous operator reconciliation. Preserving existing selector values fixes the issue.

Approved-by: rhdedgar

Approved-by: mfleader
(cherry picked from commit e2cc886)
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