)]}'
{"specs/liberty/ironic-generic-raid-interface.rst":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b325b40c4a5ffbe3d78d4a00f55a83f1d3cc4ccf","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"* The RAID configuration information is stored as JSON in"},{"line_number":182,"context_line":"  ``node.target_raid_config`` field. Operator can PUT a new value here at any"},{"line_number":183,"context_line":"  time, which is compared to ``node.raid_config`` during zapping and cleaning,"},{"line_number":184,"context_line":"  and driver may apply changes only in those stages."},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"* New driver interface called ``RAIDInterface`` will be provided for RAID"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_ab186e76","line":183,"updated":"2015-06-22 14:28:21.000000000","message":"It sounds like the operator can PUT it directly onto the node representation:\n\n PUT {...., \u0027target_raid_config\u0027: {...}} v1/nodes/NNNN\n\nWhich is not correct. Operators should PUT at v1/nodes/NNNN/states/raid","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"b773c33bf98904951e5d968a27c8d019addc5073","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"* The RAID configuration information is stored as JSON in"},{"line_number":182,"context_line":"  ``node.target_raid_config`` field. Operator can PUT a new value here at any"},{"line_number":183,"context_line":"  time, which is compared to ``node.raid_config`` during zapping and cleaning,"},{"line_number":184,"context_line":"  and driver may apply changes only in those stages."},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"* New driver interface called ``RAIDInterface`` will be provided for RAID"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_cebc846e","line":183,"in_reply_to":"fa32b979_ab186e76","updated":"2015-06-24 08:47:19.000000000","message":"Done.  Just will mention more on it down.","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b325b40c4a5ffbe3d78d4a00f55a83f1d3cc4ccf","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"* A new database field, ``node.target_raid_config``, will store the pending"},{"line_number":232,"context_line":"  RAID configuration to be applied during zapping or cleaning. This will be a"},{"line_number":233,"context_line":"  JSON dictionary."},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"* A new database field, ``node.raid_config``, will store the last applied RAID"},{"line_number":236,"context_line":"  configuration. This will also contain the timestamp of when this"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_d077a1d2","line":233,"updated":"2015-06-22 14:28:21.000000000","message":"\"... This filed will be read-only.\"\n\nUpdates to it goes at PUT v1/nodes/NNNN/states/raid","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"b773c33bf98904951e5d968a27c8d019addc5073","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"* A new database field, ``node.target_raid_config``, will store the pending"},{"line_number":232,"context_line":"  RAID configuration to be applied during zapping or cleaning. This will be a"},{"line_number":233,"context_line":"  JSON dictionary."},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"* A new database field, ``node.raid_config``, will store the last applied RAID"},{"line_number":236,"context_line":"  configuration. This will also contain the timestamp of when this"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_8e699cfa","line":233,"in_reply_to":"fa32b979_d077a1d2","updated":"2015-06-24 08:47:19.000000000","message":"Done","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b325b40c4a5ffbe3d78d4a00f55a83f1d3cc4ccf","unresolved":false,"context_lines":[{"line_number":283,"context_line":""},{"line_number":284,"context_line":"  If accepted by the driver, this information will be stored in the"},{"line_number":285,"context_line":"  ``node.target_raid_config`` field and exposed in the same manner as the power"},{"line_number":286,"context_line":"  and provision states. In other words, it may be retrieved either within the"},{"line_number":287,"context_line":"  detailed view of a ``node``, or by either of the following::"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    GET /v1/nodes/NNNN/states"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_30860ddc","line":286,"updated":"2015-06-22 14:28:21.000000000","message":"How does the user gets the current raid config?","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"653199eca93ab43f821de2e4b6b5d8d09ea2ebd4","unresolved":false,"context_lines":[{"line_number":283,"context_line":""},{"line_number":284,"context_line":"  If accepted by the driver, this information will be stored in the"},{"line_number":285,"context_line":"  ``node.target_raid_config`` field and exposed in the same manner as the power"},{"line_number":286,"context_line":"  and provision states. In other words, it may be retrieved either within the"},{"line_number":287,"context_line":"  detailed view of a ``node``, or by either of the following::"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    GET /v1/nodes/NNNN/states"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_2bf69ef9","line":286,"in_reply_to":"fa32b979_30860ddc","updated":"2015-06-22 14:29:17.000000000","message":"ops ignore this, it was before I saw there\u0027s a node.raid_config","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b325b40c4a5ffbe3d78d4a00f55a83f1d3cc4ccf","unresolved":false,"context_lines":[{"line_number":290,"context_line":"    GET /v1/nodes/NNNN/states/raid"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"If the driver doesn\u0027t support RAID configuration, then both API calls will"},{"line_number":293,"context_line":"return HTTP 404 (Not Found). Otherwise the API will return HTTP 200 (OK)."},{"line_number":294,"context_line":""},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"Client (CLI) impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_d0c48193","line":293,"updated":"2015-06-22 14:28:21.000000000","message":"This will allow the same information (target_raid_config) to get GET\u0027d from 3 different endpoints?\n\n 1) GET /v1/nodes/NNNN\n 2) GET /v1/nodes/NNNN/states\n 3) GET /v1/nodes/NNNN/states/raid\n\nI would suggest not allowing 3) because it\u0027s not allowed for neither for power or provision.","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"b773c33bf98904951e5d968a27c8d019addc5073","unresolved":false,"context_lines":[{"line_number":290,"context_line":"    GET /v1/nodes/NNNN/states/raid"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"If the driver doesn\u0027t support RAID configuration, then both API calls will"},{"line_number":293,"context_line":"return HTTP 404 (Not Found). Otherwise the API will return HTTP 200 (OK)."},{"line_number":294,"context_line":""},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"Client (CLI) impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa32b979_ae45806f","line":293,"in_reply_to":"fa32b979_d0c48193","updated":"2015-06-24 08:47:19.000000000","message":"Done. As discussed in the IRC, removing GET /v1/nodes/NNNN/states/raid to keep consistency with power and provision.  Console is different because it returns different information in GET /v1/nodes/NNNN/console and GET /v1/nodes/NNNN/states/console.  We can do it in a much better API in v2 probably.","commit_id":"b52493259d80387a48b9e71a96621b60b51604ec"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3f008e512cb9653c9ec3f0b486b47c7db6db8356","unresolved":false,"context_lines":[{"line_number":182,"context_line":"  ``node.target_raid_config`` field. Operator can use the REST API (or CLI)"},{"line_number":183,"context_line":"  to put a new value here at any time, which is compared to"},{"line_number":184,"context_line":"  ``node.raid_config`` during zapping and cleaning, and driver may apply"},{"line_number":185,"context_line":"  changes only in those stages. Refer REST API Impact section for more details."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"* New driver interface called ``RAIDInterface`` will be provided for RAID"},{"line_number":188,"context_line":"  configuration for drivers. For more details, refer to the Driver API impact"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa32b979_3619be88","line":185,"updated":"2015-06-24 12:43:31.000000000","message":"nit: Refer to `REST API Impact`_","commit_id":"bb191355b5407a0f27b6d72b23b7caa1bed3a619"}]}