powervc-driver

New configuration strategy needed to adapt to oslo-messaging

Bug #1356225 reported by Le Tian Ren on 2014-08-13

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	powervc-driver	Fix Committed	High	Le Tian Ren

Bug Description

Check the blueprint:
https://blueprints.launchpad.net/powervc-driver/+spec/new-configuration-strategy-needed-to-adapt-to-oslo-messaging

Since the change has much impact on Chef deployment, BVT and other testing, it would be better to create another long-lived feature branch to contain it as well as later change of amqp changes(from qpid python lib to oslo.messaging), and rebase back on the main branch after well tests.

Le Tian Ren (gpanda+) on 2014-08-13

Changed in powervc-driver:
assignee:	nobody → Le Tian Ren (gpanda+)
importance:	Undecided → High
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-08-26: Fix proposed to powervc-driver (experimental)

Fix proposed to branch: experimental
Review: https://review.openstack.org/116919

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-09-01: Fix merged to powervc-driver (experimental)

Reviewed: https://review.openstack.org/116919
Committed: https://git.openstack.org/cgit/stackforge/powervc-driver/commit/?id=f1098c2253ceb8a74c647151f07d1faaaddc747e
Submitter: Jenkins
Branch: experimental

commit f1098c2253ceb8a74c647151f07d1faaaddc747e
Author: Le Tian Ren <email address hidden>
Date: Mon Aug 25 18:46:04 2014 +0800

New config strategy to adapt to oslo-messaging

    * Extract and remove --config-file amqp_openstack.conf(or
      amqp-openstack-neutron and --config-file amqp-powervc.conf from sys.argv to
      create 2 AMQP cfg.ConfigOpts objects without conflicting with other options
      parsing as before, particularly those OpenStack AMQP configurations in
      [DEFAULT] section of nova.conf, cinder.conf.

* adapt all affected code to the new config strategy

Implements: blueprint
https://blueprints.launchpad.net/powervc-driver/+spec/new-configuration-strategy-needed-to-adapt-to-oslo-messaging

Closes-Bug: 1356225
Change-Id: I61a4fd0c28148e31a8cb85acea3793f40028dfb4

Le Tian Ren (gpanda+) on 2014-09-01

Changed in powervc-driver:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-09-16: Fix proposed to powervc-driver (master)

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-09-17: Change abandoned on powervc-driver (master)

Change abandoned by Le Tian Ren (<email address hidden>) on branch: master
Review: https://review.openstack.org/121796
Reason: some process nit during merge branches