Merge lp://staging/~snaiksat/neutron/sumit-fix-profileclient into lp://staging/~cisco-openstack/neutron/plugin-framework

Proposed by Sumit Naiksatam
Status: Merged
Merged at revision: 43
Proposed branch: lp://staging/~snaiksat/neutron/sumit-fix-profileclient
Merge into: lp://staging/~cisco-openstack/neutron/plugin-framework
Diff against target: 24 lines (+3/-3)
1 file modified
quantum/plugins/cisco/ucs/cisco_ucs_network_driver.py (+3/-3)
To merge this branch: bzr merge lp://staging/~snaiksat/neutron/sumit-fix-profileclient
Reviewer Review Type Date Requested Status
Edgar Magana (community) Approve
Review via email: mp+70237@code.staging.launchpad.net

Commit message

The port profile client name generated by the UCS Network Driver may exceed 17 characters, at which point the creation of the port profile client fails. The fix committed limits this name to 16 characters.

Description of the change

The port profile client name generated by the UCS Network Driver may exceed 17 characters, at which point the creation of the port profile client fails. The fix committed limits this name to 16 characters.

To post a comment you must log in.
44. By Sumit Naiksatam

Earlier fix resulted in a different issue (profile client name, was also being used as profile name, hence breaking).

Revision history for this message
Edgar Magana (emagana) wrote :

The fix is working properly.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches