Skip to content

pkg/health: migrate to slog #39465

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

Merged
merged 1 commit into from
May 21, 2025
Merged

pkg/health: migrate to slog #39465

merged 1 commit into from
May 21, 2025

Conversation

aanm
Copy link
Member

@aanm aanm commented May 9, 2025

Migrate this package to use slog instead of logrus.

@aanm aanm added area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. area/health Relates to the cilium-health component release-note/misc This PR makes changes that have no direct user impact. feature/slog labels May 9, 2025
@aanm
Copy link
Member Author

aanm commented May 9, 2025

/test

@aanm aanm force-pushed the pr/move-health-to-slog branch from f0d63e1 to 5ec631f Compare May 9, 2025 15:40
@aanm
Copy link
Member Author

aanm commented May 9, 2025

/test

@aanm aanm force-pushed the pr/move-health-to-slog branch from 5ec631f to 8d05c7b Compare May 12, 2025 07:35
@aanm
Copy link
Member Author

aanm commented May 12, 2025

/test

@aanm aanm marked this pull request as ready for review May 12, 2025 07:35
@aanm aanm requested review from a team as code owners May 12, 2025 07:35
@aanm aanm requested a review from squeed May 12, 2025 07:35
@aanm aanm enabled auto-merge May 12, 2025 09:46
@aanm aanm force-pushed the pr/move-health-to-slog branch from 8d05c7b to a3f7a3b Compare May 15, 2025 09:29
@aanm
Copy link
Member Author

aanm commented May 15, 2025

/test

Migrate this package to use slog instead of logrus.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/move-health-to-slog branch from a3f7a3b to e0eb89a Compare May 21, 2025 10:29
@aanm
Copy link
Member Author

aanm commented May 21, 2025

/test

@aanm aanm disabled auto-merge May 21, 2025 12:42
@aanm aanm merged commit 0426976 into main May 21, 2025
335 of 339 checks passed
@aanm aanm deleted the pr/move-health-to-slog branch May 21, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/health Relates to the cilium-health component area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. feature/slog release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants