Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

feat: Add support for library instrumentation#1261

Merged
losalex merged 14 commits intomainfrom
losalex/feat-instrumentation
May 9, 2022
Merged

feat: Add support for library instrumentation#1261
losalex merged 14 commits intomainfrom
losalex/feat-instrumentation

Conversation

@losalex
Copy link
Copy Markdown
Contributor

@losalex losalex commented Apr 29, 2022

This feature provides an ability to log extra entry with diagnostics structure which contains logging library information. Such entry is logged only once when first entry is written by a process using logging library.

Fixes #1286 🦕

@product-auto-label product-auto-label Bot added size: l Pull request size is large. api: logging Issues related to the googleapis/nodejs-logging API. labels Apr 29, 2022
@losalex losalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022
@losalex losalex marked this pull request as ready for review May 2, 2022 15:53
@losalex losalex requested review from a team May 2, 2022 15:53
Comment thread src/utils/instrumentation.ts Outdated
Comment thread src/utils/instrumentation.ts Outdated
Comment thread src/utils/instrumentation.ts Outdated
Comment thread src/utils/instrumentation.ts Outdated
Comment thread src/utils/instrumentation.ts
Comment thread src/utils/instrumentation.ts
Comment thread src/utils/instrumentation.ts
Copy link
Copy Markdown
Contributor

@minherz minherz left a comment

Choose a reason for hiding this comment

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

good work. maybe it worth to bring someone from Yoshi team for additional review.

Comment thread src/log.ts
Comment thread src/utils/instrumentation.ts
Comment thread src/utils/instrumentation.ts
Comment thread src/utils/instrumentation.ts Outdated
@losalex losalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 2, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 2, 2022
@losalex losalex added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2022
Copy link
Copy Markdown
Contributor

@summer-ji-eng summer-ji-eng left a comment

Choose a reason for hiding this comment

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

But familiar with the business, but overall LGTM and left couple comments. Thank you:)

Comment thread test/instrumentation.ts Outdated
Comment thread src/utils/instrumentation.ts
Comment thread test/instrumentation.ts Outdated
@losalex losalex merged commit bb864c8 into main May 9, 2022
@losalex losalex deleted the losalex/feat-instrumentation branch May 9, 2022 20:17
@rr923
Copy link
Copy Markdown

rr923 commented Dec 1, 2023

Is it possible to disable "library instrumentation"?

@rr923
Copy link
Copy Markdown

rr923 commented Dec 1, 2023

Or how can I hide this kind of entries in the log explorer? There are just too many of them.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants