stackrox icon indicating copy to clipboard operation
stackrox copied to clipboard

ROX-26151: Pin operator image in operator-bundle (3rd)

Open mclasmeier opened this issue 1 year ago • 4 comments

Description

This PR changes the existing pipeline for the operator-bundle so that instead of injecting the operator-image tag into the CSV file we resolve this tag to an image digest and then inject that into the CSV.

Note: This is a simplified PR: there are no checks about the actual availability of the "replaced" version (think: "previous") of the operator.

User-facing documentation

  • [x] CHANGELOG update is not needed
  • [x] documentation PR is not needed

Testing and quality

  • [ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
  • [x] CI results are inspected

Automated testing

I have not added any new tests.

How I validated my change

mclasmeier avatar Sep 24 '24 14:09 mclasmeier

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] avatar Sep 24 '24 14:09 openshift-ci[bot]

Images are ready for the commit at 3842086.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.6.x-565-g384208658a.

rhacs-bot avatar Sep 24 '24 14:09 rhacs-bot

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.13%. Comparing base (9d0d5f5) to head (7d3a32d). Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12797      +/-   ##
==========================================
- Coverage   48.14%   48.13%   -0.01%     
==========================================
  Files        2438     2438              
  Lines      174848   174915      +67     
==========================================
+ Hits        84184    84202      +18     
- Misses      83869    83916      +47     
- Partials     6795     6797       +2     
Flag Coverage Δ
go-unit-tests 48.13% <ø> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 24 '24 15:09 codecov[bot]

FYI, @porridge, pushed two commits.

mclasmeier avatar Sep 30 '24 08:09 mclasmeier

@mclasmeier: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/gke-sensor-integration-tests 384208658a9d263c5b3f44cc9dfcba55bb00a89d link false /test gke-sensor-integration-tests

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Sep 30 '24 09:09 openshift-ci[bot]