Node detailed list endpoint is not RESTful

Bug #1662921 reported by Rob Cresswell
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Low
Sam Betts

Bug Description

/nodes/detail isn't a correctly structured REST endpoint. This endpoint is indicating that you want to act upon a 'node' with an id of 'detail'. Listing nodes with more detail should just be passed as a parameter i.e. GET /nodes?detail=true. Interestingly, I assume this means a node couldn't be called 'detail', although I haven't tested this out.

This can actually be extended to a lot of the API, but I wanted to control the scope of the bug. For example, most of the /nodes/{node_ident}/management/ should just be PUT/PATCH on /nodes/{node_ident}.

Tags: api
description: updated
Sam Betts (sambetts)
Changed in ironic:
assignee: nobody → Sam Betts (sambetts)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: New → In Progress
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → Low
tags: added: api
Dmitry Tantsur (divius)
Changed in ironic:
status: In Progress → Fix Released
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.