🐛 (e2e) fix default-deny-all in list of allowed policies for test #2039
Conversation
Follow up to (2034)[operator-framework#2034] When there is a dual namespace deployment, the default-deny-all policy is duplicated in both namespaces. This PR updates the list of `allowedPolicies` to include both policies if a dual namespace deployment is detected.
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2039 +/- ##
=======================================
Coverage 73.81% 73.81%
=======================================
Files 80 80
Lines 7154 7154
=======================================
Hits 5281 5281
Misses 1550 1550
Partials 323 323
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tmshort 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 |
openshift-ci
bot
added
the
approved
openshift-merge-bot
bot
merged commit d844c45
into
operator-framework:main
2 participants
Follow up to 2034
When there is a dual namespace deployment, the default-deny-all policy is duplicated in both namespaces.
This PR updates the list of
allowedPoliciesto include both policies if a dual namespace deployment is detected.Description
Reviewer Checklist