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

feat(zfspv): move to klog #166

Merged
merged 3 commits into from
Jun 29, 2020
Merged

Conversation

vaniisgh
Copy link
Contributor

Why is this PR required? What issue does it fix?:
Move to klog from Sirupsen/logarus #135

What this PR does?:
Simply uses klogs to log.

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests

Signed-off-by: vaniisgh <vanisingh@live.co.uk>
@codecov-commenter
Copy link

codecov-commenter commented Jun 28, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #166   +/-   ##
=======================================
  Coverage   22.65%   22.65%           
=======================================
  Files          14       14           
  Lines         490      490           
=======================================
  Hits          111      111           
  Misses        378      378           
  Partials        1        1           

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 54f2b0b...abb33a9. Read the comment docs.

Copy link
Contributor

@pawanpraka1 pawanpraka1 left a comment

Choose a reason for hiding this comment

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

Looks Good. Thanks @vaniisgh. Can you add changelog also in this PR?

@pawanpraka1 pawanpraka1 added enhancement Add new functionality to existing feature Need community involvement Needs community involvement on some action item. labels Jun 29, 2020
@pawanpraka1 pawanpraka1 added this to the v0.9 milestone Jun 29, 2020
@pawanpraka1 pawanpraka1 changed the title Enhancement(zfspv): Move to klog feat(zfspv): move to klog Jun 29, 2020
Signed-off-by: vaniisgh <vanisingh@live.co.uk>
Signed-off-by: vaniisgh <vanisingh@live.co.uk>
@pawanpraka1 pawanpraka1 merged commit d0d1664 into openebs:master Jun 29, 2020
@pawanpraka1 pawanpraka1 linked an issue Jun 29, 2020 that may be closed by this pull request
@vaniisgh vaniisgh deleted the migrateToKlog branch June 29, 2020 06:52
pawanpraka1 added a commit to pawanpraka1/zfs-localpv that referenced this pull request Jul 9, 2020
pawanpraka1 added a commit to pawanpraka1/zfs-localpv that referenced this pull request Jul 9, 2020
pawanpraka1 added a commit to pawanpraka1/zfs-localpv that referenced this pull request Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Add new functionality to existing feature Need community involvement Needs community involvement on some action item.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

move to klog from logrus
3 participants