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

opentelemetry callback: context propagation and error exception #3378

Conversation

v1v
Copy link
Contributor

@v1v v1v commented Sep 15, 2021

SUMMARY
  1. Add context propagation then it can be integrated with some other tools.
  2. Handle task failures as errors.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/callback/opentelemetry.py

COMPONENT NAME
ADDITIONAL INFORMATION

Reported errors:

image

Context propagation:

image

In order to test this particular plugin you could use jenkinsci/opentelemetry-plugin#166.

NOTE: There is an open discussion open-telemetry/opentelemetry-specification#740 for the traceparent standardisation.

@ansibullbot ansibullbot added callback callback plugin feature This issue/PR relates to a feature request plugins plugin (any type) labels Sep 15, 2021
v1v added a commit to v1v/opentelemetry-plugin that referenced this pull request Sep 15, 2021
@ansibullbot ansibullbot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Sep 15, 2021
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Doesn't need changelog fragment / version_added since it is a new plugin.

plugins/callback/opentelemetry.py Outdated Show resolved Hide resolved
@felixfontein felixfontein added backport-3 check-before-release PR will be looked at again shortly before release and merged if possible. labels Sep 15, 2021
@ansibullbot ansibullbot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Sep 15, 2021
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Code looks good. (Cannot judge the functionality.)

@felixfontein felixfontein merged commit 0634583 into ansible-collections:main Sep 16, 2021
@patchback
Copy link

patchback bot commented Sep 16, 2021

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/06345839c6e333f9021d255af5e2707ecbba2c12/pr-3378

Backported as #3381

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Sep 16, 2021
patchback bot pushed a commit that referenced this pull request Sep 16, 2021
* opentelemetry callback: context propagation and error exception

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 0634583)
@felixfontein
Copy link
Collaborator

@v1v thanks for contributing this!

felixfontein pushed a commit that referenced this pull request Sep 16, 2021
… (#3381)

* opentelemetry callback: context propagation and error exception

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 0634583)

Co-authored-by: Victor Martinez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
callback callback plugin feature This issue/PR relates to a feature request plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants