[RFE] project ID is not verified when creating neutron resources

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

Bug Description

Environment: Ocata

According to current docs, in order to create auto-allocated network in tenant, following command should be invoked:

`neutron auto-allocated-topology-show [project_id]`

Unfortunatelly, this command does not verify if given project exists, so it's easy to create completely unusefull network components, even breaking naming convention for project_id field in database tables.

Example:

» neutron auto-allocated-topology-show 37-e088febf-64bd-4b5e-b4dd-1f5a80e
+------------+--------------------------------------+
| Field | Value |
+------------+--------------------------------------+
| id | 63f5f696-8a64-4bbb-af1e-b55a26f78f38 |
| project_id | 37-e088febf-64bd-4b5e-b4dd-1f5a80e |
| tenant_id | 37-e088febf-64bd-4b5e-b4dd-1f5a80e |
+------------+--------------------------------------+

»

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

None of the neutron commands do.

Changed in neutron:
status: New → Invalid
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

As admin, you could create a network for a tenant that does not exist. This is a 'well-known gotcha'. To address this we'd have to have neutron talk to Keystone to validate that the UUID is a valid one. Doable, but we never go around to do it for a variety of reasons (performance being one of them).

Changed in neutron:
status: Invalid → Opinion
importance: Undecided → Wishlist
summary: - [Get me a network] auto-allocated networks does not verify tenant ID
- while creating
+ project ID is not verified when creating neutron resources
tags: added: rfe
summary: - project ID is not verified when creating neutron resources
+ [RFE] project ID is not verified when creating neutron resources
Changed in neutron:
status: Opinion → Confirmed
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Let's gather feedback at the drivers meeting.

Changed in neutron:
status: Confirmed → Triaged
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
Revision history for this message
Miguel Lavalle (minsel) wrote :

This was discussed during the latest drivers meeting: http://eavesdrop.openstack.org/meetings/neutron_drivers/2017/neutron_drivers.2017-10-27-14.00.log.html#l-110. If the problem is exclusively typos when using the CLI, we could handle this in the client. Is this something that would work in your scenario / use case?

Revision history for this message
Crazik (crazik) wrote :

How do you want to check typo? Match given project-id to user's project-id?
I expected some checks like 'match if project exists with given project-id', then yes, my use case is safe then.

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

We could check that the project ID is an actual UUID. The most natural source of errors when specifying project IDs is unrecognized copy and paste errors.

Revision history for this message
Miguel Lavalle (minsel) wrote :

@Crazik

We discussed today this RFE again during the Drivers meeting. The OpenStack client (OSC) verifies the project id with Keystone for most commands. Auto allocated topology is an exception. We think that should be considered as a bug in OSC. If this were fixed in OSC, the use case you describe here would be supported. Have you considered filing a bug with OSC?

Revision history for this message
Miguel Lavalle (minsel) wrote :

We revisited this RFE on December 22nd meeting. If by next meeting, January 5th, we don't hear back from submitter, we will assume this can be solved by OSC and classify it as an OSC bug

Miguel Lavalle (minsel)
affects: neutron → osc
affects: osc → neutron
Changed in neutron:
status: Triaged → 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.