)]}'
{"specs/approved/expose-supported-power-states.rst":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/1734827"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This SPEC propose adding a new API to expose supported power states of nodes."},{"line_number":14,"context_line":"This API would be helpful to see if a power action is supported by a node"},{"line_number":15,"context_line":"because it depends on each power interface whether the power actions such as"},{"line_number":16,"context_line":"soft power off and soft reboot."}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_ec0ca93c","line":13,"updated":"2017-12-06 15:57:23.000000000","message":"nit s/propose/proposes/","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/1734827"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This SPEC propose adding a new API to expose supported power states of nodes."},{"line_number":14,"context_line":"This API would be helpful to see if a power action is supported by a node"},{"line_number":15,"context_line":"because it depends on each power interface whether the power actions such as"},{"line_number":16,"context_line":"soft power off and soft reboot."}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_563df6da","line":13,"in_reply_to":"df87a7cf_ec0ca93c","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This SPEC propose adding a new API to expose supported power states of nodes."},{"line_number":14,"context_line":"This API would be helpful to see if a power action is supported by a node"},{"line_number":15,"context_line":"because it depends on each power interface whether the power actions such as"},{"line_number":16,"context_line":"soft power off and soft reboot."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_2c51310f","line":15,"updated":"2017-12-06 15:57:23.000000000","message":"maybe \u0027because it depends on whether a power interface supports power actions such as soft power off and soft reboot\u0027.","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This SPEC propose adding a new API to expose supported power states of nodes."},{"line_number":14,"context_line":"This API would be helpful to see if a power action is supported by a node"},{"line_number":15,"context_line":"because it depends on each power interface whether the power actions such as"},{"line_number":16,"context_line":"soft power off and soft reboot."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_f6252ac3","line":15,"in_reply_to":"df87a7cf_2c51310f","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Ironic supports soft shutdown and soft reboot since Ocata. However, not all"},{"line_number":22,"context_line":"power interfaces support these new power actions. A new API would be necessary"},{"line_number":23,"context_line":"to see if a power action is supported by a node like boot devices."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_0c9c6dbe","line":23,"updated":"2017-12-06 15:57:23.000000000","message":"maybe s/node like boot devices/node. This proposed API is similar to the one for getting a node\u0027s supported boot devices./\n\nAnd you can link to https://developer.openstack.org/api-ref/baremetal/#get-supported-boot-devices","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Ironic supports soft shutdown and soft reboot since Ocata. However, not all"},{"line_number":22,"context_line":"power interfaces support these new power actions. A new API would be necessary"},{"line_number":23,"context_line":"to see if a power action is supported by a node like boot devices."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_c1329ef2","line":23,"in_reply_to":"df87a7cf_0c9c6dbe","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The new API will be introduced to get supported power states of nodes."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Alternatives"},{"line_number":31,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_4cb9152c","line":28,"updated":"2017-12-06 15:57:23.000000000","message":"Maybe add See the \u0027REST API impact\u0027 section for the details.\u0027","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The new API will be introduced to get supported power states of nodes."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Alternatives"},{"line_number":31,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_0142c63e","line":28,"in_reply_to":"df87a7cf_4cb9152c","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      \"soft power off\""},{"line_number":61,"context_line":"    ]"},{"line_number":62,"context_line":"  }"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Client (CLI) impact"},{"line_number":65,"context_line":"-------------------"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_0cdead10","line":63,"updated":"2017-12-06 15:57:23.000000000","message":"And of course, it will be available starting with a new Bare Metal API version.","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      \"soft power off\""},{"line_number":61,"context_line":"    ]"},{"line_number":62,"context_line":"  }"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Client (CLI) impact"},{"line_number":65,"context_line":"-------------------"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_e1448249","line":63,"in_reply_to":"df87a7cf_0cdead10","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"  This is similar to the existing command,"},{"line_number":95,"context_line":"  ``openstack baremetal node boot device show``. However the command"},{"line_number":96,"context_line":"  ``openstack baremetal node power show`` doesn\u0027t exist."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"* Add a new command for the supported power states::"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_6c9ad96e","line":96,"updated":"2017-12-06 15:57:23.000000000","message":"I think we should do this.\n\nIt means we\u0027d need to add another REST API (and we really have to, to do the supported thing): GET /v1/nodes/{node_ident}/states/power/\n\nAnd here, we\u0027d have (and this is consistent with support boot devices):\n- openstack baremetal node power show\n- openstack baremetal node power show --supported\n\nIn the Alternatives section (L30), you can add the other options that you suggested.","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"  This is similar to the existing command,"},{"line_number":95,"context_line":"  ``openstack baremetal node boot device show``. However the command"},{"line_number":96,"context_line":"  ``openstack baremetal node power show`` doesn\u0027t exist."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"* Add a new command for the supported power states::"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_416d2eca","line":96,"in_reply_to":"df87a7cf_6c9ad96e","updated":"2017-12-08 05:48:29.000000000","message":"Thank you for the advice. I agree on this option.","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"388bbc9fda5ab5236f606e124aab07c9e203b5c5","unresolved":false,"context_lines":[{"line_number":118,"context_line":"------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"The new API is available to see if a requested power action is supported or"},{"line_number":121,"context_line":"not. It might be helpful for the Nova driver."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Ramdisk impact"},{"line_number":124,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_ec9ca95e","line":121,"updated":"2017-12-06 15:57:23.000000000","message":"but there are no plans to use it, right? So I guess you could put \u0027None\u0027 here, before this sentence.","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"46cb7a11f87385f94826659e1f33f0e3546d21e1","unresolved":false,"context_lines":[{"line_number":118,"context_line":"------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"The new API is available to see if a requested power action is supported or"},{"line_number":121,"context_line":"not. It might be helpful for the Nova driver."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Ramdisk impact"},{"line_number":124,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_417bee79","line":121,"in_reply_to":"df87a7cf_ec9ca95e","updated":"2017-12-08 05:48:29.000000000","message":"Done","commit_id":"83bec40b710e3d5358ca3563d6213ef308986d0c"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"cb321c7ebcf0956efb5cbf0852eece96c8a5b11d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The new APIs will be introduced to get the current power status and the"},{"line_number":31,"context_line":"supported power states of nodes. See the \u0027REST API impact\u0027 section and for the"},{"line_number":32,"context_line":"details:"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"  - GET /v1/nodes/{node_ident}/states/power"}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_567a9fb9","line":31,"updated":"2017-12-11 15:39:49.000000000","message":"nit s/and for the/for/","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"cb321c7ebcf0956efb5cbf0852eece96c8a5b11d","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  - GET /v1/nodes/{node_ident}/states/power/supported"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"The new CLI will be also added for the new APIs. See the \u0027Client (CLI) impact\u0027"},{"line_number":38,"context_line":"section and for the details:"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  - ``openstack baremetal node power show [--supported] \u003cnode\u003e``"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_7662430c","line":38,"updated":"2017-12-11 15:39:49.000000000","message":"nit s/and for the/for/","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"cb321c7ebcf0956efb5cbf0852eece96c8a5b11d","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        \"soft power off\""},{"line_number":97,"context_line":"      ]"},{"line_number":98,"context_line":"    }"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"Client (CLI) impact"},{"line_number":101,"context_line":"-------------------"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_b9a3dcff","line":99,"updated":"2017-12-11 15:39:49.000000000","message":"Do we need to mention what happens if it isn\u0027t successful? eg, node doesn\u0027t exist, bad API version? I don\u0027t recall, and I think it will follow whatever we do for other commands now, so I\u0027m fine if it isn\u0027t mentioned here, but maybe others want to see it, I don\u0027t know.","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a93ec3d1235151c6644ca00162ede8033eaeecdf","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        \"soft power off\""},{"line_number":97,"context_line":"      ]"},{"line_number":98,"context_line":"    }"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"Client (CLI) impact"},{"line_number":101,"context_line":"-------------------"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_f7128409","line":99,"in_reply_to":"df87a7cf_b9a3dcff","updated":"2017-12-13 23:33:39.000000000","message":"Eh, I think it is fine to not to detail what we essentially have consensus on pattern wise elsewhere in the api.","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b3d77bab2a0f715a7a3af7bf66017a8efaac4ed6","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Without the ``--supported`` option, this command displays the power state of"},{"line_number":120,"context_line":"the specified node. When the ``--supported`` option is specified, this command"},{"line_number":121,"context_line":"displays the supported power states."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"RPC API impact"},{"line_number":124,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_93890276","line":121,"updated":"2017-12-12 07:33:09.000000000","message":"note: as we\u0027re reusing the same command, we have to make sure the output is consistent between the two cases (the same columns, etc)","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a93ec3d1235151c6644ca00162ede8033eaeecdf","unresolved":false,"context_lines":[{"line_number":210,"context_line":"Testing"},{"line_number":211,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"An API test will be added to Tempest."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":216,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"df87a7cf_37322ca9","line":213,"updated":"2017-12-13 23:33:39.000000000","message":"Just API with a fake driver or API in a functional sequence?\n\nI feel that fake is likely okay, but a full functional sequence does further help ensure that we don\u0027t break things down the road by accident.","commit_id":"2c12395ef313a4d380034a9e65b0f6af027ac022"}]}