fix(controller): add statefulset gc for podgroup.

[HalfBuddhist]

what to fix?

While scheduling a k8s statefulset using volcano, the generated podgroup of the related pods will be in Inqueue state even when setting the replica of the statefulset to zero. And the Inqueued podgroup would still occupy resources, which is not reasonable.

how to reproduce it?

scheduling a statefulset using volcano and scale it to zero.

how to fix in ths pr?

put a informer in controller, delete the pgs for the zero-replica statefulset in the add and the update event.

[volcano-sh-bot]

Welcome @HalfBuddhist!

It looks like this is your first PR to volcano-sh/volcano.

Thank you, and welcome to Volcano. :smiley:

[Monokaix]

Hi, please sign off your commit with git commit -s. And please note that https://github.com/volcano-sh/volcano/pull/3561 has fixed some problems about pg, you can refer it.

[HalfBuddhist]

1. add sign-off to commit message.
2. do the same fix in #3561 when rolling upgrade statefulset.

[Monokaix]

Thanks for your contribution ，this case should also be considered: )

[Monokaix]

Thanks for your contribution ，this case should also be considered: )

Please also consider this: https://github.com/volcano-sh/volcano/pull/3672

[stale[bot]]

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.

[hwdef]

still need

[HalfBuddhist]

Thanks for your contribution ，this case should also be considered: )

Please also consider this: #3672

Done in commit e1c17e9bf7bdeed8491867f9324dcb06ac93eceb @Monokaix

[HalfBuddhist]

/assign @k82cn

[hwdef]

PTAL @JesseStutler @Monokaix @lowang-bh

[HalfBuddhist]

/lgtm Thanks for your contribution.

BTW, It is best to compress the number of commits, 4 are too much. In addition, your commit email does not match the github email address.

Done in commit b4a2ff9a557c24083e8ddbe200d5b694d9bcfd6b @hwdef

[hwdef]

/lgtm

[Monokaix]

/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