search

monasca / monasca-docker

Docker files and setup for Monasca
Apache License 2.0
24 stars 47 forks source link

Not able to attach notification to Alarm Definitions under Grafana Dashboard #465

Closed maestropandy closed 5 years ago

maestropandy commented 5 years ago

Hi,

Using latest monasca/grafana image "monasca/grafana:4.0.0-1.5.0" as per the latest commit to this image, it should support "attach notification to alarm definition" but its not happening.

Has anyone checked it by adding "Notification_ID or Notification Name" to "alarm_action" under grafana_dasboard alarm definition page.

It was working fine with CLI, but not with grafana dashboard

Reference: 459

dougszumski commented 5 years ago

Are you sure you don't just need to rebuild the container? The patch for allowing you to add notifications to alarms was only merged recently. The dockerfile references the master branch of the Monasca Grafana App, so it should pick it up the fix automatically during image build.

matrixik commented 5 years ago

@dougszumski It's already rebuild https://hub.docker.com/r/monasca/grafana/tags/ It's not automatic we rebuilt it by hand in #460 I think he is right that there is something wrong with adding notification (when I was looking into it adding webhook notification to alarm definition was possible after second or third try, and clicking box to add it did not showed list of all notification - they started appearing after I put some letter into box, forgot to fill issue).

dougszumski commented 5 years ago

@matrixik I think Charana intended it to work as auto-complete, rather than a list. It doesn't sound right that it took two or three tries to get it to work. If you check the Grafana logs were there any Monasca API call failures, particularly relating to listing notifications? Please file a bug in the plugin if you think there's an issue. We have Charana until Friday.

maestropandy commented 5 years ago

@dougszumski It's already rebuild https://hub.docker.com/r/monasca/grafana/tags/ It's not automatic we rebuilt it by hand in #460 I think he is right that there is something wrong with adding notification (when I was looking into it adding webhook notification to alarm definition was possible after second or third try, and clicking box to add it did not showed list of all notification - they started appearing after I put some letter into box, forgot to fill issue).

@dougszumski "grafana docker logs having message "[Note]: Getting error while reading "mon-api" database using "mon' user" other than that there is no log related to listing notifications.

Mostly after creating new alarm_definitions, update it with one time "notification name"

Note: "notification name" drop down is not available, need to give details manually.

Also "matchby" parameter need value to be passed will be great to avoid alarm mapping to all hostnames

Example: Create alarm-definitions by mapping to "tenant-name" or "VM_name" instead of matching by "hostname", "device", "component"

dougszumski commented 5 years ago

@maestropandy Please can file these issues (one issue per github issue) in the related repository here: https://github.com/monasca/monasca-grafana/issues

maestropandy commented 5 years ago

@dougszumski done https://github.com/monasca/monasca-grafana/issues/19

matrixik commented 5 years ago

Issue created in monasca-grafana. Closing.