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

mock2-grpc (collector): namespaces not updated in GetConfigTypes #1393

Closed
lmroz opened this issue Nov 25, 2016 · 1 comment
Closed

mock2-grpc (collector): namespaces not updated in GetConfigTypes #1393

lmroz opened this issue Nov 25, 2016 · 1 comment

Comments

@lmroz
Copy link
Contributor

lmroz commented Nov 25, 2016

https://github.com/intelsdi-x/snap/blob/master/plugin/collector/snap-plugin-collector-mock2-grpc/mock/mock.go#L159
Namespaces used there does not match declared ones.

@IRCody
Copy link
Contributor

IRCody commented Nov 28, 2016

@lmroz: To clarify - The configPolicy is being set for a namespace that is not advertised directly via GetMetricTypes. From looking at it I would expect the result to be that the config is just never applied.

Aside from fixing this specific test plugin, is there additional validation that can be done from the framework perspective to alert/warn users about this issue?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants