Fibre Channel Zone Manager hacking violations

Bug #1285029 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Walt Boring

Bug Description

Some hacking violations have come up after the FCZM landed and need to be addressed. John commented on several of them in the post merge review for this patch.

https://review.openstack.org/#/c/76011

Tags: zonemanager
Revision history for this message
Walt Boring (walter-boring) wrote :

please fix before RC1

Changed in cinder:
assignee: Walt Boring (walter-boring) → Santhoshkumar Kolathur (skolathu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: Santhoshkumar Kolathur (skolathu) → Walt Boring (walter-boring)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/79373
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=6d9bb4bdc915b1ed6ac9d49012eb185a369cc624
Submitter: Jenkins
Branch: master

commit 6d9bb4bdc915b1ed6ac9d49012eb185a369cc624
Author: Walter A. Boring IV <email address hidden>
Date: Mon Mar 10 10:27:43 2014 -0700

    Fixed some FCZM unit tests hacking issues

    This patch is a follow up to a review that merged
    included and had some issues raised after it was merged.
    This patch includes some hacking cleanup and some
    unit test cleanup for the Fibre Channel Zone Manager.

    Change-Id: Ibb6974b2279f08720c2469893245bc888d35899c
    Closes-Bug: #1285029

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-rc1 → 2014.1
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.