UCS Manager ML2 driver enhancement to use python context manager

Bug #1498052 reported by Sandhya Dasu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-cisco
Fix Committed
Low
Sandhya Dasu

Bug Description

Use Python context manager to reduce repetitive code in the ML2 UCS Manager plugin.

Sandhya Dasu (sadasu)
Changed in networking-cisco:
assignee: nobody → Sandhya Dasu (sadasu)
Changed in networking-cisco:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-cisco (master)

Reviewed: https://review.openstack.org/225936
Committed: https://git.openstack.org/cgit/openstack/networking-cisco/commit/?id=d4e6b1a63c029fc0105675b3e4edacb2afeb2443
Submitter: Jenkins
Branch: master

commit d4e6b1a63c029fc0105675b3e4edacb2afeb2443
Author: Sandhya Dasu <email address hidden>
Date: Mon Sep 21 12:30:35 2015 -0400

    Enhancements to reduce duplicate code in UCSM plugin

    Adds python context manager support to reduce duplicate
    code and improve readability.

    Change-Id: I8d636a143fcb7d0ec5b5eebcb2372c425ad1f55a
    Closes-Bug: #1498052

Changed in networking-cisco:
status: In Progress → Fix Committed
Sam Betts (sambetts)
Changed in networking-cisco:
milestone: none → 2.0.0
importance: Undecided → Low
Revision history for this message
Abishek Subramanian (absubram) wrote :

Would like to back port this to stable/kilo branch.
Back port of bug 1491025 to stable/kilo is critical because of scale reasons. That patch however is dependent on this being back ported first.
Thoughts?

tags: added: kilo-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to networking-cisco (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/246008

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on networking-cisco (stable/kilo)

Change abandoned by Abishek Subramanian (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/246008
Reason: Removing proposal for backporting to stable/kilo. Realize this change cannot be accepted as a candidate.
Hence abandoning this.

Revision history for this message
Abishek Subramanian (absubram) wrote :

Removing proposal for backporting to stable/kilo. Realize this change cannot be accepted as a candidate.

tags: removed: kilo-backport-potential
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.