Skip to content

Conversation

@ace-n
Copy link
Contributor

@ace-n ace-n commented Mar 16, 2020

@vinod-at-google @labtopia IIRC the lack of log severity info is a known issue. (If it's not, let me know so I can hold off on merging this PR.)

@ace-n ace-n requested review from a team and lesv March 16, 2020 22:05
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 16, 2020
Comment on lines 35 to 40
System.out.println("I am an info log to stdout!");
System.err.println("I am an info log to stderr!");

// Stackdriver does not detect severity from Cloud Functions log output
LOGGER.info("I am an info log to Stackdriver!");
LOGGER.warning("I am an info log to Stackdriver!");
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought that we did look for WARNING and INFO and other key words to fill stackdriver

Copy link
Contributor Author

@ace-n ace-n Mar 16, 2020

Choose a reason for hiding this comment

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

Confirmed with @vinod-at-google (GCF PM) that this is a known problem

@ace-n ace-n added the automerge Merge the pull request once unit tests and other checks pass. label Mar 21, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 550f349 into master Mar 21, 2020
@gcf-merge-on-green gcf-merge-on-green bot deleted the gcf-logs branch March 21, 2020 03:48
olavloite pushed a commit to olavloite/java-docs-samples that referenced this pull request Mar 22, 2020
…ogleCloudPlatform#2422)

~@vinod-at-google @labtopia IIRC the lack of log severity info is a known issue. (If it's not, let me know so I can hold off on merging this PR.)~
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants