notification_driver typo in puppet-glance notify rabbit

Bug #1312658 reported by Bogdan Dobrelya on 2014-04-25
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-glance
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

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) on 2015-03-27
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  Edit
Everyone can see this information.

Other bug subscribers