Conversation
This comment was marked as outdated.
This comment was marked as outdated.
5287a67 to
4629544
Compare
4629544 to
ba2a9d1
Compare
Member
Author
|
/test |
ba2a9d1 to
5614c73
Compare
Member
Author
|
/test |
5614c73 to
0fa89a9
Compare
Member
Author
|
/test |
0fa89a9 to
a04ee30
Compare
Member
Author
|
/test |
a04ee30 to
b9a54ae
Compare
Member
Author
|
/test |
b9a54ae to
df0923b
Compare
Member
Author
|
/test |
YutaroHayakawa
approved these changes
Feb 18, 2026
df0923b to
e90a7bc
Compare
Member
Author
|
/test |
e90a7bc to
4b462e1
Compare
Member
Author
|
/test |
4b462e1 to
c26adfd
Compare
The only major change that affected Cilium is in the cilium-cli where helm v4 always requires a WaitStrategy to be set. HookOnlyStrategy waits only for hooks to complete without waiting for chart resources to be ready, which is the closest to preserve the old Wait:false behavior. Signed-off-by: André Martins <andre@cilium.io>
c26adfd to
830aadb
Compare
Member
Author
|
/test |
gandro
approved these changes
Feb 19, 2026
MrFreezeex
approved these changes
Feb 19, 2026
Member
MrFreezeex
left a comment
There was a problem hiding this comment.
Thanks! lgtm for the clustermesh side
Artyop
approved these changes
Feb 19, 2026
YutaroHayakawa
added a commit
that referenced
this pull request
Feb 27, 2026
In the #44390, we upgraded Helm to v4. In v4, they enable k8s SSA by default. This is not the same behavior as before and may cause an unexpected field ownership conflict between `cilium install/upgrade` and user's manual intervation. Since this change is unintended and we haven't prepared for this, I believe disabling it for now would be the safer option. Signed-off-by: Yutaro Hayakawa <yutaro.hayakawa@isovalent.com>
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Mar 2, 2026
In the #44390, we upgraded Helm to v4. In v4, they enable k8s SSA by default. This is not the same behavior as before and may cause an unexpected field ownership conflict between `cilium install/upgrade` and user's manual intervation. Since this change is unintended and we haven't prepared for this, I believe disabling it for now would be the safer option. Signed-off-by: Yutaro Hayakawa <yutaro.hayakawa@isovalent.com>
javiercardona-work
pushed a commit
to javiercardona-work/cilium
that referenced
this pull request
Mar 18, 2026
In the cilium#44390, we upgraded Helm to v4. In v4, they enable k8s SSA by default. This is not the same behavior as before and may cause an unexpected field ownership conflict between `cilium install/upgrade` and user's manual intervation. Since this change is unintended and we haven't prepared for this, I believe disabling it for now would be the safer option. Signed-off-by: Yutaro Hayakawa <yutaro.hayakawa@isovalent.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update helm library to v4.1.1
Signedoff-by: André Martins andre@cilium.io