karmada-io / karmada

Open, Multi-Cloud, Multi-Cluster Kubernetes Orchestration
https://karmada.io
Apache License 2.0
4.47k stars 885 forks source link

karmada controller print too many "which is not managed by Karmada" log #5781

Open CharlesQQ opened 1 day ago

CharlesQQ commented 1 day ago

What would you like to be added:

Set the level of the following log to level 5

work_status_controller.go:173] Ignore resource(kind=ConfigMap, default/xxxxx) which is not managed by Karmada.

Why is this needed: The karmada member clusters have multiple configmap resources, which are used by the respective controllers of the member clusters to select the master. By default, they will be updated every 10 seconds. The karmada work status controller will detect these configmap changes, but because they are not resources controlled by karmada, The following log will always be printed:

work_status_controller.go:173] Ignore resource(kind=ConfigMap, default/xxxxx) which is not managed by Karmada.
RainbowMango commented 1 day ago

It makes sense to me. The developers may wish to run at V(4), but this log might be verbose. Would you like to send a PR for this?

CharlesQQ commented 1 day ago

Would you like to send a PR for this?

Of course!