✨Operator support MultipleHubs.

[xuezhaojun]

Summary

The PR is to support MultiHubs also in the operator.

The PR is related to: #443

---

Some conversations that need highlight:

• ✨Operator support MultipleHubs. #524 (comment): about why and how to let user know the connectivity status of bootstrapkuebeconfigs.
• ✨Operator support MultipleHubs. #524 (comment): Some controllers skip the reconcile when "hubAcceptsClient" is false. For example, if the hubAcceptsClient is false, the Avaiablie status of a managedcluster don't get update.

[codecov]

Codecov Report

Attention: Patch coverage is 79.16667% with 10 lines in your changes missing coverage. Please review.

Project coverage is 62.40%. Comparing base (a14450a) to head (7b61bec).
Report is 1 commits behind head on main.

Files	Patch %	Lines
...lers/klusterletcontroller/klusterlet_controller.go	46.15%	6 Missing and 1 partial ⚠️
...llers/ssarcontroller/klusterlet_ssar_controller.go	87.50%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #524      +/-   ##
==========================================
- Coverage   62.44%   62.40%   -0.05%     
==========================================
  Files         138      138              
  Lines       11610    11605       -5     
==========================================
- Hits         7250     7242       -8     
- Misses       3593     3597       +4     
+ Partials      767      766       -1     

Flag	Coverage Δ
unit	62.40% <79.16%> (-0.05%)	⬇️

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.

[xuezhaojun]

/assign @qiujian16

[xuezhaojun]

/assign @elgnay

[qiujian16]

this looks like a bug cc @elgnay

[elgnay]

• the lease contoller only watch managedcluster resource

The lease contoller watches both the managedcluster and the lease;

• the lease contoller only watch managedcluster resource, no periodical check on the managedcluster lease

The the managed cluster will be requeued after LeaseDurationSeconds.

• if the hubAcceptsClient is false, the controller will skip the reconciliation

That's true. The current logic of the controller does not handle hubAcceptsClient==false very well. Some other controllers may have the similar issue.

[xuezhaojun]

@qiujian16 The e2e part is removed and will join the refactor PR, the intergration test added, please take another look, thanks!

[qiujian16]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

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

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

The pull request process is described here

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

[qiujian16]

/lgtm