Inconsistent params names in the API
Bug #1259619 reported by
Lucas Alvares Gomes
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Low
|
Imre Farkas |
Bug Description
There's many methods on the API that is suppose to receive the UUID of that resource (Nodes, Chassis or Ports), in the moment there's no consistent with the name of those paremeters and it's bad for the auto generated documentation.
For e.g in the v1/node.py[1] file:
Some methods use 'node_id' and some others use 'uuid', the latter is even worst because conflicts with the name of the uuid stdlib.
So this should be fixed, I suggest using "*_uuid" (eg node_uuid, chassis_uuid or port_uuid) for all those parameters since our API should only accept UUIDs (numeric IDs are internal only)
[1] https:/
Changed in ironic: | |
milestone: | none → icehouse-2 |
assignee: | nobody → Lucas Alvares Gomes (lucasagomes) |
importance: | Undecided → Low |
assignee: | Lucas Alvares Gomes (lucasagomes) → Imre Farkas (ifarkas) |
description: | updated |
Changed in ironic: | |
status: | New → Triaged |
Changed in ironic: | |
status: | Fix Committed → Fix Released |
Changed in ironic: | |
milestone: | icehouse-2 → 2014.1 |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/64684
Review: https:/