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

Explicitly target a deployment or a statefulset #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Explicitly target a deployment or a statefulset #18

wants to merge 1 commit into from

Conversation

aermakov-zalando
Copy link
Contributor

The most common source of invalid PDBs is the users creating a deployment or a statefulset and a cronjob with the same application and then simply using an application=foo selector. We can add a quick workaround that will explicitly target pods belonging to a deployment or a statefulset to avoid this situation.

TODO: tests.

The most common source of invalid PDBs is the users creating a
deployment or a statefulset and a cronjob with the same application and
then simply using an application=foo selector. We can add a quick
workaround that will explicitly target pods belonging to a deployment or
a statefulset to avoid this situation.

Signed-off-by: Alexey Ermakov <[email protected]>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 37.993% when pulling 527ce9f on aermakov-zalando:avoid-cronjobs into 4c8af52 on mikkeloscar:master.

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.

2 participants