Skip to content

Export trace key in logging-{winston,bunyan}#2602

Merged
stephenplusplus merged 2 commits intogoogleapis:masterfrom
draffensperger:export-logging-key
Sep 13, 2017
Merged

Export trace key in logging-{winston,bunyan}#2602
stephenplusplus merged 2 commits intogoogleapis:masterfrom
draffensperger:export-logging-key

Conversation

@draffensperger
Copy link
Contributor

This makes the LOGGING_TRACE_KEY constant exported by the logging-{winston,bunyan} modules per conversation here: #2574 (comment)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 11, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 871ebd5 on draffensperger:export-logging-key into 5c7035b on GoogleCloudPlatform:master.

Copy link
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

LGTM w/ nit.

const recordWithTrace = extend({
'logging.googleapis.com/trace': 'projects/project1/traces/trace1'
}, RECORD);
var recordWithTrace = extend({}, RECORD);

This comment was marked as spam.

This comment was marked as spam.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 917f65f on draffensperger:export-logging-key into 5c7035b on GoogleCloudPlatform:master.

@draffensperger
Copy link
Contributor Author

Is this good to merge? @stephenplusplus, did you want to take a look?

@stephenplusplus stephenplusplus merged commit a4cdf49 into googleapis:master Sep 13, 2017
@stephenplusplus
Copy link
Contributor

Sorry I didn't catch this one. LGTM!

@draffensperger draffensperger deleted the export-logging-key branch September 13, 2017 14:51
stephenplusplus pushed a commit to lukesneeringer/google-cloud-node that referenced this pull request Sep 18, 2017
* Export trace key in logging-{winston,bunyan}

* Fix var to const
miguelvelezsa pushed a commit that referenced this pull request Jan 14, 2026
* Export trace key in logging-{winston,bunyan}

* Fix var to const
GautamSharda pushed a commit that referenced this pull request Jan 14, 2026
* Export trace key in logging-{winston,bunyan}

* Fix var to const
GautamSharda pushed a commit that referenced this pull request Jan 20, 2026
* Export trace key in logging-{winston,bunyan}

* Fix var to const
GautamSharda pushed a commit that referenced this pull request Feb 3, 2026
* Export trace key in logging-{winston,bunyan}

* Fix var to const
sofisl pushed a commit that referenced this pull request Feb 4, 2026
* Export trace key in logging-{winston,bunyan}

* Fix var to const
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging-bunyan api: logging-winston cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants