-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Document the safe-to-evict annotation on the webhook deployment #3961
Conversation
/approve Just a small syntax nit |
The annotation is set by default because by default we assume Tekton will be installed in a small cluster with only one replica of the webhook deployment, and that replica should not be disturbed. If an operator configures multiple replicas they can remove the annotation to give a Cluster Autoscaler more freedom to scale down nodes.
/retest |
/lgtm |
@nikhil-thomas: changing LGTM is restricted to collaborators 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/test-infra repository. |
/lgtm |
Switching my lgtm to an approve. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pierretasci, sbwsg 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 |
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
Release Notes