-
Notifications
You must be signed in to change notification settings - Fork 199
Add ServerSideApply field to HelmRelease API #1384
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
Conversation
8f0bf3c to
dd4f654
Compare
matheuscscp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing! Thank you so much! 🚀 (just a nit)
|
@cappyzawa Once both comments are fixed we can merge 🙌 |
|
I'll squash the fixup commits once this gets approved. |
matheuscscp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
stefanprodan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @cappyzawa 🏅
Add ServerSideApply field to Install, Upgrade, and Rollback specs
allowing users to control server-side apply behavior per action.
- Install.ServerSideApply: *bool (default based on UseHelm3Defaults)
- Upgrade.ServerSideApply: *string ("true", "false", "auto")
- Rollback.ServerSideApply: *string ("true", "false", "auto")
User-specified values take precedence over defaults. When not
specified, the existing default behavior is preserved.
Signed-off-by: cappyzawa <cappyzawa@gmail.com>
Add end-to-end tests to verify the ServerSideApply field works correctly for install, upgrade, and rollback operations. The tests verify that when serverSideApply is set, the Helm release uses the SSA apply method (apply_method: ssa in the release secret). Signed-off-by: cappyzawa <cappyzawa@gmail.com>
848a73d to
280192a
Compare
|
Squashed the fixups, ready for merge 🙏 |
Resolves: #1381