Red Hat Bugzilla – Bug 1310803
[RFE] Add support for Nuage to puppet-neutron in 7.3
Last modified: 2016-09-22 12:29:00 EDT
Created attachment 1129411 [details]
Comparison between 7.3 and 8.0 versions
Description of problem:
Code to enable support for Nuage has been merged upstream. The code that was pulled-in is for 8.0 and we would need to pull in 7.3 version.
We need only the following two files from https://review.openstack.org/#/c/214798/
1. lib/puppet/provider/neutron_plugin_nuage/ini_setting.rb (patch set 8)
2. lib/puppet/type/neutron_plugin_nuage.rb (patch set 8)
Attaching the 7.3 and 8.0 version files for reference.
So, after digging around a bit, I now see that the referenced patch was added to puppet-neutron on master before liberty branched, meaning the patch (and a subsequent fix) are on the current upstream master and stable/liberty. To clarify the request here, you are saying basically everything in the master/liberty patch is unneeded _except_ for the two referenced files, is that correct? If so, I think the best path here, so this gets at least a bit of proper testing for kilo, is for you (or Ramkumar, who seems to have done the original work) to propose a patch upstream to the stable/kilo branch. I would think this should include at least the relevant spec (perhaps updated) from the original patch. I may be able to attempt to do the same thing later today, but I am not that familiar with nuage, so it would make more sense for someone who knows it to submit the patch, which I can then backport to OSP7 (and remove the previous two, as shown in ). Does that sound reasonable?
Has there been any progress on the backport, Lokesh?
We are working on verifying the changes with the latest OSP-D 7.3 build. After Nuage integration verification is done, will start working on up-streaming these changes.
Added Openstack gerrit review ID 298792, that has the backported changes to stable/kilo. Please update OSP7.3 to include the changes.
*** Bug 1323846 has been marked as a duplicate of this bug. ***
Need acks to get this into osp 7.x
At this point, we have a workaround to manually update the required packages, and given OSPD 7.x is not planned for further releases, let's just close this out.