The pipeline should return a `zaqar.common.pipeline.Pipeline` object instead of a list

Bug #1433158 reported by Flavio Percoco
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Undecided
Doraly Navarro

Bug Description

An example of the current code:

 108 @decorators.lazy_property(write=False)
 109 def queue_controller(self):
 110 stages = _get_storage_pipeline('queue', self.conf)
 111 stages.append(self._storage.queue_controller)
 112 return stages

It should actually return an instance of `pipeline.Pipeline` which is our proxy for these plugins.

Changed in zaqar:
assignee: nobody → Doraly Navarro (doralynavarro)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to zaqar (master)

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

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

Reviewed: https://review.openstack.org/165153
Committed: https://git.openstack.org/cgit/openstack/zaqar/commit/?id=286e11f081329becd8abb049deac17c2fd00ebe4
Submitter: Jenkins
Branch: master

commit 286e11f081329becd8abb049deac17c2fd00ebe4
Author: dynarro <email address hidden>
Date: Tue Mar 17 18:20:29 2015 +0100

    Make the Pipeline inmutable

    This patch makes pipeline inmutable by removing the append method.
    the pipeline metadata now returns a pipeline instance.

    Change-Id: I9b90933a83ca1d423d3718f438d3717c39ef78d4
    Closes-Bug: #1433158

Changed in zaqar:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in zaqar:
milestone: none → kilo-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in zaqar:
milestone: kilo-3 → 2015.1.0
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.