Skip to content
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

Logging: support request-correlated logging in App Engine standard python37 runtime #6118

Merged
merged 4 commits into from
Sep 27, 2018

Conversation

myelin
Copy link
Contributor

@myelin myelin commented Sep 26, 2018

The python37 runtime provides the GOOGLE_CLOUD_PROJECT environment variable instead of GCLOUD_PROJECT, and for log messages to be correlated with the request in StackDriver, their 'trace' value has to be passed as a top level member of the log record rather than as an appengine.googleapis.com/trace_id label.

…ntime.

The python37 runtime provides the GOOGLE_CLOUD_PROJECT environment
variable instead of GCLOUD_PROJECT, and for log messages to be
correlated with the request in StackDriver, their 'trace' value has to
be passed as a top level member of the log record rather than as an
appengine.googleapis.com/trace_id label.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 26, 2018
@tseaver tseaver changed the title Support request-correlated logging in App Engine standard python37 runtime Logging: support request-correlated logging in App Engine standard python37 runtime Sep 27, 2018
@tseaver tseaver added the api: logging Issues related to the Cloud Logging API. label Sep 27, 2018
@myelin
Copy link
Contributor Author

myelin commented Sep 27, 2018

I broke something with this last CL -- fix coming soon.

@theacodes theacodes merged commit 571a7bf into googleapis:master Sep 27, 2018
@theacodes
Copy link
Contributor

Thank you, @myelin!

_GAE_PROJECT_ENV_STANDARD: 'test_project',
_GAE_SERVICE_ENV: 'test_service',
_GAE_VERSION_ENV: 'test_version',
}):

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants