ROX-26901: Add securitypolicies CRD to Konflux operator-bundle#13264
ROX-26901: Add securitypolicies CRD to Konflux operator-bundle#13264mclasmeier merged 1 commit intomasterfrom
Conversation
|
Skipping CI for Draft Pull Request. |
|
Images are ready for the commit at b88ca0c. To use with deploy scripts, first |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #13264 +/- ##
=======================================
Coverage 48.50% 48.50%
=======================================
Files 2468 2468
Lines 178017 178017
=======================================
+ Hits 86345 86348 +3
+ Misses 84738 84736 -2
+ Partials 6934 6933 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
f941a93 to
03ca1fd
Compare
bd03d16 to
b88ca0c
Compare
msugakov
left a comment
There was a problem hiding this comment.
The same is present in release-4.6 branch, therefore approving.
https://github.com/stackrox/stackrox/blob/release-4.6/operator/Makefile#L444
| RUN mkdir -p build/ && \ | ||
| rm -rf build/bundle && \ | ||
| cp -a bundle build/ && \ | ||
| cp -v ../config-controller/config/crd/bases/config.stackrox.io_securitypolicies.yaml build/bundle/manifests/ && \ |
There was a problem hiding this comment.
I must say I'm not a huge fan of the duplication between the Makefile and this dockerfile
We missed this once, we will likely miss something again in the future...
I'll file a ticket to track this.
There was a problem hiding this comment.
There was a problem hiding this comment.
Co-authored-by: Moritz Clasmeier <mclasmeier@redhat.com>
Co-authored-by: Moritz Clasmeier <mclasmeier@redhat.com>
Description
Recently a new CRD was added to operator-bundles. This PR does the same for the Konflux-built bundles.
The same commit is present also in #13154, which is where the resulting artifacts have been inspected to verify that the CRD is included.
User-facing documentation
Testing and quality
Automated testing
None currently.