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

NAS-134228 / 25.10 / remove ipmi_/sensors.py from public api #15709

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

yocalebo
Copy link
Contributor

This isn't used by anyone except internally for our alert framework. Remove it from public api.

@yocalebo yocalebo requested a review from a team February 14, 2025 18:21
@bugclerk bugclerk changed the title remove ipmi_/sensors.py from public api NAS-134228 / 25.10 / remove ipmi_/sensors.py from public api Feb 14, 2025
@bugclerk
Copy link
Contributor

@truenas truenas deleted a comment from bugclerk Feb 14, 2025
@yocalebo yocalebo requested a review from creatorcary February 14, 2025 18:41
@yocalebo yocalebo merged commit ca85a02 into master Feb 15, 2025
2 checks passed
@yocalebo yocalebo deleted the ipmi-sensors branch February 15, 2025 15:48
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Feb 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants