Resolve podgroup residual
background
Lack of external modules to manage PodGroup status based on workload status, making Volcano unable to perceive the lifecycle of upper level workloads (success/failure/pause/termination/scaling, etc.) and unable to manage PodGroup correctly. The following scenario will appear: - a. In scenarios where tasks (such as Pod/Job) run successfully/fail, the corresponding PodGroup remains and occupies the queue quota. - b. In scenarios where tasks (such as Deployment/StatefulSet) are scaled down to 0, the corresponding PodGroup remains and occupies the queue quota
Solution ideas
- When pod is in its final state:succeed、failed, clear the podgroup corresponding to pod。If pod has a controller owner reference, the check will be skipped
- When the number of statefulset replicas is 0, clear the podgroup corresponding to the statefulset
- When the job is in a paused state, clear the podgroup corresponding to the job (support for job pause operation is required)
Welcome @HDJZX!
It looks like this is your first PR to volcano-sh/volcano 馃帀.
Thank you, and welcome to Volcano. :smiley:
Hi, welcome. Please add some backgroud information to explain why this is needed.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign wpeng102
You can assign the PR to them by writing /assign @wpeng102 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
Update background and solutions https://github.com/volcano-sh/volcano/pull/3182#issue-1980922569
Please squash your commits and make CI happy, lgtm except this.
I mean merge to one commit and git commit with -s : )
ok, done.
/lgtm
@Monokaix can you review again ? I remove unused dependencies from unit testing
/ok-to-test
@Monokaix Sorry,I seem unable to trigger CI.
Thansk for your contribution, the CI is triggered.
@Monokaix Can you help trigger it once,Thanks
@Monokaix Can you help trigger it once,Thanks
sorry, I have no tright to trigger ci, please help @william-wang
@william-wang Can you help trigger it once,Thanks
@Monokaix Can you review it? lgtm can also trigger ci,thanks
/lgtm
Hi, is there any problem here?
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.
@HDJZX please rebase and squash commits to only one.
/needs-rebase
/lgtm