-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
🌱 Support patch ForceOverwriteConditions option #3643
🌱 Support patch ForceOverwriteConditions option #3643
Conversation
fe947a8
to
5445e2a
Compare
/milestone v0.3.10 |
5445e2a
to
7dda7ee
Compare
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
only a minor nit, not blocking
7dda7ee
to
b005aca
Compare
/lgtm |
Signed-off-by: Vince Prignano <vincepri@vmware.com>
b005aca
to
2ea9aeb
Compare
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
@ncdc Over to you for a final review / approval |
/assign @detiber If you have some time to review / approve :) |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: detiber The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Signed-off-by: Vince Prignano vincepri@vmware.com
What this PR does / why we need it:
This PR adds support for an option in the patch helper that allows controllers to force-overwrite conditions in case of a conflict.
This option should only be used in owning controllers, for example the Machine controller can set it when patching machines. While nothing will stop a different controller from using this option on non-owned objects, this is true as well if folks aren't using our patch helper.
/assign @ncdc @fabriziopandini