-
Notifications
You must be signed in to change notification settings - Fork 47
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
✨ Move the e2e into the operator-controler-system ns #927
Conversation
To allow access to the docker-registry certificate, move the e2e resources into the same namespace as the operator-controller-system namespace. Signed-off-by: Todd Short <tshort@redhat.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #927 +/- ##
=======================================
Coverage 77.38% 77.38%
=======================================
Files 17 17
Lines 1163 1163
=======================================
Hits 900 900
Misses 183 183
Partials 80 80
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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
/approve
…framework#927)" This ends up overwriting the security policy of the operator-controller-system namespace, which really isn't something we want to do in a valid e2e test. Many of the e2e components need to run as root in order to do the things they need to do. This reverts commit aa48e70.
) This ends up overwriting the security policy of the operator-controller-system namespace, which really isn't something we want to do in a valid e2e test. Many of the e2e components need to run as root in order to do the things they need to do. This reverts commit aa48e70.
To allow access to the docker-registry certificate, move the e2e resources into the same namespace as the operator-controller-system namespace.
Description
Reviewer Checklist