🐛 update AddOnKubeClientRegistrationDriver type

[elgnay]

Summary

Related issue(s)

Fixes #

Summary by CodeRabbit

• Changes
  • AddOnKubeClientRegistrationDriver is now optional in the registration configuration, allowing for more flexible setup scenarios.

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

[coderabbitai]

Walkthrough

Changes to RegistrationConfiguration struct convert the AddOnKubeClientRegistrationDriver field from a value type to a pointer type, with corresponding nil-safety updates to the auto-generated DeepCopyInto method to handle the optional pointer field.

Changes

Cohort / File(s)	Change Summary
Field Type Refactor operator/v1/types_klusterlet.go	Changed AddOnKubeClientRegistrationDriver from AddOnRegistrationDriver (value) to *AddOnRegistrationDriver (pointer), making the field optional
Deep Copy Implementation operator/v1/zz_generated.deepcopy.go	Updated RegistrationConfiguration.DeepCopyInto() to guard pointer copying with nil-check before allocation and deep copy

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

lgtm

Suggested reviewers

• deads2k
• mdelder

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is entirely empty/template only with no summary, explanation of changes, or related issues filled in, making it impossible to understand the context or motivation.	Add a summary explaining why AddOnKubeClientRegistrationDriver was changed to a pointer, its impact, and link to any related issues.
Title check	❓ Inconclusive	The title references the main change (updating AddOnKubeClientRegistrationDriver type) but includes a bug fix emoji and uses non-descriptive phrasing that doesn't clearly explain what was updated or why.	Replace with a more specific title like 'Make AddOnKubeClientRegistrationDriver an optional pointer field' to clearly convey the nature of the type change.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[qiujian16]

/approve
/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: elgnay, qiujian16

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 [qiujian16]

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