Enable auto-registration when Rancher Prime is deployed via Helm with registration code

[susesamu]

issue: #125

Fixes automatic SCC registration when Rancher Prime is installed via Helm with registration.enabled=true and registration.regCode set.

Previously, the operator would skip processing the scc-registration Secret created by Helm due to a label ownership conflict (Helm sets app.kubernetes.io/managed-by: Helm, but the operator expected rancher-scc-operator).

[Copilot]

Pull request overview

This PR fixes automatic SCC registration when Rancher Prime is installed via Helm with registration.enabled=true and registration.regCode by adjusting how the operator determines whether it should process the Helm-created scc-registration Secret, avoiding conflicts with Helm’s app.kubernetes.io/managed-by: Helm label.

Changes:

• Preserve (do not overwrite) the existing app.kubernetes.io/managed-by label on Helm-created entrypoint Secrets while still allowing the operator to process them.
• Introduce SCC-specific ownership helpers (scc.cattle.io/managed-by) and associated unit tests.
• Replace maps.Copy label merges in the Secret controller with explicit merge logic.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
pkg/controllers/resources.go	Updates registration param extraction/label production to preserve existing k8s managed-by.
pkg/controllers/helpers/ownership.go	Adds SCC-specific ownership helpers and management decision logic.
pkg/controllers/helpers/ownership_scc_test.go	Adds unit tests for the new SCC-specific ownership helpers.
pkg/controllers/controller.go	Updates Secret reconciliation to use SCC ownership rules for Helm entrypoint secrets and preserve Helm’s managed-by label during label merges.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[mallardduck]

Just a few more nits - great work

[mallardduck]

lgtm, if @alemorcuq also approves we can merge in the AM

[alemorcuq]

LGTM, great work.