-
Notifications
You must be signed in to change notification settings - Fork 37
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
adding namespace-has-resourcequota policy #101
adding namespace-has-resourcequota policy #101
Conversation
/assign @etsauer |
@deweya ; can you:
that should fix the CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just needs CI fixing.
8196784
to
539e18f
Compare
539e18f
to
7fc86d3
Compare
@garethahealy Fixed the CI. That policy regen is cool stuff |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deweya, garethahealy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What is this PR About?
Adds
namespace-has-resourcequota
. Closes #55Check list
_test/conftest-unittests.sh
_test/opa-profile.sh
_test/gatekeeper-integrationtests.sh
POLICIES.md
The other namespace-has-* policy didn't have a test case in _test/opa-profile.sh or _test/gatekeeper-integrationtests.sh, so I didn't add tests there since this one is similar.
cc: @redhat-cop/rego-policies