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

Replace k10-generate-gold-backup-policy ClusterPolicy #998

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mattbator
Copy link
Contributor

Related Issue(s)

N/A

Description

Replaces k10-generate-gold-backup-policy with a version that checks for duplicate policies and can more easily be adapted to environmental needs. Filenames, policy/rule names, category, and keywords have been updated to reflect Kasten K10 rebranding to Veeam Kasten.

Checklist

  • I have read the policy contribution guidelines.
  • I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
  • I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

Signed-off-by: Matt Bator <matt.bator@kasten.io>
Signed-off-by: Matt Bator <matt.bator@kasten.io>
This must not exist. The line is only for the policy itself and not any supporting resources.

Signed-off-by: Chip Zoller <chipzoller@gmail.com>
Copy link
Member

@chipzoller chipzoller left a comment

Choose a reason for hiding this comment

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

One minor thing. I noticed when you had done this to previous samples, it broke our ability to render them to Markdown :)

Comment on lines +21 to +23
Generates a Kasten policy for a namespace that includes any Deployment or StatefulSet with a "dataprotection=kasten-example" label, if the policy does not already exist.

Use with "kasten-data-protection-by-label" policy to require "dataprotection" labeling on workloads.
Copy link
Member

Choose a reason for hiding this comment

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

When using a folded line (block) scalar like this, you can't have an empty line or else it breaks rendering.

Suggested change
Generates a Kasten policy for a namespace that includes any Deployment or StatefulSet with a "dataprotection=kasten-example" label, if the policy does not already exist.
Use with "kasten-data-protection-by-label" policy to require "dataprotection" labeling on workloads.
Generates a Kasten policy for a namespace that includes any Deployment or StatefulSet with a "dataprotection=kasten-example" label, if the policy does not already exist. Use with "kasten-data-protection-by-label" policy to require "dataprotection" labeling on workloads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants