)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"de94bae2b0da9f11b7ee300eab7eb601124e3e8d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Include usage in \u0027inventory list\u0027, \u0027inventory show\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It seems odd to show everything about a given resource provider except"},{"line_number":10,"context_line":"usage. Correct this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Iafedac3ff7a5ae60e6027741c44904babf64d3f1"},{"line_number":13,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f560f44_9da516a5","line":10,"updated":"2020-08-06 20:09:27.000000000","message":"In what way is it odd? Can you describe an example or a use case/reason you wished to see usage info when listing or showing inventory?\n\nI want to understand the motivation for adding this being that there are already commands around showing usage:\n\nhttps://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-usage-show\n\nhttps://docs.openstack.org/osc-placement/latest/cli/index.html#resource-usage-show","commit_id":"b16ce1edc6435cb93fa50757c1a1e3801cdd6e5a"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"69497734225d94ac863d9a71423063d0bda10e1f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Include usage in \u0027inventory list\u0027, \u0027inventory show\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It seems odd to show everything about a given resource provider except"},{"line_number":10,"context_line":"usage. Correct this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Iafedac3ff7a5ae60e6027741c44904babf64d3f1"},{"line_number":13,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f560f44_6d416aa9","line":10,"in_reply_to":"9f560f44_9da516a5","updated":"2020-08-07 09:53:02.000000000","message":"\u003e In what way is it odd? Can you describe an example or a use\n \u003e case/reason you wished to see usage info when listing or showing\n \u003e inventory?\n\nUpdated the commit message to better state my reasoning, along with my intent to deprecate those separate commands in the future (I don\u0027t think they should exist as separate commands). As I mentioned on IRC, I run into this every time I need to do a bit of querying of placement and I doubt I\u0027m alone. I can\u0027t think of a practical case where you would be querying available inventory from placement and wouldn\u0027t find the usage information helpful. osc-placement is mostly a debugging tool for admins, after all.\n\n \u003e I want to understand the motivation for adding this being that\n \u003e there are already commands around showing usage:\n \u003e \n \u003e https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-usage-show\n\nAs noted above, IMO this command should not exist, at least not in its current form. Its output should be part of \u0027resource provider inventory (list|show)\u0027.\n\nIf we were to retain this as a separate command, I\u0027d like to provide far more detailed output that we currently do. Namely, I\u0027d like to show a breakdown of usage by each allocation. This is currently available, but only through the rather obscure \u0027resource provider show $RP_UUID --allocations\u0027 command.\n\n \u003e https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-usage-show\n\nThis is different. This shows total usage for a given project, not per-RP usage. This should stay unless/until we have an API and related command to describe total amounts of inventory available for use by the project, in which case we should fold this command into that one.","commit_id":"b16ce1edc6435cb93fa50757c1a1e3801cdd6e5a"}]}