Tendrl / specifications

Tendrl specs go here
GNU Lesser General Public License v3.0
6 stars 16 forks source link

Remove existing performance monitoring component and replace it with a new monitoring-integration component. #179

Closed nnDarshan closed 7 years ago

nnDarshan commented 7 years ago

The new monitoring-integration component would primarily focus on dynamic configuration of grafana, based on the topology changes in etcd. Some of these responsibilities would be:

anivargi commented 7 years ago

@nnDarshan please update the issue with the spec.

mbukatov commented 7 years ago

I see this item linked from https://github.com/Tendrl/specifications/milestone/3, so my question is: what should I review to evaluate configuration and installation impact on tendrl-ansible? I'm asking because this issue refers to work related to a previous milestone and I'm little confused why is this included in the next milestone as well.

rishubhjain commented 7 years ago

@mbukatov Monitoring-integration was introduced in the previous milestone. It is supposed to replace performance monitoring component. In every milestone there are some functionalities that are added to monitoring-integration. Some of the functionalities mentioned in issue description of this issue are implemented in 3rd milestone as well.

mbukatov commented 7 years ago

@rishubhjain ok, I understand that we improve tendrl-monitoring-integration in every milestone, but I'm unable to find out which changes/features are done in https://github.com/Tendrl/specifications/milestone/3 in particular. Shouldn't we have a different issue/specification for this? If we share single issue/specification in 3 different milestones, how do we agree on what is done in a particular milestone?

mbukatov commented 7 years ago

Ok, based on discussion with @r0h4n , I'm going to assume there is no additional impact on configuration/installation.

rishubhjain commented 7 years ago

For other features, a new issue is filed https://github.com/Tendrl/specifications/issues/247