Invalid tenant ID is allowed when creating a network

Bug #1290408 reported by Gastón Severina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Undecided
Unassigned

Bug Description

The following will create a network for an invalid tenant ID:

$neutron net-create --tenant_id xxxxx test

Tenant ID should be validated before creating any entity (network, port, subnet, etc)

Changed in neutron:
assignee: nobody → Gastón Severina (gseverina)
Changed in neutron:
status: New → Confirmed
Changed in neutron:
assignee: Gastón Severina (gseverina) → Harikrishna (harikrishna-mallavolu)
Revision history for this message
Harikrishna (harikrishna-mallavolu) wrote :

I have fix for this problem but the test cases are failing because the tenant-id in test case's are randomly generated, I replaced the randomly generated tenant-id with the available tenant-id. Surprisingly the test's passed. For the fix to be praposed do I need to update the test-suite as well ??? can any one suggest me on this. Thanks.

Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

I think this should go through the discussion first.

Neutron in my opinion should not care about validity of tenantIDs, especially because these are external entities that are theoretically a subject to change.
Also, that allows to use special tenant_ids for management tasks.

My opinion is that neutron should not check tenant_id validity.

Changed in neutron:
status: In Progress → Incomplete
Changed in neutron:
assignee: Harikrishna (harikrishna-mallavolu) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

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