Skip to content

Commit

Permalink
Merge pull request #85402 from RainbowMango/pr_remove_metrics_hidden_log
Browse files Browse the repository at this point in the history
Remove metric be hidden log temporarily
  • Loading branch information
k8s-ci-robot authored Nov 22, 2019
2 parents 3c5dad6 + c22ffa2 commit d87c921
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion staging/src/k8s.io/component-base/metrics/desc.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,8 @@ func (d *Desc) determineDeprecationStatus(version semver.Version) {
return
}
if shouldHide(&version, selfVersion) {
klog.Warningf("This metric(%s) has been deprecated for more than one release, hiding.", d.fqName)
// TODO(RainbowMango): Remove this log temporarily. https://github.com/kubernetes/kubernetes/issues/85369
// klog.Warningf("This metric(%s) has been deprecated for more than one release, hiding.", d.fqName)
d.isHidden = true
}
})
Expand Down
3 changes: 2 additions & 1 deletion staging/src/k8s.io/component-base/metrics/metric.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,8 @@ func (r *lazyMetric) determineDeprecationStatus(version semver.Version) {
return
}
if shouldHide(&version, selfVersion) {
klog.Warningf("This metric has been deprecated for more than one release, hiding.")
// TODO(RainbowMango): Remove this log temporarily. https://github.com/kubernetes/kubernetes/issues/85369
// klog.Warningf("This metric has been deprecated for more than one release, hiding.")
r.isHidden = true
}
})
Expand Down

0 comments on commit d87c921

Please sign in to comment.