-
Notifications
You must be signed in to change notification settings - Fork 30
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
Verify operator CSV images are from allowed registries #1092
Conversation
e995c8f
to
09b805a
Compare
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.
LGTM, one typo
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1092 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 113 113
Lines 5621 5672 +51
=========================================
+ Hits 5621 5672 +51
|
09b805a
to
dd2064e
Compare
@nmars, one more thing. We require signed commits in this repo. |
Head branch was pushed to by a user without write access
dd2064e
to
d44d7cb
Compare
@lcarva signed commit added. |
/ok-to-test Edit: lol, looks like I don't have the power to approve workflows |
Thanks for the PR! |
This commit adds a rule that verifies that images from operator bundle CSVs are hosted on approved registries.
resolves: CVP-4149