Update KPIMON chart - add toJson for config.json #464
Conversation
| @@ -42,6 +42,10 @@ config: | |||
| kpimon: | |||
| e2tEndpoint: "onos-e2t:5150" | |||
| kpmVersion: "v2" | |||
| config_json: | |||
There was a problem hiding this comment.
Did you update the yang model already?
There was a problem hiding this comment.
This JSON should be generated based on the yang model if you want to use onos-config to update the values.
There was a problem hiding this comment.
Did you update the yang model already?
Not yet - will do it sooner. However, still it requires setting the granularity period at the initial stage.
There was a problem hiding this comment.
Yeah but you need to make sure the models are match otherwise the onos-config cannot be used for setting values because the schema is different. I think it affects report_period as well.
There was a problem hiding this comment.
Yeah but you need to make sure the models are match otherwise the onos-config cannot be used for setting values because the schema is different. I think it affects report_period as well.
Yeah ok. I think when I work on the yang model for period configuration in runtime, I will check this out. At this moment, I think this PR is good enough.
No description provided.