Skip to content

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Feb 28, 2025

No description provided.

m-bull and others added 3 commits February 22, 2025 10:10
- Compatability with Dell servers
- Added health round up panels
- Add variables for different groups e.g overcloud vs compute
Prometheus will regard metrics collected over this perioid as stale as
such they won't show up in grafana.
@jovial jovial requested a review from a team as a code owner February 28, 2025 18:15
@product-auto-label product-auto-label bot added size: l monitoring All things related to observability & telemetry labels Feb 28, 2025
@jovial jovial marked this pull request as draft February 28, 2025 18:16
jovial and others added 3 commits February 28, 2025 18:25
The redfish dashboard was updated to use a new UUID, however this caused
errors in the Grafana logs and Grafana didn't update the dashboard.
Reverting this to the old UUID fixes the issue.
@m-bull m-bull force-pushed the bugfix/edfish-exporter-2.0/dell branch from cc2c5e5 to 15e2cba Compare March 7, 2025 15:58
@jovial jovial requested a review from JohnGarbutt March 10, 2025 17:53
@jovial jovial marked this pull request as ready for review March 10, 2025 17:53
@MoteHue MoteHue changed the base branch from redfish-exporter-2.0 to stackhpc/2024.1 May 1, 2025 08:54
This isn't working yet (takes far too long to scrape),let's
split it out into a separate patch.

Also removed the query lookback delta config, as this was
working around the much longer scrape times.
@MoteHue MoteHue force-pushed the bugfix/edfish-exporter-2.0/dell branch from ee33b89 to 7c5935b Compare May 1, 2025 09:09
@MoteHue
Copy link
Contributor

MoteHue commented May 1, 2025

We've decided to push the log collection out for now, to be proposed in a new PR later. I'll leave the conversation threads open here so they can be seen in that work.

@MoteHue MoteHue requested a review from Alex-Welsh May 1, 2025 09:16
@MoteHue MoteHue changed the title Improvements to redfish exporter v2 for Dell systems Redfish Exporter v2.x May 1, 2025
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.

Let's merge and follow up with anything else

@MoteHue
Copy link
Contributor

MoteHue commented May 2, 2025

We've decided to push the log collection out for now, to be proposed in a new PR later. I'll leave the conversation threads open here so they can be seen in that work.

Need to resolve the convos to merge :(

@dougszumski dougszumski merged commit 782f689 into stackhpc/2024.1 May 2, 2025
25 checks passed
@dougszumski dougszumski deleted the bugfix/edfish-exporter-2.0/dell branch May 2, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monitoring All things related to observability & telemetry size: xl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants