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.

Sign up for GitHub

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

Jump to bottom

(WIP):add opentelemetry to membrane #780

Merged
merged 32 commits into from
Nov 21, 2023
Merged

(WIP):add opentelemetry to membrane #780

merged 32 commits into from
Nov 21, 2023

Conversation

k-klemmer
Copy link
Collaborator

is still WIP, needs tests etc.

@k-klemmer k-klemmer requested a review from t-burch November 2, 2023 08:40
@t-burch t-burch marked this pull request as draft November 2, 2023 09:31
k-klemmer and others added 25 commits November 2, 2023 11:17
@k-klemmer
refactor opentelemetry interceptor
d71e683
@k-klemmer
add example
836e9c7
@k-klemmer
reset distribution/conf/proxies.xml
53a2c3b
@k-klemmer
refactoring: extract methods
002e982
@k-klemmer
refactoring: extract methods, imports
456ea96
@k-klemmer
refactoring: remove unused log, simply inhered method without override
ad59e88
@k-klemmer
handle response + refactoring
132366e
@k-klemmer
make final variables also static
6805582
@k-klemmer
refactor import
6cfebf2
@k-klemmer
fix: response handling
923eacd
@predic8
Refactored, cleanup
1f71da1
@k-klemmer
fix global instance + example readme refactor
06c1592
@k-klemmer
refactor README.md
6f847fc
@predic8
Readme
676c071
@k-klemmer
refactor README.md
9261270
@k-klemmer
refactor README.md
d2d07f0
@k-klemmer
handle abort
eb3f91b
@k-klemmer
fix example
e2882e6
@k-klemmer
add example test
1b1363a
@k-klemmer
fix port in test
61ff389
@k-klemmer
refactor README.md + example
ce4f75d
@k-klemmer
refactor README.md + example
62464de
@k-klemmer
refactor test
5a6cf29
@predic8
format
9df1725
@k-klemmer
refactor tests
ad48ce7
@t-burch t-burch marked this pull request as ready for review November 21, 2023 10:30
@predic8 predic8 merged commit 2c3dcd3 into master Nov 21, 2023
2 checks passed
@predic8 predic8 deleted the opentelemetry branch November 21, 2023 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t-burch t-burch t-burch approved these changes

@predic8 predic8 predic8 approved these changes

Assignees

@k-klemmer k-klemmer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@k-klemmer @predic8 @t-burch