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(metrics): report versions of client, datanode, and metanode #3107

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lily-lee
Copy link
Contributor

What this PR does / why we need it:

report version information of client, metanode, datanode

Which issue this PR fixes:

fixes #3005

Special notes for your reviewer:

Release note:

Copy link

netlify bot commented Feb 21, 2024

Deploy Preview for cubefs-check ready!

Name Link
🔨 Latest commit 3a7c9e5
🔍 Latest deploy log https://app.netlify.com/sites/cubefs-check/deploys/6642cc963b1918000884743e
😎 Deploy Preview https://deploy-preview-3107--cubefs-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov bot commented Feb 21, 2024

Codecov Report

Attention: Patch coverage is 0% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 41.89%. Comparing base (bdffd66) to head (bc3be92).
Report is 29 commits behind head on master.

❗ Current head bc3be92 differs from pull request most recent head 3a7c9e5. Consider uploading reports for the commit 3a7c9e5 to get more accurate results

Files Patch % Lines
proto/version.go 0.00% 16 Missing ⚠️
datanode/metric.go 0.00% 4 Missing ⚠️
metanode/metrics.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3107      +/-   ##
==========================================
+ Coverage   40.49%   41.89%   +1.40%     
==========================================
  Files         603      581      -22     
  Lines      125057   119829    -5228     
==========================================
- Hits        50643    50208     -435     
+ Misses      70051    65300    -4751     
+ Partials     4363     4321      -42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: client, meta, datanode, reported Version information
2 participants