🐛 Prevent showing duplicate entry under .status.activeRevisions

[pedjak]

Description

Updating ClusterExtension with duplicate entry under .status.activeRevisions fails.
Thus, we repopulate it from the installed and rolling out revisions.

Reviewer Checklist

• API Go Documentation
• Tests: Unit Tests (and E2E Tests, if appropriate)
• Comprehensive Commit Messages
• Links to related GitHub Issue(s)

[netlify]

✅ Deploy Preview for olmv1 ready!

Name	Link
🔨 Latest commit	489a9e6
🔍 Latest deploy log	https://app.netlify.com/projects/olmv1/deploys/69669574b2629a000822544c
😎 Deploy Preview	https://deploy-preview-2444--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 project configuration.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.41%. Comparing base (1fa4169) to head (489a9e6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2444      +/-   ##
==========================================
- Coverage   73.00%   69.41%   -3.59%     
==========================================
  Files         100      101       +1     
  Lines        7641     7719      +78     
==========================================
- Hits         5578     5358     -220     
- Misses       1625     1924     +299     
+ Partials      438      437       -1     

Flag	Coverage Δ
e2e	46.23% <0.00%> (-0.63%)	⬇️
experimental-e2e	13.52% <0.00%> (-35.17%)	⬇️
unit	57.04% <100.00%> (-0.08%)	⬇️

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.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[tmshort]

/lgtm

[tmshort]

/approve

[grokspawn]

This codecov result is insane, considering that there is one line of modified code, with an associated test. I wonder if this is a grandfathered deficit which we see now because of the change in the module.

[rashmigottipati]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: grokspawn, rashmigottipati, tmshort

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [grokspawn,tmshort]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[pedjak]

/override codecov/project

[openshift-ci]

@pedjak: Overrode contexts on behalf of pedjak: codecov/project

Details

In response to this:

/override codecov/project

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.

[grokspawn]

/override codecov/project

@pedjak this was not a required test, so it didn't block. Unfortunately, the /override command does not override GH branch protection required tests.