Add the ClusterTaintPolicy controller to handle taints on the clusters

[XiShanYongYe-Chang]

What type of PR is this?

/kind feature

What this PR does / why we need it:

Add the ClusterTaintPolicy controller to handle taints on the clusters

Which issue(s) this PR fixes:
Part of #6317

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

`karmada-controller-manager`: Added the `clustertaintpolicy` controller to manage cluster taint based on ClusterTaintPolicy.

[XiShanYongYe-Chang]

Waiting for #6319

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 31.57895% with 117 lines in your changes missing coverage. Please review.

Project coverage is 49.27%. Comparing base (5730c74) to head (3a72ee9).
Report is 2 commits behind head on master.

Files with missing lines	Patch %	Lines
...controllers/taint/clustertaintpolicy_controller.go	40.15%	79 Missing ⚠️
pkg/util/helper/taint.go	0.00%	24 Missing ⚠️
cmd/controller-manager/app/controllermanager.go	0.00%	14 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6368      +/-   ##
==========================================
+ Coverage   49.21%   49.27%   +0.06%     
==========================================
  Files         684      686       +2     
  Lines       55327    55833     +506     
==========================================
+ Hits        27228    27514     +286     
- Misses      26326    26521     +195     
- Partials     1773     1798      +25     

Flag	Coverage Δ
unittests	49.27% <31.57%> (+0.06%)	⬆️

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.

[RainbowMango]

Please rebase this

[XiShanYongYe-Chang]

/hold
Under Testing.

[RainbowMango]

/assign

[Copilot]

Pull Request Overview

This PR adds a new ClusterTaintPolicy controller to manage taints on clusters, along with the associated helper functions and controller integrations.

• Introduces a GenerateTaintsMessage helper for a standardized taint description.
• Implements the ClusterTaintPolicy controller with reconciliation logic based on cluster match conditions.
• Updates the cluster controller to use the new helper and integrates the new controller into the controller manager.

Reviewed Changes

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

File	Description
pkg/util/helper/taint.go	Added GenerateTaintsMessage to format cluster taint info
pkg/controllers/taint/clustertaintpolicy_controller.go	New controller to apply taint policies on clusters
pkg/controllers/cluster/cluster_controller.go	Replaced local event message generation with new helper
cmd/controller-manager/app/controllermanager.go	Integrated the new ClusterTaintPolicy controller

[Copilot]

[nitpick] Consider using a strings.Builder for concatenating the taint messages in GenerateTaintsMessage to improve performance and readability when processing a larger number of taints.

Suggested change

	var msg string
	for i, taint := range taints {
	if i != 0 {
	msg += ","
	}
	if taint.Value != "" {
	msg += strings.Join([]string{`{`,
	`Key:` + fmt.Sprintf("%v", taint.Key) + `,`,
	`Value:` + fmt.Sprintf("%v", taint.Value) + `,`,
	`Effect:` + fmt.Sprintf("%v", taint.Effect),
	`}`,
	}, "")
	} else {
	msg += strings.Join([]string{`{`,
	`Key:` + fmt.Sprintf("%v", taint.Key) + `,`,
	`Effect:` + fmt.Sprintf("%v", taint.Effect),
	`}`,
	}, "")
	}
	}
	return fmt.Sprintf("cluster now has taints([%s])", msg)
	var builder strings.Builder
	for i, taint := range taints {
	if i != 0 {
	builder.WriteString(",")
	}
	if taint.Value != "" {
	builder.WriteString("{")
	builder.WriteString("Key:" + fmt.Sprintf("%v", taint.Key) + ",")
	builder.WriteString("Value:" + fmt.Sprintf("%v", taint.Value) + ",")
	builder.WriteString("Effect:" + fmt.Sprintf("%v", taint.Effect))
	builder.WriteString("}")
	} else {
	builder.WriteString("{")
	builder.WriteString("Key:" + fmt.Sprintf("%v", taint.Key) + ",")
	builder.WriteString("Effect:" + fmt.Sprintf("%v", taint.Effect))
	builder.WriteString("}")
	}
	}
	return fmt.Sprintf("cluster now has taints([%s])", builder.String())

[XiShanYongYe-Chang]

Good suggestion, but in this pr I just move the function site, I want to refactor it in a single pr.

[RainbowMango]

/lgtm

[RainbowMango]

/lgtm
/approve

[karmada-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RainbowMango

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:

• cmd/OWNERS [RainbowMango]
• pkg/controllers/OWNERS [RainbowMango]
• pkg/util/OWNERS [RainbowMango]

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

[RainbowMango]

/hold cancel