Skip to content

[OTel instrumentation] Add path params and endpoint in opts to perform_request#2179

Merged
picandocodigo merged 48 commits intoelastic:mainfrom
estolfo:otel-native-2
Sep 29, 2023
Merged

[OTel instrumentation] Add path params and endpoint in opts to perform_request#2179
picandocodigo merged 48 commits intoelastic:mainfrom
estolfo:otel-native-2

Conversation

@estolfo
Copy link
Copy Markdown
Contributor

@estolfo estolfo commented Aug 25, 2023

This PR complements a PR in the transport client

This PR does the following:

  • Adds the path params and endpoint id to the perform_request args list so that the transport client can use the values for the OpenTelemetry instrumentation
  • Updates all endpoint files with the new functionality and new perform_request method signature (code was regenerated)
  • Adds a unit test for each of the endpoints verifying that the correct endpoint id and path params are passed

@estolfo estolfo marked this pull request as draft August 25, 2023 11:23
@estolfo estolfo force-pushed the otel-native-2 branch 2 times, most recently from f7938c6 to f56b5e8 Compare August 30, 2023 08:47
@estolfo estolfo marked this pull request as ready for review September 6, 2023 13:13
@estolfo estolfo requested a review from szabosteve September 26, 2023 19:42
Copy link
Copy Markdown
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

Thanks for writing this up! I left some smaller suggestions, please take or leave them. Documentation changes LGTM!

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Update docs/open-telemetry.asciidoc

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>

Minor change to docs
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