configure the fieldmanager of the toolchainclusterresources controller

[metlos]

update to the new toolchain-common and configure the fieldmanager of the
toolchainclusterresources controller.

Related PRs:

• replace the last use of the old apply client with the SSA client toolchain-common#499
• configure the field owner for the toolchainclusterresources controller that now uses SSA host-operator#1215

Summary by CodeRabbit

• Chores
  • Updated dependencies to the latest version for improved stability.
  • Refined internal reconciliation configuration for better resource management.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The PR updates the ToolchainClusterResources reconciler initialization in cmd/main.go to set the new FieldManager field to a constant from the constants package, and bumps the toolchain-common dependency to a newer version.

Changes

Cohort / File(s)	Summary
Reconciler Configuration cmd/main.go	Added import of constants package and configured FieldManager field for ToolchainClusterResources reconciler with constants.MemberOperatorFieldManager value.
Dependency Update go.mod	Bumped github.com/codeready-toolchain/toolchain-common from v0.0.0-20251103123811-4066619e5740 to v0.0.0-20251125180949-6ed5aeed75e4.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

• Straightforward configuration change with a single field assignment
• Routine dependency version bump with no breaking changes indicated

Possibly related PRs

• Update API & toolchain-common (to check removal of che config) #705: Shares the same toolchain-common dependency bump across repositories.

Suggested labels

approved

Suggested reviewers

• xcoulon
• rajivnathan
• MatousJobanek

Poem

🐰 A field manager arrives, so neat,
Constants guide with steady beat,
Dependencies refresh and dance,
Reconcilers now take their chance!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and clearly summarizes the main change: configuring the FieldManager for the ToolchainClusterResources controller, which is directly reflected in the code modifications.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 61ed5ca and 1cf1f28.

⛔ Files ignored due to path filters (1)

• go.sum is excluded by !**/*.sum

📒 Files selected for processing (2)

• cmd/main.go (2 hunks)
• go.mod (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

cmd/main.go (2)

deploy/resources.go (1)

• ToolchainClusterTemplateFS (6-6)

pkg/constants/field_manager.go (1)

• MemberOperatorFieldManager (3-3)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: GolangCI Lint
• GitHub Check: Build & push operator bundles for e2e tests
• GitHub Check: Test with Coverage

🔇 Additional comments (3)

cmd/main.go (2)

10-10: LGTM! Import added correctly.

The constants package import is properly placed and necessary for accessing MemberOperatorFieldManager.

---

227-232: LGTM! FieldManager configured correctly.

The ToolchainClusterResources reconciler is properly initialized with the new FieldManager field set to constants.MemberOperatorFieldManager ("kubesaw-member-operator"). The multi-line formatting also improves readability.

go.mod (1)

5-5: Manual verification required in CI environment; no known security advisories found for this version.

The pseudo-version v0.0.0-20251125180949-6ed5aeed75e4 follows valid Go versioning syntax and shows no published security advisories in public databases. However, definitive verification that this version includes the FieldManager field and is free from vulnerabilities requires running govulncheck ./... in your CI environment, as sandbox restrictions prevent direct dependency inspection. Ensure the version hash (6ed5aeed75e4) corresponds to an expected commit in the codeready-toolchain/toolchain-common repository before merging.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.53%. Comparing base (91ef108) to head (f2f7196).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #713   +/-   ##
=======================================
  Coverage   82.53%   82.53%           
=======================================
  Files          48       48           
  Lines        3596     3596           
=======================================
  Hits         2968     2968           
  Misses        477      477           
  Partials      151      151           

🚀 New features to boost your workflow:

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

[rsoaresd]

🚀

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexeykazakov, MatousJobanek, metlos, rsoaresd

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 [MatousJobanek,alexeykazakov,metlos,rsoaresd]

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud