volcano
volcano copied to clipboard
support nodegroup as a plugin
issue #1830
related pr #2224
design doc #2227
queue.spec.Affinity
is ready by https://github.com/volcano-sh/apis/pull/72 and merged in master branch by #2319, so it is time to push the feature.
Should add some code for testing?
@qiankunli What's the model relationship between queue and nodegroup. Do we allow to configure two queues that affinity the same nodegroup?
@qiankunli please help to add the UT, e2e :)
What's the model relationship between queue and nodegroup.
queue and nodegroup are many-to-many relationships
Do we allow to configure two queues that affinity the same nodegroup?
yes
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign qiankunli
You can assign the PR to them by writing /assign @qiankunli
in a comment when ready.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
Hi, How long is it available?
@qiankunli please make the ci happy.
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
still need /reopen
@hwdef: Reopened this PR.
In response to this:
still need /reopen
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: To complete the pull request process, please ask for approval from qiankunli after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
any process?
any process? +1
any process
how to calculate the deserved resource in proportion plugin when hard bind queue and node? @qiankunli
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
Does this 1.8.0 version come with this plugin
This PR is replaced with a new PR #3132, so we close it.