Not existing pool can be passed to floating_ip_bulk

Bug #1377072 reported by Haiwei Xu
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Won't Fix
Low
Unassigned

Bug Description

Default floating-ip-pool only contains 'public'

$ nova floating-ip-pool-list
+--------+
| name |
+--------+
| public |
+--------+

But when creating new floating-ip-bulk with '--pool' option which does not exist, the pool can be set successfully.

$ nova floating-ip-bulk-create 192.0.50.0/25 --pool private

| - | 192.0.50.1 | - | private | eth0 |
| - | 192.0.50.2 | - | private | eth0 |
| - | 192.0.50.3 | - | private | eth0 |
| - | 192.0.50.4 | - | private | eth0 |
| - | 192.0.50.5 | - | private | eth0 |
| - | 192.0.50.6 | - | private | eth0 |
| - | 192.0.50.7 | - | private | eth0 |
| - | 192.0.50.8 | - | private | eth0 |
| - | 192.0.50.9 | - | private | eth0 |
| - | 192.0.50.10 | - | private | eth0 |
| - | 192.0.50.11 | - | private | eth0 |
| - | 192.0.50.12 | - | private | eth0 |
| - | 192.0.50.13 | - | private | eth0 |

Tags: network
Haiwei Xu (xu-haiwei)
Changed in nova:
assignee: nobody → Haiwei Xu (xu-haiwei)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/125915
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
Joe Gordon (jogo) wrote :

patch was abandoned, not in progress anymore

Changed in nova:
status: In Progress → Confirmed
assignee: Haiwei Xu (xu-haiwei) → nobody
tags: added: low-hanging-fruit
Changed in nova:
assignee: nobody → tcs_openstack_group (tcs-openstack-group)
assignee: tcs_openstack_group (tcs-openstack-group) → Amandeep (rattenpal-amandeep)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Amandeep (<email address hidden>) on branch: master
Review: https://review.openstack.org/139556

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Removing "In Progress" status and assignee as change is abandoned.

Changed in nova:
status: In Progress → Confirmed
assignee: Amandeep (rattenpal-amandeep) → nobody
dnovosel (dnovosel)
Changed in nova:
assignee: nobody → dnovosel (dnovosel)
dnovosel (dnovosel)
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
dnovosel (dnovosel) wrote :

I'm not sure this is a bug. If the pool doesn't exist it is created with the name provided using the pool flag. What's wrong with this approach? As far as I know there is no other mechanism to create a pool.

Changed in nova:
status: In Progress → Confirmed
importance: Undecided → Low
Revision history for this message
dnovosel (dnovosel) wrote :

Why was this moved to confirmed from in progress? I have a fix that works [although some test cases still fail, but I can fix these].

All that said, as I stated previously, I don't think this is a bug. What forum is the correct one to discuss this?

Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote :

The age and number of assignee changes suggest that this isn't really
a low-hanging-fruit. I'm removing the tag to not confuse new folks who
are searching for that tag.

tags: added: network
removed: low-hanging-fruit
Revision history for this message
Sujitha (sujitha-neti) wrote :

There are no open reviews for this bug report since a long time.
I'm changing the status from "In Progress" to "New"
(as it was never confirmed. It was just moved from InProgress to Confirmed)
and remove the assignee.
Feel free to add yourself as assignee and push a review for it.

Changed in nova:
assignee: dnovosel (dnovosel) → nobody
status: Confirmed → New
Revision history for this message
Sean Dague (sdague) wrote :

This is part of nova-net that is now deprecated. We probably won't fix bugs like these.

Changed in nova:
status: New → Won't Fix
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.