Skip to content

Addition of pcp2opentelemetry tool #2219

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lmchilton
Copy link
Contributor

Supports translation of pcp metrics into opentelemetry format. Supports opentelemetry protocol http json.

Updated relevent qa tests; added one new one (1977) to test pcp2opentelemetry http functionality.

Added man page and makefile. Includes updates and
changes from review.

@lmchilton lmchilton requested review from natoscott and wcohen and removed request for natoscott June 6, 2025 19:58
@lmchilton lmchilton force-pushed the pcp2opentelemetry-new branch 2 times, most recently from bbf74e3 to 46e705b Compare June 11, 2025 14:58
@natoscott
Copy link
Member

@lmchilton this is failing in the CI build because the new pcp2opentelemetry tool subdir is missing from the src/GNUmakefile - this patch should fix it:

$ git diff
src/GNUmakefile
--- /tmp/git-blob-wXFv0g/GNUmakefile    2025-06-12 10:07:23.480332266 +1000
+++ src/GNUmakefile     2025-06-12 09:55:30.834563600 +1000
@@ -65,6 +65,7 @@ OTHER_SUBDIRS += \
        pcp2influxdb \
        pcp2json \
        pcp2openmetrics \
+       pcp2opentelemetry \
        pcp2spark \
        pcp2xlsx \
        pcp2xml \

Supports translation of pcp metrics into opentelemetry
format. Supports opentelemetry protocol http json.
Updated relevant qa tests; added one new one (1977)
to test pcp2opentelemetry http functionality.
Added man page and makefile. Includes updates and
changes from review.
@lmchilton lmchilton force-pushed the pcp2opentelemetry-new branch from 46e705b to a581d27 Compare June 12, 2025 13:23
@wcohen
Copy link
Contributor

wcohen commented Jun 23, 2025

Need an entry in qa/groups for the new test 1977. Without the entry the the 1977 test doesn't get installed.
When looking through the test qa/1131.out wondering why the change adding "age:" {...} on the pcp2openmetrics?

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.

3 participants