feat: clear up the orphan mon_groups which are created by cadvisor #3058
Conversation
|
Hi @srl11. Thanks for your PR. I'm waiting for a google member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
04d703e
to
9ff1dd3
Compare
9ff1dd3
to
8e28dba
Compare
|
Why do you keep mon group names? All groups have "cadvisor" prefix. |
All groups have "cadvisor" prefix, but some of them are still working(by cadvisor, for some case it may used by other program). We can use the name to quickly determine whether the mon_group is out of date, related container has exited, and should be removed. |
Revival of #3057, handling the orphan mon_groups.
if resctrl_interval!=0, cache the mon_group created by cadvisor, which has cadvisor monGroupPrefix, and clear up the expired mon_group when a new container is being creating.
@Creatone