Ceilometer glance and nova notifications don't work

Bug #1433075 reported by Vadim Rovachev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Bartłomiej Piotrowski
5.1.x
Fix Committed
Critical
Bartłomiej Piotrowski
6.0.x
Fix Committed
Critical
Bartłomiej Piotrowski

Bug Description

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "6.1"
  api: "1.0"
  build_number: "200"
  build_id: "2015-03-15_22-54-44"
  nailgun_sha: "713e6684f9f54e29acfe6b8ebf641b9de2292628"
  python-fuelclient_sha: "cc64fff91fb0d78e5a02e7b93ceff224296f84fb"
  astute_sha: "93e427ac49109fa3fd8b0e1d0bb3d14092be2e8c"
  fuellib_sha: "553cb0cffa40a5f57313f962b6ec6a9bd89306ba"
  ostf_sha: "e86c961ceacfa5a8398b6cbda7b70a5f06afb476"
  fuelmain_sha: "c97fd8a789645bda48d06da224f994f8b52d82f5"

After merge patch https://review.openstack.org/#/c/161185/ glance and nova notifications don't work.

Parameter $ceilometer:

https://github.com/stackforge/fuel-library/blob/master/deployment/puppet/glance/manifests/notify/rabbitmq.pp#L54
https://github.com/stackforge/fuel-library/blob/master/deployment/puppet/openstack/manifests/nova/controller.pp#L94

is not transferred to these files

Tags: ceilometer
Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :

We've got broken a half of ceilometer notifications, so mark this bug as critical for ceilometer

Changed in fuel:
status: New → Confirmed
importance: Undecided → Critical
milestone: none → 6.1
assignee: nobody → Bartlomiej Piotrowski (bpiotrowski)
tags: added: ceilometer
Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :

As I can see, this patch https://review.openstack.org/#/c/161185/ was backported to 6.0 and 5.1 stable branches. So, need to fix it there too.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/165081

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/165081
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=690543dcf80120e0975ee1d8b4c38d071ba71893
Submitter: Jenkins
Branch: master

commit 690543dcf80120e0975ee1d8b4c38d071ba71893
Author: Bartłomiej Piotrowski <email address hidden>
Date: Tue Mar 17 15:02:36 2015 +0100

    Pass $ceilometer to openstack::glance and openstack::controller

    Commit I00db51ceb4121e73bfe7e256b895e4fae122a496 added possibility to
    control whether Ceilometer notifications are sent, but didn't add
    appropriate parameter to higher level classes, making notifications
    disabled even if Ceilometer is enabled.

    Additionally backport I3f2c29adbfb605ef288962f72ba81c11811cb357 to
    glance module.

    Change-Id: I2db7c13cd9c59e29e42e5000961eeedc6717e1ee
    Closes-Bug: 1433075
    Related-Bug: 1425982

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/6.0)

Fix proposed to branch: stable/6.0
Review: https://review.openstack.org/167127

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.1)

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/167128

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/6.0)

Reviewed: https://review.openstack.org/167127
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=847200e5e6830a9e57cc73e5783ad3a9469407fe
Submitter: Jenkins
Branch: stable/6.0

commit 847200e5e6830a9e57cc73e5783ad3a9469407fe
Author: Bartłomiej Piotrowski <email address hidden>
Date: Tue Mar 17 15:02:36 2015 +0100

    Pass $ceilometer to openstack::glance and openstack::controller

    Commit I00db51ceb4121e73bfe7e256b895e4fae122a496 added possibility to
    control whether Ceilometer notifications are sent, but didn't add
    appropriate parameter to higher level classes, making notifications
    disabled even if Ceilometer is enabled.

    Additionally backport I3f2c29adbfb605ef288962f72ba81c11811cb357 to
    glance module.

    Change-Id: I2db7c13cd9c59e29e42e5000961eeedc6717e1ee
    Closes-Bug: 1433075
    Related-Bug: 1425982

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.1)

Reviewed: https://review.openstack.org/167128
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=04d9daf599fd2a4feb9ae8770fe8ecaca34b114e
Submitter: Jenkins
Branch: stable/5.1

commit 04d9daf599fd2a4feb9ae8770fe8ecaca34b114e
Author: Bartłomiej Piotrowski <email address hidden>
Date: Tue Mar 17 15:02:36 2015 +0100

    Pass $ceilometer to openstack::glance and openstack::controller

    Commit I00db51ceb4121e73bfe7e256b895e4fae122a496 added possibility to
    control whether Ceilometer notifications are sent, but didn't add
    appropriate parameter to higher level classes, making notifications
    disabled even if Ceilometer is enabled.

    Additionally backport I3f2c29adbfb605ef288962f72ba81c11811cb357 to
    glance module.

    Change-Id: I2db7c13cd9c59e29e42e5000961eeedc6717e1ee
    Closes-Bug: 1433075
    Related-Bug: 1425982

Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #234 for CentOS. All OSTF Ceilometer notifications tests passed

"build_id": "2015-03-26_22-54-44", "ostf_sha": "a4cf5f218c6aea98105b10c97a4aed8115c15867", "build_number": "234", "release_versions": {"2014.2-6.1": {"VERSION": {"build_id": "2015-03-26_22-54-44", "ostf_sha": "a4cf5f218c6aea98105b10c97a4aed8115c15867", "build_number": "234", "api": "1.0", "nailgun_sha": "b163f6fc77d6639aaffd9dd992e1ad96951c3bbf", "production": "docker", "python-fuelclient_sha": "e5e8389d8d481561a4d7107a99daae07c6ec5177", "astute_sha": "3f1ece0318e5e93eaf48802fefabf512ca1dce40", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "320b5f46fc1b2798f9e86ed7df51d3bda1686c10", "fuellib_sha": "9c7716bc2ce6075065d7d9dcf96f4c94662c0b56"}}}, "auth_required": true, "api": "1.0", "nailgun_sha": "b163f6fc77d6639aaffd9dd992e1ad96951c3bbf", "production": "docker", "python-fuelclient_sha": "e5e8389d8d481561a4d7107a99daae07c6ec5177", "astute_sha": "3f1ece0318e5e93eaf48802fefabf512ca1dce40", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "320b5f46fc1b2798f9e86ed7df51d3bda1686c10", "fuellib_sha": "9c7716bc2ce6075065d7d9dcf96f4c94662c0b56"

Changed in fuel:
status: Fix Committed → Fix Released
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.