Skip to content

feat: add --ignore-annotation flag to override reloader.stakater.com/ignore#1146

Open
riptide-01 wants to merge 1 commit into
stakater:masterfrom
riptide-01:feat/ignore-annotation-flag
Open

feat: add --ignore-annotation flag to override reloader.stakater.com/ignore#1146
riptide-01 wants to merge 1 commit into
stakater:masterfrom
riptide-01:feat/ignore-annotation-flag

Conversation

@riptide-01
Copy link
Copy Markdown

Allow users to customise the annotation key used to mark ConfigMaps and Secrets as ignored by Reloader, consistent with how every other annotation key is already overridable via a dedicated CLI flag.

Changes:

  • internal/pkg/util/util.go: register --ignore-annotation persistent flag backed by options.IgnoreResourceAnnotation
  • deployments/kubernetes/chart/reloader/templates/deployment.yaml: pass --ignore-annotation when custom_annotations.ignore is set in Helm values
  • deployments/kubernetes/chart/reloader/values.yaml: document the new custom_annotations.ignore key in the example comment
  • README.md: add --ignore-annotation to the Annotation Key Overrides table and mention the flag alongside the Resource-Level Ignore Annotation section

…ignore

Allow users to customise the annotation key used to mark ConfigMaps and
Secrets as ignored by Reloader, consistent with how every other annotation
key is already overridable via a dedicated CLI flag.

Changes:
- internal/pkg/util/util.go: register --ignore-annotation persistent flag
  backed by options.IgnoreResourceAnnotation
- deployments/kubernetes/chart/reloader/templates/deployment.yaml: pass
  --ignore-annotation when custom_annotations.ignore is set in Helm values
- deployments/kubernetes/chart/reloader/values.yaml: document the new
  custom_annotations.ignore key in the example comment
- README.md: add --ignore-annotation to the Annotation Key Overrides table
  and mention the flag alongside the Resource-Level Ignore Annotation section

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant