Segmentation_id beyond specified range does not raise an error

Bug #1332475 reported by sandeep mane
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Wishlist
Unassigned

Bug Description

Problem in segment-Id range, it should belongs to given range,
command to reproduce :

neutron net-create demo_net --provider:network_type gre --provider:Segmentation_id 2000

right now its allowing to create.

expected:
neutron should give an error if we give Segmentation_id beyond specified range.

Changed in neutron:
assignee: nobody → Puneet Arora (puneet-arora)
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

I think that was done specifically to allow admin to allocate networks with specific segmentation_ids while regular ranges work with those IDs that are allocated.

I believe that is intentional and should be restricted.

Changed in neutron:
status: New → Opinion
Changed in neutron:
status: Opinion → Incomplete
assignee: Puneet Arora (puneet-arora) → nobody
tags: added: low-hanging-fruit
Lubosz Kosnik (diltram)
Changed in neutron:
assignee: nobody → Lubosz Kosnik (lubosz-kosnik)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Incomplete → In Progress
Revision history for this message
Sean M. Collins (scollins) wrote : Re: neutron should give an error if we give Segmentation_id beyond specified range

This is still an opinion bug, looks like changing it to invalid caused someone to think it should be worked on.....

Changed in neutron:
status: In Progress → Opinion
Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/240882
Reason: As per reviewers' comments, there is no need to fix the intended behavior. That said, we should perhaps document it somewhere so that it's clear.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

The segmentation range is to be used internally by Neutron for tenant networks, the admin instead is in full charge of his/her own destiny.

We should probably document this somewhere, but in practice no fix is required.

summary: - neutron should give an error if we give Segmentation_id beyond specified
- range
+ Segmentation_id beyond specified range does not raise an error
Changed in neutron:
status: Opinion → Incomplete
assignee: Lubosz Kosnik (diltram) → nobody
Revision history for this message
Kyle Mestery (mestery) wrote :

Lets make this "Won't Fix" instead then.

Changed in neutron:
status: Incomplete → 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.