[openstack-dev] [puppet] [neutron] Serious bug in puppet neutron cli json output parsing.
colleen at gazlene.net
Wed Dec 30 16:53:53 UTC 2015
On Wed, Dec 30, 2015 at 8:37 AM, Sofer Athlan-Guyot <sathlang at redhat.com>
> I have added neutron people as they may help to find a solution.
> After banging my head against the wall for a whole afternoon I
> discovered a serious bug in puppet neutron module.
> I not going to repeat here the detail of the bug report. Basically:
> - neutron_port
> - neutron_subnet
> - neutron_router
> - neutron_network
> may break idempotency randomly and won't work at all when clifftablib is
> removed from the package dependency of python-openstackclient
> So the problem is that neutron cli json output on liberty (at least, and
> maybe before) is not consistent and may come from cliff or clifftablib.
> I didn't test it but the same may apply to openstack cli. As we don't
> use the openstack cli json output it's not a issue (for puppet modules)
> The available solution I can see are:
> 1. go back to parsing csv, shell output (revert )
> 2. find a way to leverage openstacklib parsing for neutron as well
> 3. keep json and parse the right output (cliff) and find a way to make
> sure that it is always used by stevedore
> 4. ?
> Last I checked, which was quite a while ago, openstackclient didn't
support everything we were using from the neutron client. I would like to
reevaluate that and go with option 2 if we can. Otherwise option 1 seems
> From my point of view 3) is not a option, but other may disagree.
> The problem is tricky and the fact that the CI cannot detect this is
> So before Mitaka, the json parsing should go. I would love to see an
> interface that all puppet modules would use (solution 2). The current
> openstacklib parses openstack client well enough. The neutron command
> is not that different and I think there is space for code reuse. This
> would be a long term solution. It would bring the advantage of having
> only one interface to change if it was decided to use the API directly
> for instance
> In the meantime, a quick solution to this bug must be found.
> Looking forward to your comments.
>  https://bugs.launchpad.net/puppet-neutron/+bug/1530163
>  https://bugs.launchpad.net/python-neutronclient/+bug/1529914
>  https://review.openstack.org/#/c/238156/
>  https://review.openstack.org/#/c/262223/
> Sofer Athlan-Guyot
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OpenStack-dev