[API] filter multi floating ranges in case of neutron

Bug #1490657 reported by Aleksey Kasatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Unassigned

Bug Description

Multiple floating ranges are not supported in manifests nor in UI. But API allows to set it. Nailgun's serializer serializes only first range when several ranges are set. See https://bugs.launchpad.net/fuel/+bug/1490578

So, it's required to filter multiple floating ranges in case of Neutron in API (reject such request or take jut first range fom provided list).

Dmitry Pyzhov (dpyzhov)
Changed in fuel:
importance: Undecided → High
Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

Should be fixed in validator by rejecting multiple floating ranges for Neutron, but only in case bug #1490578 is moved to 8.0.

Artem Roma (aroma-x)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Artem Roma (aroma-x)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/219255

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Artem Roma (aroma-x) → Ivan Kliuk (ivankliuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/219255
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=e2ea8c0d8f2c413b3c281f47eb97f4c3cb3e1011
Submitter: Jenkins
Branch: master

commit e2ea8c0d8f2c413b3c281f47eb97f4c3cb3e1011
Author: Artem Roma <email address hidden>
Date: Tue Sep 1 15:22:50 2015 +0300

    Prohibit setting of multiple floating IP ranges for Neutron

    We should forbid settings of multiple floating IP-address ranges for Neutron
    networks as there's not any support of processing such data via both
    Puppet manifests and UI. Thus, now validation error occurs when user tries
    to submit aforementioned network parameters.

    Change-Id: I9a3f4a1cb0438f276611f7d09e6a8021ffadf299
    Closes-Bug: #1490657

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on-verification
Revision history for this message
Alexander Saprykin (cutwater) wrote :

Verified on ISO 288

network_1.yaml

...
  floating_ranges:
  - - 172.16.0.130
    - 172.16.0.140
  - - 172.16.0.141
    - 172.16.0.150
...

[root@fuel ~]# fuel network --env 1 --upload
DEPRECATION WARNING: /etc/fuel/client/config.yaml exists and will be used as the source for settings. This behavior is deprecated. Please specify the path to your custom settings file in the FUELCLIENT_CUSTOM_SETTINGS environment variable.
Error uploading configuration: Setting of multiple floating IP ranges is prohibited

tags: removed: on-verification
Changed in fuel:
status: Fix Committed → Fix Released
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.