Skip to content

Fix PCP metric reporting#59

Merged
kdvalin merged 2 commits intomainfrom
fix/pcp
Jan 27, 2026
Merged

Fix PCP metric reporting#59
kdvalin merged 2 commits intomainfrom
fix/pcp

Conversation

@kdvalin
Copy link
Member

@kdvalin kdvalin commented Jan 26, 2026

Description

This fixes PCP reporting issues reported by #56.

Simply this adds a reset file to provide the PCP functions so metrics are well defined.

Before/After Comparison

Before

PCP would issue warnings like the message below
err.log

After

PCP warnings are no longer present

Clerical Stuff

This closes #56

Relates to JIRA: RPOPC-761

bash -x output

pyperf_run.log

pmrep output

pyperf_pmrep.txt

This adds openmetrics_pyperf_reset file, which silences warnings about
missing metrics.  This should fix a bug where metrics are not being
reported as well.
@kdvalin kdvalin requested a review from a team January 26, 2026 18:35
@github-actions
Copy link

This relates to RPOPC-761

Copy link
Contributor

@dvalinrh dvalinrh left a comment

Choose a reason for hiding this comment

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

LGTM

@dvalinrh dvalinrh added the group_review_lgtm Indicates approval after a group review meeting label Jan 27, 2026
@kdvalin kdvalin merged commit c20c571 into main Jan 27, 2026
7 of 8 checks passed
@kdvalin kdvalin deleted the fix/pcp branch January 27, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

group_review_lgtm Indicates approval after a group review meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pcp appears to be broke

2 participants