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

Add documentation for podpreset opt-out annotation #4067

Merged
merged 1 commit into from
Jun 20, 2017

Conversation

jpeeler
Copy link
Contributor

@jpeeler jpeeler commented Jun 13, 2017

The new format simply adds a prefix of "podpreset-" to the current
annotation that records what presets have acted on a pod. The new naming
makes it such that there is no chance of collision with the newly
introduced opt-out annotation (or future ones yet to be added).

Opt-out annotation PR:
kubernetes/kubernetes#44965


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 13, 2017
@jpeeler jpeeler changed the title Update PodPreset modification annotation format Add documentation for podpreset opt-out annotation Jun 13, 2017
@derekwaynecarr
Copy link
Member

@jpeeler -- this pr is needed against release-1.7 branch.

@jpeeler jpeeler changed the base branch from master to release-1.7 June 16, 2017 21:25
@jpeeler jpeeler force-pushed the master branch 2 times, most recently from ed232d1 to a666143 Compare June 16, 2017 21:30

There may be instances where you wish for a pod to not be altered by any pod
preset mutations. For these events, one can add an annotation in the pod spec
of the form: `podpreset.admission.kubernetes.io/PodPresetOptOut: "true"`.
Copy link
Member

Choose a reason for hiding this comment

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

should be podpreset.admission.kubernetes.io/exclude

@derekwaynecarr
Copy link
Member

looks like a rebase is needed

This adds the annotation for having the podpreset admission controller
to skip (opt-out) manipulating the pod spec.

Also, the annotation format for what presets have acted on a pod has
been modified to add a prefix of "podpreset-". The new naming makes it such
that there is no chance of collision with the newly introduced opt-out
annotation (or future ones yet to be added).

Opt-out annotation PR:
kubernetes/kubernetes#44965
@derekwaynecarr
Copy link
Member

LGTM

@chenopis chenopis assigned chenopis and unassigned jessfraz Jun 20, 2017
@chenopis chenopis merged commit c1c6faf into kubernetes:release-1.7 Jun 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants