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

Revert "[prometheus check] Actually use provided "metrics_mapper" (#3… #3861

Merged
merged 1 commit into from
Jun 4, 2019

Conversation

ahmed-mez
Copy link
Contributor

@ahmed-mez ahmed-mez commented Jun 4, 2019

The proper way to change a metric name is by setting up the optional <NEW_METRIC_NAME> field in the metrics param.

@ahmed-mez ahmed-mez requested review from a team as code owners June 4, 2019 16:37
@codecov
Copy link

codecov bot commented Jun 4, 2019

Codecov Report

Merging #3861 into master will decrease coverage by 2.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3861      +/-   ##
==========================================
- Coverage   86.82%   84.15%   -2.68%     
==========================================
  Files         729       62     -667     
  Lines       37220     4884   -32336     
  Branches     4350      596    -3754     
==========================================
- Hits        32316     4110   -28206     
+ Misses       3744      643    -3101     
+ Partials     1160      131    -1029

1 similar comment
@codecov
Copy link

codecov bot commented Jun 4, 2019

Codecov Report

Merging #3861 into master will decrease coverage by 2.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3861      +/-   ##
==========================================
- Coverage   86.82%   84.15%   -2.68%     
==========================================
  Files         729       62     -667     
  Lines       37220     4884   -32336     
  Branches     4350      596    -3754     
==========================================
- Hits        32316     4110   -28206     
+ Misses       3744      643    -3101     
+ Partials     1160      131    -1029

@ahmed-mez ahmed-mez merged commit 5c55cb5 into master Jun 4, 2019
@ofek ofek deleted the revert-3446-patch-1 branch June 4, 2019 16:43
ofek pushed a commit that referenced this pull request Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants