Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated object metrics from controllers #5083

Open
5 tasks
darkowlzz opened this issue Nov 21, 2024 · 1 comment
Open
5 tasks

Remove deprecated object metrics from controllers #5083

darkowlzz opened this issue Nov 21, 2024 · 1 comment
Labels
area/monitoring Monitoring related issues and pull requests good first issue Good for newcomers umbrella-issue Umbrella issue for tracking progress of a larger effort

Comments

@darkowlzz
Copy link
Contributor

darkowlzz commented Nov 21, 2024

In Flux v2.1, the object metrics (suspend and readiness) were deprecated from the flux controllers. See the deprecation notice in the metrics docs. See #4128 for detailed motivation and the recommended metrics collection strategy.

Since then, the newly refactored controllers have removed the deprecated metrics: helm-controller and image-automation-controller.
The same should now be removed from the rest of the controllers.

The following lists the controllers that should remove the deprecated metrics, it also includes the reconcilers that record these metrics with reference links to a few of them in their respective controller repositories:

  • source-controller (for GitRepository, Bucket, HelmRepository, HelmChart and OCIRepository)
  • kustomize-controller (for Kustomization)
  • image-reflector-controller (for both ImageRepository and ImagePolicy)
  • notification-controller (for Receiver only)

Following the above changes, the metrics docs page should be updated accordingly:

  • Update metrics docs to remove the deprecation notice and alert at the top of the page (metrics docs source)
@darkowlzz darkowlzz added good first issue Good for newcomers area/monitoring Monitoring related issues and pull requests labels Nov 21, 2024
@stefanprodan
Copy link
Member

To clarify, these are the deprecated metrics that are going to be removed:

  • gotk_reconcile_condition
  • gotk_suspend_status

@stefanprodan stefanprodan added the umbrella-issue Umbrella issue for tracking progress of a larger effort label Nov 21, 2024
hasithsen added a commit to hasithsen/source-controller that referenced this issue Dec 8, 2024
hasithsen added a commit to hasithsen/notification-controller that referenced this issue Dec 8, 2024
hasithsen added a commit to hasithsen/kustomize-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/image-reflector-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/image-reflector-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/kustomize-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/source-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/notification-controller that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/website that referenced this issue Dec 9, 2024
hasithsen added a commit to hasithsen/kustomize-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/kustomize-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/kustomize-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/source-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/notification-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/image-reflector-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/image-reflector-controller that referenced this issue Dec 11, 2024
hasithsen added a commit to hasithsen/image-reflector-controller that referenced this issue Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/monitoring Monitoring related issues and pull requests good first issue Good for newcomers umbrella-issue Umbrella issue for tracking progress of a larger effort
Projects
None yet
Development

No branches or pull requests

2 participants