Skip to content

INFRA-709 Rated dwpd alerts #1077

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 6 commits into from
Jun 13, 2025
Merged

INFRA-709 Rated dwpd alerts #1077

merged 6 commits into from
Jun 13, 2025

Conversation

technowhizz
Copy link
Contributor

This PR allows the operator to specify the rated DWPD of a disk and have it show up in prometheus

@technowhizz technowhizz requested a review from jovial May 16, 2024 17:40
@technowhizz technowhizz self-assigned this May 16, 2024
@technowhizz technowhizz requested a review from a team as a code owner May 16, 2024 17:40
@technowhizz technowhizz requested a review from jovial June 12, 2024 14:28
Copy link
Member

@dougszumski dougszumski left a comment

Choose a reason for hiding this comment

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

Excellent piece of work @technowhizz . Please see if you can simplify it by writing out the DWPD file separately. Unless I've missed something?

@Alex-Welsh
Copy link
Member

@technowhizz what do we need to do to get this moving again? It looks like a lot of work and I don't want it to go to waste

@dougszumski
Copy link
Member

@technowhizz what do we need to do to get this moving again? It looks like a lot of work and I don't want it to go to waste

Now we have started on the Python script it makes sense to finish that first and then come back to this.

@Alex-Welsh Alex-Welsh added enhancement New feature or request antelope Targets the Antelope OpenStack release monitoring All things related to observability & telemetry labels Nov 15, 2024
@technowhizz technowhizz changed the title Rated dwpd alerts INFRA-709 Rated dwpd alerts Dec 13, 2024
@technowhizz technowhizz changed the base branch from stackhpc/2023.1 to stackhpc/2024.1 January 24, 2025 11:19
@technowhizz
Copy link
Contributor Author

I need to update the release note

@Alex-Welsh
Copy link
Member

@technowhizz could you resolve the linter issues so we can try and get this merged? It's been stuck for a while

@technowhizz technowhizz force-pushed the rated-dwpd-alerts branch 7 times, most recently from 3476cc7 to 185ba2d Compare May 12, 2025 23:35
Copy link
Member

@dougszumski dougszumski left a comment

Choose a reason for hiding this comment

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

This needs updating in another PR, but should work as-is, for now.

@dougszumski dougszumski merged commit c5d8484 into stackhpc/2024.1 Jun 13, 2025
20 of 22 checks passed
@dougszumski dougszumski deleted the rated-dwpd-alerts branch June 13, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
antelope Targets the Antelope OpenStack release enhancement New feature or request monitoring All things related to observability & telemetry size: xl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants