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

Added missing option 'audit-match-kind-only' to helm #1245

Merged
merged 1 commit into from
Apr 13, 2021
Merged

Added missing option 'audit-match-kind-only' to helm #1245

merged 1 commit into from
Apr 13, 2021

Conversation

e96wic
Copy link
Contributor

@e96wic e96wic commented Apr 13, 2021

Added missing option 'audit-match-kind-only' to helm:

This option was missing from the helm chart before. I found it in the docs: https://open-policy-agent.github.io/gatekeeper/website/docs/audit#audit-using-kinds-specified-in-the-constraints-only

Special notes for your reviewer:

Signed-off-by: Eike Wichern <13048266+e96wic@users.noreply.github.com>
Copy link
Member

@sozercan sozercan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

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

LGTM

@maxsmythe maxsmythe merged commit 4b4ae44 into open-policy-agent:master Apr 13, 2021
@DrackThor
Copy link

When is this feature planned to be available in the released HELM chart?
Currently I only see it in the 'manifest_staging' chart.

@maxsmythe
Copy link
Contributor

@sozercan @ritazh @shomron Time to cut a release? Now that we should be k8s v1.22-compatible? Wait on creating v1 constraints/templates as well?

@sozercan
Copy link
Member

Added to community call agenda for quick sync tomorrow.

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.

4 participants