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: Add metrics module #220

Merged
merged 8 commits into from
Jul 20, 2022
Merged

feat: Add metrics module #220

merged 8 commits into from
Jul 20, 2022

Conversation

diwufeiwen
Copy link
Contributor

@diwufeiwen diwufeiwen commented Jul 19, 2022

@diwufeiwen diwufeiwen self-assigned this Jul 19, 2022
@diwufeiwen diwufeiwen force-pushed the feat/lt/metrics_impl branch from 38b8c8b to 0035246 Compare July 19, 2022 08:01
@diwufeiwen diwufeiwen changed the title feat: add mertics config & exporter feat: Add metrics module Jul 19, 2022
@diwufeiwen diwufeiwen force-pushed the feat/lt/metrics_impl branch from 42a67c0 to 8d75ad2 Compare July 19, 2022 08:16
@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2022

Codecov Report

Merging #220 (6c005ae) into master (2bf3c58) will decrease coverage by 0.35%.
The diff coverage is 1.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
- Coverage   15.47%   15.12%   -0.36%     
==========================================
  Files          56       57       +1     
  Lines        5403     5502      +99     
==========================================
- Hits          836      832       -4     
- Misses       4488     4591     +103     
  Partials       79       79              
Flag Coverage Δ
unittests 15.12% <1.88%> (-0.36%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
main.go 0.00% <0.00%> (ø)
metrics/exporter.go 0.00% <0.00%> (ø)
models/sqlite/shared_params.go 58.82% <ø> (ø)
service/address_service.go 0.00% <ø> (ø)
service/message_service.go 0.00% <0.00%> (ø)
service/message_state_refresh.go 0.00% <0.00%> (ø)
config/config.go 100.00% <100.00%> (ø)

@diwufeiwen diwufeiwen force-pushed the feat/lt/metrics_impl branch 2 times, most recently from e5a5f6c to 4a2b137 Compare July 19, 2022 09:43
@diwufeiwen diwufeiwen requested review from simlecode and hunjixin July 19, 2022 09:54
@diwufeiwen diwufeiwen force-pushed the feat/lt/metrics_impl branch 2 times, most recently from 11aae61 to 0b745b7 Compare July 20, 2022 03:10
docs/zh/metrics-config-desc.md Outdated Show resolved Hide resolved
metrics/exporter.go Outdated Show resolved Hide resolved
metrics/exporter.go Outdated Show resolved Hide resolved
@diwufeiwen diwufeiwen force-pushed the feat/lt/metrics_impl branch from 6c005ae to 80b6aeb Compare July 20, 2022 07:44
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.

[venus] messager增加metrics,方便用户查看消息推送状态
4 participants