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

Use long option for quiet cache #224

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

chu11
Copy link
Contributor

@chu11 chu11 commented Dec 5, 2024

Hi, FreeIPMI maintainer here. A user reported a potential issue with the FreeIPMI master branch. I don't use prometheus so this fix is going off code inspection only and is untested.


Problem: The short option -Q for "quiet cache" was legacied in FreeIPMI in early 2018. It has the potential to be removed some day in the future.

Solution: Use the long option --quiet-cache instead.

Problem: The short option -Q for "quiet cache" was legacied in FreeIPMI
in early 2018.  It has the potential to be removed some day in the future.

Solution: Use the long option --quiet-cache instead.

Signed-off-by: Albert Chu <[email protected]>
Copy link
Contributor

@bitfehler bitfehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was blissfully unaware of that deprecation 😇 Thanks a lot!

@bitfehler bitfehler merged commit 1ae273d into prometheus-community:master Dec 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants