chore: bump helm-operator fork to new rebase

[porridge]

Description

Our changes to helm-operator were rebased on top of the most recent release. This change switches to using this code.

This also happens to bump the k8s libraries.

In this repo, one minor change to a function was needed due to a slightly incompatible controller-runtime API change.

In the helm-operator repo, there were more changes needed due to more strict lint checking. Please review them in the context of this PR.

Most differences are due to differing context. Actual changes to the code made during the rebase were in patches:

• 4 - changed spelling of Failed to Failds in a few places to reduce the constant reformatting churn - when upstreaming we'll need to change this to correct spelling, but for now it's just a PITA; nolint:unparam added in handlePending to silence overzealous "first value returned is always nil" warning; error message formatting due to linter complaints
• 5+11 - merged into a single patch, fixed ginkgo linter complaint
• 7 - removed unused watchDescription
• 10, 12 - fixed ginkgo linter complaint

Also a round of make -C operator generate manifests bundle was needed.

Yet another change was needed to compliance operator test do deal with a breaking change in controller runtime.

User-facing documentation

• CHANGELOG is updated OR update is not needed
• documentation PR is created and is linked above OR 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
• CI results are inspected

Automated testing

• no changes to tests needed

How I validated my change

CI is enough.

[openshift-ci]

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

[rhacs-bot]

Images are ready for the commit at 4721599.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.7.x-724-g47215999fe.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.08%. Comparing base (2be1cc8) to head (bff1abc).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #14233   +/-   ##
=======================================
  Coverage   49.08%   49.08%           
=======================================
  Files        2514     2514           
  Lines      182769   182769           
=======================================
+ Hits        89705    89707    +2     
+ Misses      85944    85943    -1     
+ Partials     7120     7119    -1     

Flag	Coverage Δ
go-unit-tests	49.08% <ø> (+<0.01%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[porridge]

/test all

[rhacs-bot]

/retest

[rhacs-bot]

/retest

[rhacs-bot]

/retest

[rhacs-bot]

Images are ready for the commit at 36821dd.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.7.x-733-g36821dd1ba.

[porridge]

Looks like the test is passing but the cluster destroy is failing.

INFO[2025-02-12T11:10:35Z] Running step operator-e2e-tests-stackrox-stackrox-e2e-test. 
INFO[2025-02-12T11:49:37Z] Step operator-e2e-tests-stackrox-stackrox-e2e-test succeeded after 39m1s. 

[rhacs-bot]

Images are ready for the commit at bff1abc.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.8.x-4-gbff1abc21e.