quantum port-create --fixed-ip ignores additional invalid items
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Invalid
|
Undecided
|
Unassigned | ||
python-neutronclient |
Fix Released
|
Medium
|
Akihiro Motoki |
Bug Description
When doing a port-create - and possibly other requests - you can add any old rubbish to the request, and provided the minimum required info is present, there is no error :
ubuntu@
Created a new port:
+------
| Field | Value |
+------
| admin_state_up | True |
| device_id | |
| device_owner | |
| fixed_ips | {"subnet_id": "1972dfa7-
| id | 78b68e9f-
| mac_address | fa:16:3e:a0:b6:82 |
| name | |
| network_id | 20147087-
| status | ACTIVE |
| tenant_id | 5e0d8b02dd0a448
+------
ubuntu@
I have validated that the RUBBISH:SOMETHING is passed to the server, and is silently ignored.
This means that e.g. typos in commands can generate something other than the user was expecting :
ubuntu@
Created a new port:
+------
| Field | Value |
+------
| admin_state_up | True |
| device_id | |
| device_owner | |
| fixed_ips | {"subnet_id": "1972dfa7-
| id | 008336f8-
| mac_address | fa:16:3e:c6:ca:05 |
| name | |
| network_id | 20147087-
| status | ACTIVE |
| tenant_id | 5e0d8b02dd0a448
+------
I thought I was asking for 10.0.0.6, but I got 10.0.0.5
Would a 400 error not be more appropriate here ?
affects: | tempest → quantum |
Changed in neutron: | |
assignee: | nobody → Reedip (reedip-banerjee) |
assignee: | Reedip (reedip-banerjee) → nobody |
Changed in python-neutronclient: | |
assignee: | nobody → Reedip (reedip-banerjee) |
assignee: | Reedip (reedip-banerjee) → nobody |
Changed in python-neutronclient: | |
assignee: | Akihiro Motoki (amotoki) → li,chen (chen-li) |
Changed in python-neutronclient: | |
assignee: | li,chen (chen-li) → Akihiro Motoki (amotoki) |
Changed in python-neutronclient: | |
assignee: | Akihiro Motoki (amotoki) → li,chen (chen-li) |
Changed in python-neutronclient: | |
assignee: | li,chen (chen-li) → Akihiro Motoki (amotoki) |
Changed in python-neutronclient: | |
assignee: | Akihiro Motoki (amotoki) → Reedip (reedip-banerjee) |
Changed in python-neutronclient: | |
assignee: | Reedip (reedip-banerjee) → Akihiro Motoki (amotoki) |
Probably you are correct about this, but you should move this bug to the quantum projects side.