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

WIP: [OSD-22023] Add new deduplicated metric for vcpus. #686

Merged

Conversation

bergmannf
Copy link
Contributor

This metric is being forwarded, as a deduplicated metrics.

See: OSD-22023 for more information.

@bergmannf bergmannf force-pushed the osd-22023-add-deduped-vcpus-metric branch from a3660eb to ac5ef62 Compare April 16, 2024 10:52
@bergmannf bergmannf force-pushed the osd-22023-add-deduped-vcpus-metric branch from ac5ef62 to a18267e Compare April 16, 2024 12:05
@typeid
Copy link
Contributor

typeid commented Apr 22, 2024

@bergmannf I believe you need to additionally add the metric to the template, see #680

@bergmannf bergmannf force-pushed the osd-22023-add-deduped-vcpus-metric branch 2 times, most recently from 786496e to ba126a0 Compare April 23, 2024 07:56
@typeid
Copy link
Contributor

typeid commented Apr 24, 2024

/retest

@douglascamata
Copy link
Contributor

@bergmannf @typeid: it seems like the build fails because of an issue we had in the past, which is already fixed in the more recent commits. If you pull-in the latest commits from main it should solve the issue.

@douglascamata douglascamata self-assigned this Apr 24, 2024
This metric is being forwarded, as a deduplicated metrics.

See: OSD-22023 for more information.
@bergmannf bergmannf force-pushed the osd-22023-add-deduped-vcpus-metric branch from ba126a0 to 8b9bd75 Compare April 24, 2024 09:52
Copy link
Contributor

@philipgough philipgough left a comment

Choose a reason for hiding this comment

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

/lgtm

@philipgough philipgough merged commit d2353fb into rhobs:main Apr 24, 2024
3 checks passed
@juzhao juzhao mentioned this pull request Jun 4, 2024
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.

5 participants