notification_driver typo in puppet-glance notify rabbit

Bug #1312658 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-glance
Fix Released
Undecided
Bogdan Dobrelya

Bug Description

"notification_driver = messaging" should be used in IceHouse due to "notifier_strategy = rabbit" deprecation.
But there is wrong stanza in puppet-glance now: notifier_driver = rabbit
See
https://github.com/openstack/glance/blob/master/etc/glance-api.conf
https://github.com/openstack/glance/blob/master/glance/notifier.py#L47
https://github.com/openstack/glance/blob/master/glance/notifier.py#L68-77

Changed in puppet-glance:
assignee: nobody → Bogdan Dobrelya (bogdando)
status: New → In Progress
description: updated
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to puppet-glance (master)

Reviewed: https://review.openstack.org/90022
Committed: https://git.openstack.org/cgit/stackforge/puppet-glance/commit/?id=d40f3d8622378c08aa89a557b75cfa7d5a0525cb
Submitter: Jenkins
Branch: master

commit d40f3d8622378c08aa89a557b75cfa7d5a0525cb
Author: Bogdan Dobrelya <email address hidden>
Date: Thu Apr 24 10:50:43 2014 +0300

    Add multiple rabbit hosts support

    * Use new oslo.messaging conf options for rabbit_hosts and
      rabbit_ha_queues for glance::notify::rabbitmq class
    * Warn on deprecated rabbit_durable_queues and use amqp_* instead
    * Fix/add rspecs
    * Fix a mess with wrong notification_driver usage
    * Update README with usage example

    DocImpact
    Closes-bug: #1259193
    Closes-bug: #1312054
    Closes-bug: #1312658
    Closes-bug: #1313553

    Change-Id: I16b8e71db20602986a7afe8d24e7faba91a0997f
    Signed-off-by: Bogdan Dobrelya <email address hidden>

Changed in puppet-glance:
status: In Progress → Fix Committed
Mathieu Gagné (mgagne)
Changed in puppet-glance:
milestone: none → 4.0.0
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.