Skip to content

Add OTel instrumentation to CLI plugins#5051

Merged
laurazard merged 2 commits intodocker:masterfrom
laurazard:add-plugin-command-metrics
May 15, 2024
Merged

Add OTel instrumentation to CLI plugins#5051
laurazard merged 2 commits intodocker:masterfrom
laurazard:add-plugin-command-metrics

Conversation

@laurazard
Copy link
Member

@laurazard laurazard commented May 2, 2024

- What I did

Instrument CLI plugin commands to track command.time meter, same as the regular CLI commands.

$ DOCKER_CLI_OTEL_EXPORTER_OTLP_ENDPOINT=http://localhost:4321 ./build/docker compose down

image

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

image

@codecov-commenter
Copy link

codecov-commenter commented May 2, 2024

Codecov Report

Attention: Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.

Project coverage is 61.32%. Comparing base (02537ea) to head (5f4f4f6).
Report is 490 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5051      +/-   ##
==========================================
- Coverage   61.35%   61.32%   -0.04%     
==========================================
  Files         298      295       -3     
  Lines       20685    20687       +2     
==========================================
- Hits        12691    12686       -5     
- Misses       7093     7099       +6     
- Partials      901      902       +1     

@laurazard laurazard force-pushed the add-plugin-command-metrics branch from 822b9b8 to d32860d Compare May 6, 2024 09:52
@laurazard laurazard self-assigned this May 6, 2024
@laurazard laurazard force-pushed the add-plugin-command-metrics branch from d32860d to 142cd99 Compare May 6, 2024 12:29
@laurazard laurazard marked this pull request as ready for review May 6, 2024 12:32
@laurazard laurazard changed the title WIP: add instrumentation to plugins Add OTel instrumentation to CLI plugins May 6, 2024
@laurazard laurazard force-pushed the add-plugin-command-metrics branch 3 times, most recently from d607756 to 8f0b236 Compare May 7, 2024 09:51
@laurazard laurazard requested review from jsternberg and vvoland May 7, 2024 09:53
@laurazard laurazard force-pushed the add-plugin-command-metrics branch from 8f0b236 to ae0bc20 Compare May 7, 2024 12:46
@laurazard laurazard requested a review from milas May 7, 2024 13:25
@laurazard laurazard force-pushed the add-plugin-command-metrics branch 2 times, most recently from 594f11d to f7ffd69 Compare May 13, 2024 13:43
@laurazard laurazard force-pushed the add-plugin-command-metrics branch 2 times, most recently from 3ddb0a8 to 9141f02 Compare May 14, 2024 09:48
@laurazard laurazard force-pushed the add-plugin-command-metrics branch from 9141f02 to aaffe49 Compare May 14, 2024 15:37
@laurazard laurazard requested a review from vvoland May 14, 2024 15:38
Copy link
Collaborator

@vvoland vvoland left a comment

Choose a reason for hiding this comment

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

LGTM

@laurazard laurazard requested a review from krissetto May 14, 2024 15:58
Copy link
Contributor

@krissetto krissetto 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 go 🚀

@laurazard laurazard force-pushed the add-plugin-command-metrics branch from aaffe49 to e675a0a Compare May 14, 2024 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants