A CDM memory alarm is created each time the maas_cdm playbook is run

Bug #1416443 reported by git-harry
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-ansible
Invalid
Low
git-harry
Icehouse
Fix Released
Low
Jesse Pretorius
Juno
Fix Released
Low
Jesse Pretorius
Trunk
Invalid
Low
git-harry

Bug Description

To duplicate this bug:

1. run ansible-playbook playbooks/monitoring/maas_cdm.yml
2. check the number of alarms for the memory check
3. run ansible-playbook playbooks/monitoring/maas_cdm.yml
4. check the number of alarms for the memory check

Each time maas_cdm is run a new alarm is created, this is because quotes are being added when testing for the alarm name.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/151675
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=dcff5594efbc7b892859a3f3fe93070ea23c6be3
Submitter: Jenkins
Branch: master

commit dcff5594efbc7b892859a3f3fe93070ea23c6be3
Author: git-harry <email address hidden>
Date: Fri Jan 30 15:23:07 2015 +0000

    Remove quotes to prevent duplicate alarms

    The memory alarm has a space in the name. This is causing it to get
    surrounded by single quotes, which in turn means grep finds no matches.

    This patch removes the quotes to prevent the issue. There may be a more
    general issue to address with quotes but it is outside the scope of
    this patch.

    Change-Id: I5ce02ee8499e593f5813ffe277d3869e7c28d691
    Closes-bug: #1416443

Changed in openstack-ansible:
status: New → Fix Committed
Changed in openstack-ansible:
importance: Undecided → Low
assignee: nobody → git-harry (git-harry)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (juno)

Fix proposed to branch: juno
Review: https://review.openstack.org/151723

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (icehouse)

Fix proposed to branch: icehouse
Review: https://review.openstack.org/151724

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (juno)

Reviewed: https://review.openstack.org/151723
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=54ddf43bca94410eb603557cc052a8f1c1f42737
Submitter: Jenkins
Branch: juno

commit 54ddf43bca94410eb603557cc052a8f1c1f42737
Author: git-harry <email address hidden>
Date: Fri Jan 30 15:23:07 2015 +0000

    Remove quotes to prevent duplicate alarms

    The memory alarm has a space in the name. This is causing it to get
    surrounded by single quotes, which in turn means grep finds no matches.

    This patch removes the quotes to prevent the issue. There may be a more
    general issue to address with quotes but it is outside the scope of
    this patch.

    Change-Id: I5ce02ee8499e593f5813ffe277d3869e7c28d691
    Closes-bug: #1416443
    (cherry picked from commit dcff5594efbc7b892859a3f3fe93070ea23c6be3)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (icehouse)

Reviewed: https://review.openstack.org/151724
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=e38c24c7f07439a538c38bd0aa3c9fb79b132649
Submitter: Jenkins
Branch: icehouse

commit e38c24c7f07439a538c38bd0aa3c9fb79b132649
Author: git-harry <email address hidden>
Date: Fri Jan 30 15:23:07 2015 +0000

    Remove quotes to prevent duplicate alarms

    The memory alarm has a space in the name. This is causing it to get
    surrounded by single quotes, which in turn means grep finds no matches.

    This patch removes the quotes to prevent the issue. There may be a more
    general issue to address with quotes but it is outside the scope of
    this patch.

    Change-Id: I5ce02ee8499e593f5813ffe277d3869e7c28d691
    Closes-bug: #1416443
    (cherry picked from commit dcff5594efbc7b892859a3f3fe93070ea23c6be3)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.