LX

Results 383 comments of LX

@wzshiming Hey, Can you fix the CI and resolve the conflicts?

I'm not sure if this change is necessary, or I prefer not to accept it. volcano allows you to modify the scheduler name to implement multiple schedulers in one cluster....

please check this code. https://github.com/volcano-sh/volcano/blob/0c9b846f2e871e6658122fcf2efdf653a1d99946/cmd/scheduler/app/options/options.go#L118C66-L118C86

We can add this field instead of fixing its name

> > We can add this field instead of fixing its name > > set a separate custom flag for helm? Add a field in helm to modify `scheduler-name`

> Is there any update for this PR? The recent server room accident led to the restart of the Volcano scheduler, and other dependencies were blocked due to the Webhook....

> Is there any update for this PR? The recent server room accident led to the restart of the Volcano scheduler, and other dependencies were blocked due to the Webhook....

Overall it's ok, but I prefer `cr` to `configmap` I think we should modify a somewhat annoying legacy problem, that is, if the parsing configuration fails, the default configuration will...

> Overall, we have three approaches to mount scheduling policies: > > * Using a single ConfigMap — This is similar to the implementation in the current proposal. The advantage...