🌱 Revert deprecation of client.Apply#3307
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbueringer The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/cherry-pick release-0.22 |
|
@sbueringer: once the present PR merges, I will cherry-pick it on top of DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Signed-off-by: Stefan Büringer buringerst@vmware.com
305b585 to
6e1e8b2
Compare
erikgb
left a comment
There was a problem hiding this comment.
I think this is a good change, considering there is currently no migration path for all use of the deprecated patch type.
/lgtm
|
LGTM label has been added. DetailsGit tree hash: 66a76f66ea16d23e0fbf47a64fe2c5258b95c432 |
|
@sbueringer: new pull request created: #3308 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Ups forgot to hold, @alvaroaleman fine for you as well? |
Signed-off-by: Stefan Büringer buringerst@vmware.com
Related to #3183