Skip to content
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

feat: allow specifying tolerations for registry-proxy #1296

Merged
merged 1 commit into from
Oct 31, 2019
Merged

Conversation

eysi09
Copy link
Collaborator

@eysi09 eysi09 commented Oct 31, 2019

What this PR does / why we need it:

Enables users to set tolerations for the registry-proxy DaemonSet. From the docs:

For setting tolerations on the registry-proxy when using in-cluster building.
The registry-proxy is a DaemonSet that proxies connections to the docker registry service on each node.

Use this only if you're doing in-cluster building and the nodes in your cluster
have taints.

Which issue(s) this PR fixes:

Fixes #1279

Special notes for your reviewer:

The description for the individual toleration fields and the validation rules are derived from the K8s API reference.

Copy link
Collaborator

@edvald edvald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One super minor comment, otherwise looks great!

.default("Equal")
.description(dedent`
Operator represents a key's relationship to the value. Valid operators are Exists and Equal. Defaults to
Equal. Exists is equivalent to wildcard for value, so that a pod can tolerate all taints of a
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to have quotes around the value names.

@edvald edvald merged commit 1fc8310 into master Oct 31, 2019
@edvald edvald deleted the tolerations branch October 31, 2019 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No way to add tolerations to garden-registry-proxy
2 participants