Closed joelanford closed 2 months ago
Name | Link |
---|---|
Latest commit | bb4171feb983afaa56539e4ad548a8b5707f159b |
Latest deploy log | https://app.netlify.com/sites/olmv1/deploys/669a5f69f7ab3400083b9d94 |
Deploy Preview | https://deploy-preview-1072--olmv1.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
The "Verify PR title" test is failing because I have an Issue/PR number in the title. In this case, I think it makes sense for the title to reference the PR that we're reverting, so I'm going to ignore that error.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 72.85%. Comparing base (
95b9f0d
) to head (bb4171f
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This reverts commit 95b9f0dc5943497028deb47969310dec45792a2d.
Description
This reverts the #1038, which breaks the standard install of OLMv1.
It appears that the code from #1038 still requires the OLMv1 system service account to have
* / * / *
permissions (or at least permission to list/watch everything).The e2e tests from that PR pass because they layer
* / * / *
permissions back in for both the OLM SA and the ClusterExtension SA, which means that our SA features are essentially untested.Reviewer Checklist