Implement ResourceInterpreterCustomization ValidatingAdmission

[chaunceyjiang]

Signed-off-by: chaunceyjiang chaunceyjiang@gmail.com

What type of PR is this?
/kind feature

What this PR does / why we need it:
Prevent modifying and creating ResourceInterpreterCustomization using the same interpretation rules.

Which issue(s) this PR fixes:

Part of #2371

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Prevent modifying and creating `ResourceInterpreterCustomization` using the same interpretation rules.

[codecov-commenter]

Codecov Report

Merging #2755 (74a2887) into master (31f97ac) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2755      +/-   ##
==========================================
+ Coverage   33.46%   33.47%   +0.01%     
==========================================
  Files         200      200              
  Lines       19636    19636              
==========================================
+ Hits         6572     6574       +2     
+ Misses      12669    12668       -1     
+ Partials      395      394       -1     

Flag	Coverage Δ
unittests	33.47% <ø> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/search/proxy/store/util.go	92.98% <0.00%> (-0.59%)	⬇️
pkg/util/worker.go	71.42% <0.00%> (+4.76%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[XiShanYongYe-Chang]

Thanks a lot~
A change to a file may have been missed: pkg/karmadactl/cmdinit/karmada/webhook_configuration.go

[chaunceyjiang]

This judgement may conflict with what was said at today's community meeting: InterpretDependency operation can support multiple different script definitions.

Yes, let's pending this PR first, and we'll deal with this PR after the community has a clearer description of the above issue.

[XiShanYongYe-Chang]

/assign

[ikaven1024]

Shall we check the script in the webhook? Like

karmada/pkg/karmadactl/interpret/check.go

Lines 83 to 93 in 63a67d7

	func checkScrip(script string) error {
	ctx, cancel := context.WithTimeout(context.TODO(), time.Second)
	defer cancel()
	l, err := luavm.NewWithContext(ctx)
	if err != nil {
	return err
	}
	defer l.Close()
	_, err = l.LoadString(script)
	return err
	}

[XiShanYongYe-Chang]

Shall we check the script in the webhook?

+1 I agree with this point.

[RainbowMango]

/assign

[XiShanYongYe-Chang]

Thanks~
other parts lgtm

[XiShanYongYe-Chang]

Thanks!
/cc @RainbowMango @ikaven1024

[ikaven1024]

Other LGTM

[RainbowMango]

/approve

Leave LGTM to @XiShanYongYe-Chang @ikaven1024

[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:

• OWNERS [RainbowMango]

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

[XiShanYongYe-Chang]

/lgtm