)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"5574b86ccf7087865173761793062b522171f866","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Follow-up patch for the Redfish spec"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains small fixes to the original Redfish spec, including:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"* Remove virtual media deployment from the first version of the driver"},{"line_number":12,"context_line":"* Remove the mention to virtualbmc for creating an virtual environment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a30ddce_8864aa49","line":9,"updated":"2017-03-08 23:27:27.000000000","message":"I would love to have commit id of the Redfish spec here.","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"32134ffae56f39202508c1d79caf5c3b0f098d83","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Follow-up patch for the Redfish spec"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains small fixes to the original Redfish spec, including:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"* Remove virtual media deployment from the first version of the driver"},{"line_number":12,"context_line":"* Remove the mention to virtualbmc for creating an virtual environment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a30ddce_1a00ee00","line":9,"in_reply_to":"9a30ddce_8864aa49","updated":"2017-03-09 09:59:26.000000000","message":"Good point, will do","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"cb4753cbca4ee0ba036c99ad4b410659b01e921c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch contains small fixes to the original Redfish spec [0],"},{"line_number":10,"context_line":"including:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"* Remove virtual media deployment from the first version of the driver"},{"line_number":13,"context_line":"* Remove the mention to virtualbmc for creating an virtual environment"},{"line_number":14,"context_line":"  with DevStack"},{"line_number":15,"context_line":"* Change the wording in the work items to be more direct"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a30ddce_d98e2dcf","line":12,"range":{"start_line":12,"start_character":2,"end_line":12,"end_character":70},"updated":"2017-03-10 11:52:31.000000000","message":"What is the reason for removing virtual media based deployment? Is it more from scoping for Pike release?","commit_id":"db51370f2556abe787840262ab368f43a6065131"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9b7f1a22b55e9d26efaf343e4e2aa8e58c066f7b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch contains small fixes to the original Redfish spec [0],"},{"line_number":10,"context_line":"including:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"* Remove virtual media deployment from the first version of the driver"},{"line_number":13,"context_line":"* Remove the mention to virtualbmc for creating an virtual environment"},{"line_number":14,"context_line":"  with DevStack"},{"line_number":15,"context_line":"* Change the wording in the work items to be more direct"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a30ddce_64c6759b","line":12,"updated":"2017-03-10 13:46:36.000000000","message":"and I\u0027d say that adding virtual media later will only need an RFE, not a spec.","commit_id":"db51370f2556abe787840262ab368f43a6065131"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"63baf3770499a95e0665a287733129b2a5c62fe3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch contains small fixes to the original Redfish spec [0],"},{"line_number":10,"context_line":"including:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"* Remove virtual media deployment from the first version of the driver"},{"line_number":13,"context_line":"* Remove the mention to virtualbmc for creating an virtual environment"},{"line_number":14,"context_line":"  with DevStack"},{"line_number":15,"context_line":"* Change the wording in the work items to be more direct"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a30ddce_3458308a","line":12,"range":{"start_line":12,"start_character":2,"end_line":12,"end_character":70},"in_reply_to":"9a30ddce_d98e2dcf","updated":"2017-03-10 12:16:38.000000000","message":"Hi,\n\nYes, totally about scope. We want to get a basic driver working first and from there we can start adding new features. I think that virtual media, inspection and other features that are already supported by other drivers in Ironic can come later as RFE\u0027s (so that we don\u0027t need to re-write a spec for it)","commit_id":"db51370f2556abe787840262ab368f43a6065131"}],"specs/approved/ironic-redfish.rst":[{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"5574b86ccf7087865173761793062b522171f866","unresolved":false,"context_lines":[{"line_number":138,"context_line":"----------"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``redfish`` hardware type, power and management interfaces."},{"line_number":141,"context_line":"* Writing unit-tests for the new code."},{"line_number":142,"context_line":"* Modify the Ironic DevStack module to setup a virtual environment that"},{"line_number":143,"context_line":"  is able to test nodes using the new Redfish driver."},{"line_number":144,"context_line":"* Writing documentation."}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_884e6a94","line":141,"updated":"2017-03-08 23:27:27.000000000","message":"in don\u0027t like the inconsistency in the tense. Maybe:\ns/writing/write/\n(same in the last bullet)","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6bfa22a053e36aa45e5fe2df4099105d11e97072","unresolved":false,"context_lines":[{"line_number":138,"context_line":"----------"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``redfish`` hardware type, power and management interfaces."},{"line_number":141,"context_line":"* Writing unit-tests for the new code."},{"line_number":142,"context_line":"* Modify the Ironic DevStack module to setup a virtual environment that"},{"line_number":143,"context_line":"  is able to test nodes using the new Redfish driver."},{"line_number":144,"context_line":"* Writing documentation."}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_df9488e3","line":141,"in_reply_to":"9a30ddce_884e6a94","updated":"2017-03-09 09:59:09.000000000","message":"++ thanks, I\u0027m going to fix it","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"5574b86ccf7087865173761793062b522171f866","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``redfish`` hardware type, power and management interfaces."},{"line_number":141,"context_line":"* Writing unit-tests for the new code."},{"line_number":142,"context_line":"* Modify the Ironic DevStack module to setup a virtual environment that"},{"line_number":143,"context_line":"  is able to test nodes using the new Redfish driver."},{"line_number":144,"context_line":"* Writing documentation."},{"line_number":145,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_a8b7ae84","line":142,"updated":"2017-03-08 23:27:27.000000000","message":"shouldn\u0027t Ironic DevStack be lowercase?","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6bfa22a053e36aa45e5fe2df4099105d11e97072","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``redfish`` hardware type, power and management interfaces."},{"line_number":141,"context_line":"* Writing unit-tests for the new code."},{"line_number":142,"context_line":"* Modify the Ironic DevStack module to setup a virtual environment that"},{"line_number":143,"context_line":"  is able to test nodes using the new Redfish driver."},{"line_number":144,"context_line":"* Writing documentation."},{"line_number":145,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_bf5a744e","line":142,"in_reply_to":"9a30ddce_a8b7ae84","updated":"2017-03-09 09:59:09.000000000","message":"Yeah maybe, I always thought DevStack was the right way to write it, just like OpenStack.\n\nBut idk, the DevStack docs [0] itself doesn\u0027t seem very consistent. So yeah, I don\u0027t know heh\n\nShould I change it ? I mean, does it matter really ?\n\n[0] https://docs.openstack.org/developer/devstack/","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"5574b86ccf7087865173761793062b522171f866","unresolved":false,"context_lines":[{"line_number":152,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":153,"context_line":"Unit-tests will be implemented for Redfish support."},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"DevStack will be update to setup the nodes with the redfish driver and"},{"line_number":156,"context_line":"the libvirt mockup that is shipped with Sushy allowing it to be tests"},{"line_number":157,"context_line":"in gate against virtual machines."},{"line_number":158,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_a8a08e44","line":155,"updated":"2017-03-08 23:27:27.000000000","message":"s/update/updated/\nshouldn\u0027t devstack be lowercase?","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6bfa22a053e36aa45e5fe2df4099105d11e97072","unresolved":false,"context_lines":[{"line_number":152,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":153,"context_line":"Unit-tests will be implemented for Redfish support."},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"DevStack will be update to setup the nodes with the redfish driver and"},{"line_number":156,"context_line":"the libvirt mockup that is shipped with Sushy allowing it to be tests"},{"line_number":157,"context_line":"in gate against virtual machines."},{"line_number":158,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a30ddce_1f19c088","line":155,"in_reply_to":"9a30ddce_a8a08e44","updated":"2017-03-09 09:59:09.000000000","message":"++ updated\n\nre: DevStack, idk see comment @ L142","commit_id":"cc5c33f3013d87b01a347305b4c18b94d2c8cdbb"}]}