-
Notifications
You must be signed in to change notification settings - Fork 98
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
Trace API function for trace writer project ID #548
Trace API function for trace writer project ID #548
Conversation
src/trace-api.js
Outdated
TraceAgent.prototype.getWriterProjectId = function() { | ||
var traceWriter; | ||
try { | ||
traceWriter = TraceWriter.get(); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
6446e09
to
18f130d
Compare
…odejs into get-writer-project-id
18f130d
to
835a00c
Compare
}); | ||
}); | ||
|
||
it('should return get the project ID if set in config', function() { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
ee273d8
to
5ab8031
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for putting this together!
No problem. If all is good, can you go ahead and merge this? |
This will be used by the best-effort logs correlation changes to be able to format the
LogEntry.trace
field in the specifiedprojects/[PROJECT-ID]/traces/[TRACE-ID]
format (per https://cloud.google.com/logging/docs/reference/v2/rest/v2/LogEntry).