Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Control/metrics.go should not panic on failure #733

Closed
tiffanyfay opened this issue Feb 25, 2016 · 0 comments
Closed

Control/metrics.go should not panic on failure #733

tiffanyfay opened this issue Feb 25, 2016 · 0 comments

Comments

@tiffanyfay
Copy link
Contributor

In AddLoadedMetricType it calls panic if the loaded plugin's config policy doesn't exist.
@tiffanyfay tiffanyfay mentioned this issue Feb 25, 2016
Merged
@pittma pittma closed this as completed in cf857f5 Feb 25, 2016
pittma added a commit that referenced this issue Feb 25, 2016
@pittma
Merge pull request #732 from tiffanyfj/tb/control-metrics-logging
95cf723 
Fixes #211 and #733: Adds logging to control/metrics.go and removes panic
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tiffanyfay