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
Added image and snapshot metrics #3
Conversation
I find your lack of tests disturbing~ |
Does what is currently there seem sane though? (yet to add folds + tests + pollster format) |
This looks fine to me, pending tests. |
merge with magikarp, it's changed since you forked |
Adding in some tests I missed also |
any hard-coded source dict key/vals should be moved to Tags |
Conflicts: lib/Ceilometer/Infer.hs
$(makeLenses ''PDImage) | ||
|
||
newtype PDImageP = PDImageP { _pdImagePVal :: PFValue64 } | ||
deriving (Show, Read, Eq, Typeable) |
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.
what's ImageP? ImagePoint? sounds a bit like ImagePipe
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.
should I s/ImageP/ImagePollster/g ?
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.
probably, we tend to append "P" for pipe versions of stuff
I haven't checked, but cleans & rebuilds to make sure we have no warnings |
Succeeded by #5 |
Added support for Image Events and Pollsters
Added Snapshot Events
Updated Infer.hs for instance_disk
Fixed some prism bugs
Added missing tests
@tranma