[RFE] node name isn't supported in the REST API for creating a port

Bug #1439901 reported by Ruby Loo
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Ironic
Triaged
Wishlist
Anup

Bug Description

A port can be created via a POST /v1/ports, with 'node_uuid' and 'address' specified in the body.

Now that we have logical node names, we should allow the port creation to accept the name of the node instead of the UUID of the node. Eg, add 'node' with either a UUID or name as its value, and deprecate 'node_uuid'.

Although a node name could be specified, the port would continue to be associated with a node UUID in the db; that won't change.

Sinval (sinvalneto01)
Changed in ironic:
assignee: nobody → Sinval (sinvalneto01)
Sinval (sinvalneto01)
Changed in ironic:
status: Confirmed → In Progress
Revision history for this message
Sinval (sinvalneto01) wrote :

Change submitted for review:
https://review.openstack.org/#/c/193439/

Revision history for this message
Tan Lin (tan-lin-good) wrote :

Hi Sinval, are you still working on this patch?

Revision history for this message
Sinval (sinvalneto01) wrote :

Hi Tan Lin, actually not, feel free to work on it.

Dmitry Tantsur (divius)
Changed in ironic:
assignee: Sinval (sinvalneto01) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic (master)

Change abandoned by Dmitry Tantsur (<email address hidden>) on branch: master
Review: https://review.openstack.org/193439
Reason: Hi! This patch hasn't been updated for more than a year, so I'm abandoning it.

Dmitry Tantsur (divius)
Changed in ironic:
status: In Progress → Triaged
tags: added: api
Revision history for this message
Ruby Loo (rloo) wrote :

I changed this to be an rfe (or do folks think it isn't) since it affects the API. I'm good with this being rfe-approved w/o needing a spec.

summary: - node name isn't supported in the REST API for creating a port
+ [RFE] node name isn't supported in the REST API for creating a port
tags: added: rfe
Revision history for this message
Jay Faulkner (jason-oldos) wrote :

Approved per conversation in IRC with Lucas and Ruby. We all agree it's a useful thing.

To be clear: this is just an API shortcut, the port will still be associated with the UUID in the DB.

tags: added: rfe-approved
removed: rfe
Ruby Loo (rloo)
description: updated
Ruby Loo (rloo)
Changed in ironic:
assignee: nobody → Anup (anup-d-navare)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ironic (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/405586

Revision history for this message
Julia Kreger (juliaashleykreger) wrote :

This is still valid. As of 20240202, only a node UUID is permitted.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.