UCS Manager ML2 driver enhancement to use python context manager

Bug #1498052 reported by Sandhya Dasu on 2015-09-21
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-cisco
Low
Sandhya Dasu

Bug Description

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

Sandhya Dasu (sadasu) on 2015-09-21
Changed in networking-cisco:
assignee: nobody → Sandhya Dasu (sadasu)
Changed in networking-cisco:
status: New → In Progress

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) on 2015-11-04
Changed in networking-cisco:
milestone: none → 2.0.0
importance: Undecided → Low
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

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.

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  Edit
Everyone can see this information.

Other bug subscribers