-
Notifications
You must be signed in to change notification settings - Fork 38
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/Add version metrics #2344
Feat/Add version metrics #2344
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2344 +/- ##
==========================================
- Coverage 29.88% 29.88% -0.01%
==========================================
Files 398 398
Lines 29806 29806
==========================================
- Hits 8909 8907 -2
- Misses 20180 20182 +2
Partials 717 717
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
270c5e9
to
99605d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
out-of-context, code ok
- use a separate `neofs_ir` namespace for IR apps instead of `neofs_node`; - move `epoch` metrics to a `state` subsystem instead of `object`; - SN's epoch metrics do not have "inner-ring" help message now. The old ones are deprecated and will be removed when the next minor release comes out. Signed-off-by: Pavel Karpy <carpawell@morphbits.io>
Signed-off-by: Pavel Karpy <carpawell@morphbits.io>
Forgotten but necessary renaming. Signed-off-by: Pavel Karpy <carpawell@morphbits.io>
99605d0
to
045a284
Compare
Closes #2326. Gonna look smth like that:
@roman-khimov, @cthulhu-rider, @532910, @aprasolova, @anyone_looking_at_metrics, please, look at the first commit. Something was certainly wrong, pay attention to it and make me know if some SW depends on it and we are not able to make such changes OR such changes must be synced before the merge.