[Umbrella] Align generation and observedGeneration fields by reflectstatus and aggregatestatus operation

[yike21]

What would you like to be added:
There are some third-party resources and native resources that need to align .metadata.generation and .status.observedGeneration fields. Please ref to #4866 .

Why is this needed:

Continue the work of #4867 and break down the tasks to be done.

• Native resource
  • GroupVersion: apps/v1
    Kind:
    • Deployment @veophi fix: Deployment generation/observedGeneration bug #4867
    • DaemonSet @whitewindmills
    • StatefulSet @liangyuanpeng
• Thirdparty resource
  • GroupVersion: apps.kruise.io/v1alpha1
    Kind:
    • CloneSet
    • DaemonSet
  • GroupVersion: apps.kruise.io/v1beta1
    Kind:
    • StatefulSet
  • GroupVersion: helm.toolkit.fluxcd.io/v2beta1
    Kind:
    • HelmRelease
  • GroupVersion: kustomize.toolkit.fluxcd.io/v1
    Kind:
    • Kustomization
  • GroupVersion: source.toolkit.fluxcd.io/v1
    Kind:
    • GitRepository
  • GroupVersion: source.toolkit.fluxcd.io/v1beta2
    Kind:
    • Bucket
    • HelmChart
    • HelmRepository
    • OCIRepository

We need to examine the meaning of .metadata.generation and .status.observedGeneration fields for each resource above, and then modify the ReflectStatus and AggregateStatus operations to align them. The modifications are likely to be similar, so please wait for CloneSet and Deployment modifications to be completed.

/help

[karmada-bot]

@yike21:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

What would you like to be added:
There are some third-party resources and native resources that need to align .metadata.generation and .status.observedGeneration fields. Please ref to #4866 .

Why is this needed:

Continue the work of #4867 and break down the tasks to be done.

• native resource
• GroupVersion: apps/v1
  kind:
  • Deployment @veophi fix: Deployment generation/observedGeneration bug #4867
  • DaemonSet
  • StatefulSet
• Thirdparty resource
• GroupVersion: apps.kruise.io/v1alpha1
  kind:
  • CloneSet
  • DaemonSet
• GroupVersion: apps.kruise.io/v1beta1
  kind:
  • StatefulSet
• GroupVersion: helm.toolkit.fluxcd.io/v2beta1
  kind:
  • HelmRelease
• GroupVersion: kustomize.toolkit.fluxcd.io/v1
  kind:
  • Kustomization
• GroupVersion: source.toolkit.fluxcd.io/v1
  kind:
  • GitRepository
• GroupVersion: source.toolkit.fluxcd.io/v1beta2
  kind:
  • Bucket
  • HelmChart
  • HelmRepository
  • OCIRepository

We need to examine the meaning of .metadata.generation and .status.observedGeneration fields for each resource above, and then modify the ReflectStatus and AggregateStatus operations to align them. The modifications are likely to be similar, so please wait for CloneSet and Deployment modifications to be completed.

/help

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.

[liangyuanpeng]

I want to try it for StatefulSet

[yike21]

I want to try it for StatefulSet

Thanks a lot~

[XiShanYongYe-Chang]

@yike21 Good Job!

[whitewindmills]

I wanna pick up DaemonSet, thanks.

[yike21]

I wanna pick up DaemonSet, thanks.

ok, thanks a lot~