-
Notifications
You must be signed in to change notification settings - Fork 448
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
Update the kubernetes object's status with server-side apply #2271
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
/help |
@andreyvelich: Please ensure the request meets the requirements listed here. If this request no longer meets these requirements, the label can be removed In 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. |
@andreyvelich I would like to work on this issue. Can you assign it to me ? |
Thank you for your time @aditya7302! |
Can I work on this @andreyvelich ? Now ? |
@tenzen-y can I work on this ? |
Feel free to take this issue with |
Thank you /assign @gjyotin305 |
Hi @tenzen-y could you check it out |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
/remove-lifecycle stale |
/kind feature
Describe the solution you'd like
[A clear and concise description of what you want to happen.]
I'd like to replace the client-side apply with the server-side apply when the controller updates the any statuses like
katib/pkg/controller.v1beta1/experiment/experiment_controller_status.go
Line 27 in fc858d1
Server-side applies allow us to avoid updating errors due to conflicts, and then it would reduce confusions for users.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Love this feature? Give it a 👍 We prioritize the features with the most 👍
The text was updated successfully, but these errors were encountered: