Skip to content

DNM testing zuul trigger#36

Closed
vyzigold wants to merge 2 commits into
mainfrom
18.0-fr5
Closed

DNM testing zuul trigger#36
vyzigold wants to merge 2 commits into
mainfrom
18.0-fr5

Conversation

@vyzigold
Copy link
Copy Markdown
Contributor

No description provided.

vyzigold and others added 2 commits May 11, 2026 06:47
Parse availability_zone, flavor_id, and flavor_name
from ceilometer resource_metadata and expose them as
Prometheus labels in the sg-core ceilometer handler
plugin. This brings sg-core's metric output to parity
with ceilometer's built-in prometheus exporter.

- Add AvailabilityZone field to Metadata struct
- Add Flavor struct with ID and Name fields
- Emit availability_zone, flavor_id, flavor_name
  labels in genLabels()
- Increase pre-allocated label array size 12 to 15
- Add LB and compute test data to JSON fixtures
- Add genLabels unit tests for LB and compute

Generated-By: Claude-Code claude-opus-4-6
…-to-18.0-fr5

[18.0-fr5] [OSPRH-28762] Add AZ and flavor labels for LB/compute metrics
@openshift-ci openshift-ci Bot requested review from jlarriba and vkmc May 18, 2026 10:12
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 18, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vyzigold

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vyzigold vyzigold closed this May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant