Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@surbhigarg92
Copy link
Contributor

@surbhigarg92 surbhigarg92 commented Dec 10, 2024

This PR add preview feature End to end tracing for nodejs. https://cloud.google.com/spanner/docs/tracing-overview#end-to-end-side-tracing

@surbhigarg92 surbhigarg92 requested review from a team as code owners December 10, 2024 11:12
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: spanner Issues related to the googleapis/nodejs-spanner API. and removed size: s Pull request size is small. labels Dec 10, 2024
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

@snippet-bot
Copy link

snippet-bot bot commented Dec 10, 2024

No region tags are edited in this PR.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label Dec 10, 2024
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xl Pull request size is extra large. labels Dec 10, 2024
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: s Pull request size is small. labels Dec 10, 2024
@surbhigarg92 surbhigarg92 changed the title E2e tracing feat: Enable e2e tracing Dec 10, 2024
@surbhigarg92 surbhigarg92 force-pushed the e2eTracing branch 3 times, most recently from e0c6599 to 35e68fd Compare December 10, 2024 18:24
resourceName: string,
enableTracing?: boolean
) {
const headers: {[k: string]: string} = {};
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Can we initialize CLOUD_RESOURCE_HEADER here itself

@surbhigarg92 surbhigarg92 merged commit 3cc257e into main Dec 24, 2024
18 checks passed
@surbhigarg92 surbhigarg92 deleted the e2eTracing branch December 24, 2024 10:58
alkatrivedi pushed a commit that referenced this pull request Dec 27, 2024
* e2eTracing

* context propagation

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Add end to end tracing header and refactore resource header

* Add end to end tracing header and refactore resource header

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/nodejs-spanner API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants