Skip to content

Add github.com/blendle/zapdriver to list of extensions #596

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

Merged
merged 1 commit into from
May 30, 2018

Conversation

JeanMertz
Copy link
Contributor

See: #452 (comment)

For those interested, we've built a small Zap-based Stackdriver library that handles most of the heavy lifting of getting Stackdriver-formatted logs:

https://github.com/blendle/zapdriver

It produces "LogEntry v2" compatible logs to which @piotrkowalczuk pointed. It also includes some helper functions that produce the appropriate fields, such as HTTP, SourceLocation, Label, and Operation.

Feel free to take it for a spin and report any issues you find. I'm sure there is still some functionality missing, but it did get us going.

@CLAassistant
Copy link

CLAassistant commented May 27, 2018

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented May 27, 2018

Codecov Report

Merging #596 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #596   +/-   ##
=======================================
  Coverage   97.47%   97.47%           
=======================================
  Files          39       39           
  Lines        2017     2017           
=======================================
  Hits         1966     1966           
  Misses         43       43           
  Partials        8        8

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2dc8d10...c173905. Read the comment docs.

@akshayjshah
Copy link
Contributor

Thanks for the addition, @JeanMertz!

@akshayjshah akshayjshah merged commit ceb747c into uber-go:master May 30, 2018
@JeanMertz JeanMertz deleted the patch-1 branch May 30, 2018 22:12
cgxxv pushed a commit to cgxxv/zap that referenced this pull request Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants