[RFE] port list should have --node argument

Bug #1616242 reported by Ruby Loo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ironicclient
Won't Fix
Wishlist
Ruby Loo

Bug Description

The ironic REST API GET /v1/ports has 'node' (value is name or UUID) and 'node_uuid' query parameters [1], used to filter the results to only ports of the specified node. The ironic CLI 'ironic port-list' should also allow for that via an optional --node argument (which maps to the API's 'node' query parameter).

We already have 'ironic node-port-list' which provides the same functionality. Do we want to provide two different CLIs to do the same thing? I am hoping that one day in the future, everyone will be using the openstackclient plugin, and we will deprecate and then remove the ironic CLI. So I'm not too worried about the ironic CLI. The actual intention of this RFE is to add --node to the python API, so that we could implement 'openstack baremetal port list --node' instead of implementing 'openstack baremetal node port list'. The latter doesn't seem to fit with the way the OSC commands are meant to be structured [2].

[1] http://developer.openstack.org/api-ref/baremetal/index.html?expanded=list-ports-detail#list-ports

[2] http://docs.openstack.org/developer/python-openstackclient/commands.html

Tags: rfe
Ruby Loo (rloo)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ironicclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/359520

Changed in python-ironicclient:
status: New → In Progress
Ruby Loo (rloo)
description: updated
Ruby Loo (rloo)
Changed in python-ironicclient:
status: In Progress → Won't Fix
Revision history for this message
Ruby Loo (rloo) wrote :

Not going to do this, since there doesn't seem to be anyone that really wants it.

See Dmitry's comment in https://review.openstack.org/#/c/359520/ -- he isn't sure we should do it since we already have an equivalent, 'ironic node-port-list'.

The email thread didn't receive many comments; Vlad was in favour of it, but didn't feel strongly about it: http://lists.openstack.org/pipermail/openstack-dev/2016-August/102461.html

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-ironicclient (master)

Change abandoned by Ruby Loo (<email address hidden>) on branch: master
Review: https://review.openstack.org/359520
Reason: I marked the rfe/bug as 'Won't Fix' and am abandoning this, since no one replied with a 'yes, i'd really love to have this' :)

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.