add scheduler-name field in charts

[lengrongfu]

add scheduler-name field in chart file to configuration custome scheduler name

From: #3748 (comment)

[lengrongfu]

/cc @hwdef @Monokaix

[hwdef]

Sorry I didn't express myself clearly,
What I mean is that parameters are added only when schedulerName is set by helm.
So, you code should like this

volcano/installer/helm/chart/volcano/templates/controllers.yaml

Line 11 in 7dfa8c6

{{- if .Values.custom.common_labels }}

[lengrongfu]

Sorry I didn't express myself clearly, What I mean is that parameters are added only when schedulerName is set by helm. So, you code should like this

volcano/installer/helm/chart/volcano/templates/controllers.yaml

Line 11 in 7dfa8c6

{{- if .Values.custom.common_labels }}

Oh, then i know, thanks~

[hwdef]

The code can be optimized again.
You can add code like this,
In this way, there will be no changes in volcano-development.yaml when schedulerName is not set.

volcano/installer/helm/chart/volcano/templates/admission.yaml

Line 5 in 7dfa8c6

{{ $admission_ns := or .Values.custom.admission_ns .Values.custom.default_ns }}

[hwdef]

/lgtm

[Monokaix]

The webhook modification will be done at your another pr?

[lengrongfu]

The webhook modification will be done at your another pr?

I think so, try to keep the functional independence of each PR.

[Monokaix]

/ok-to-test
/approve

[volcano-sh-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Monokaix

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

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