)]}'
{"specs/juno/ilo-ipa-deploy-driver.rst":[{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"79e4c474fe12155d6ec1b4d74197ba5a7b9a07a5","unresolved":false,"context_lines":[{"line_number":16,"context_line":"conductor node.  But IPA has the following issues:"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":19,"context_line":"  it unreliability and security issues."},{"line_number":20,"context_line":"- Deployers require an extra tftp service to be running on the conductor node."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_347d426e","line":19,"updated":"2014-07-31 13:57:17.000000000","message":"I would like to see a brief summary of these issues.","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"0f937b0559c6ea08094196efd866640e13c25765","unresolved":false,"context_lines":[{"line_number":16,"context_line":"conductor node.  But IPA has the following issues:"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":19,"context_line":"  it unreliability and security issues."},{"line_number":20,"context_line":"- Deployers require an extra tftp service to be running on the conductor node."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_abd00c2c","line":19,"in_reply_to":"baada198_347d426e","updated":"2014-07-31 17:06:40.000000000","message":"Okay, will list some of them, please have a look at them.","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"79e4c474fe12155d6ec1b4d74197ba5a7b9a07a5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Alternatives"},{"line_number":38,"context_line":"------------"},{"line_number":39,"context_line":"The proliant machines can continue to work booting the agent ramdisk with PXE,"},{"line_number":40,"context_line":"but virtual media customers can take advantage of it."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Data model impact"},{"line_number":43,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_549396a3","line":40,"updated":"2014-07-31 13:57:17.000000000","message":"I think \"but virtual media customers can take advantage of it\" can be removed here.","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"0f937b0559c6ea08094196efd866640e13c25765","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Alternatives"},{"line_number":38,"context_line":"------------"},{"line_number":39,"context_line":"The proliant machines can continue to work booting the agent ramdisk with PXE,"},{"line_number":40,"context_line":"but virtual media customers can take advantage of it."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Data model impact"},{"line_number":43,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_f56bc1b6","line":40,"in_reply_to":"baada198_549396a3","updated":"2014-07-31 17:06:40.000000000","message":"Done","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"79e4c474fe12155d6ec1b4d74197ba5a7b9a07a5","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Data model impact"},{"line_number":43,"context_line":"-----------------"},{"line_number":44,"context_line":"The new deploy driver will use two new parameters:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- driver_info[\u0027deploy_iso\u0027] - This will be used to boot up the node before the"},{"line_number":47,"context_line":"  deploy."}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_748eda7b","line":44,"updated":"2014-07-31 13:57:17.000000000","message":"One new parameter :) I also think you might want to explicitly say \"use one new parameter in `Node.driver_info`\".","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"0f937b0559c6ea08094196efd866640e13c25765","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Data model impact"},{"line_number":43,"context_line":"-----------------"},{"line_number":44,"context_line":"The new deploy driver will use two new parameters:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- driver_info[\u0027deploy_iso\u0027] - This will be used to boot up the node before the"},{"line_number":47,"context_line":"  deploy."}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_ebda044a","line":44,"in_reply_to":"baada198_748eda7b","updated":"2014-07-31 17:06:40.000000000","message":"Done","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"79e4c474fe12155d6ec1b4d74197ba5a7b9a07a5","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Other deployer impact"},{"line_number":78,"context_line":"---------------------"},{"line_number":79,"context_line":"This method of deploy doesn\u0027t require an extra service (like tftp service"},{"line_number":80,"context_line":"incase of pxe driver) to be running on the conductor node."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Developer impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_948d2e7f","line":79,"updated":"2014-07-31 13:57:17.000000000","message":"Nit: maybe s/doesn\u0027t require/no longer requires/","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"0f937b0559c6ea08094196efd866640e13c25765","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Other deployer impact"},{"line_number":78,"context_line":"---------------------"},{"line_number":79,"context_line":"This method of deploy doesn\u0027t require an extra service (like tftp service"},{"line_number":80,"context_line":"incase of pxe driver) to be running on the conductor node."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Developer impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_755fb10f","line":79,"in_reply_to":"baada198_948d2e7f","updated":"2014-07-31 17:06:40.000000000","message":"Done","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"79e4c474fe12155d6ec1b4d74197ba5a7b9a07a5","unresolved":false,"context_lines":[{"line_number":105,"context_line":"Unit tests will be added for all the code."},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"Tempest tests need to be added for carrying out deploy, but will be decided"},{"line_number":108,"context_line":"later."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Documentation Impact"},{"line_number":111,"context_line":"\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":1,"id":"baada198_b488726c","line":108,"updated":"2014-07-31 13:57:17.000000000","message":"Is it useful to add tempest tests that exercise this driver, if it can\u0027t be run in OpenStack CI? I think I\u0027d prefer to see a commitment to third-party CI for this driver, but I think it\u0027s reasonable to submit this spec without Tempest or third-party CI.","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"0f937b0559c6ea08094196efd866640e13c25765","unresolved":false,"context_lines":[{"line_number":105,"context_line":"Unit tests will be added for all the code."},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"Tempest tests need to be added for carrying out deploy, but will be decided"},{"line_number":108,"context_line":"later."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Documentation Impact"},{"line_number":111,"context_line":"\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":1,"id":"baada198_55a70d09","line":108,"in_reply_to":"baada198_b488726c","updated":"2014-07-31 17:06:40.000000000","message":"We would like to add tempest tests, but we don\u0027t have hardware right now.  We will surely consider it later.","commit_id":"039f6f858ea5da95e87c9d3866c7ab2ed04322ab"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"1821536605b7bc27298103aa13fdddb43b8bdaa8","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    UDP."},{"line_number":22,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":23,"context_line":"    requests might be required in routers to enable PXE working for the whole"},{"line_number":24,"context_line":"    network."},{"line_number":25,"context_line":"- Deployers require an extra tftp service to be running on the conductor node."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_69527cdf","line":24,"updated":"2014-07-31 18:03:44.000000000","message":"Can you also mention something about securely transmitting secrets OOB? :)","commit_id":"aeec2db797a15536cf110a42bff14eba5ec11517"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"2fe20a60cd4216130c8ce86cfd994bb2823fcbe6","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    UDP."},{"line_number":22,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":23,"context_line":"    requests might be required in routers to enable PXE working for the whole"},{"line_number":24,"context_line":"    network."},{"line_number":25,"context_line":"- Deployers require an extra tftp service to be running on the conductor node."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_89fb50bd","line":24,"in_reply_to":"baada198_69527cdf","updated":"2014-07-31 18:11:21.000000000","message":"Done","commit_id":"aeec2db797a15536cf110a42bff14eba5ec11517"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"4ae15d5668ba54e79c83d9526b1eed8ddfa0f089","unresolved":false,"context_lines":[{"line_number":18,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":19,"context_line":"  the following issues:"},{"line_number":20,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"},{"line_number":21,"context_line":"    UDP."},{"line_number":22,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":23,"context_line":"    requests might be required in routers to enable PXE working for the whole"},{"line_number":24,"context_line":"    network."}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_c8cb7f1b","line":21,"updated":"2014-07-31 20:21:09.000000000","message":"Something on or around this line is breaking the .rst parsing in the tests.","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"39837f195ee211c5d3949a0399c3683b3a7c8c9f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":19,"context_line":"  the following issues:"},{"line_number":20,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"},{"line_number":21,"context_line":"    UDP."},{"line_number":22,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":23,"context_line":"    requests might be required in routers to enable PXE working for the whole"},{"line_number":24,"context_line":"    network."}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_be8d962c","line":21,"in_reply_to":"baada198_c8cb7f1b","updated":"2014-08-01 18:07:12.000000000","message":"Done","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"48a3947cb07c57ba2910e441e0bd7c0cfc427d62","unresolved":false,"context_lines":[{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":"The below review introduced a new mechanism for booting proliant machines with"},{"line_number":32,"context_line":"virtual media."},{"line_number":33,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":36,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_dabfa8c8","line":33,"updated":"2014-08-01 12:43:43.000000000","message":"please make your spec depend on it","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"39837f195ee211c5d3949a0399c3683b3a7c8c9f","unresolved":false,"context_lines":[{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":"The below review introduced a new mechanism for booting proliant machines with"},{"line_number":32,"context_line":"virtual media."},{"line_number":33,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":36,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_4b1ae26b","line":33,"in_reply_to":"baada198_dabfa8c8","updated":"2014-08-01 18:07:12.000000000","message":"Done","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"4ae15d5668ba54e79c83d9526b1eed8ddfa0f089","unresolved":false,"context_lines":[{"line_number":33,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":36,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"},{"line_number":37,"context_line":"introduced in the above review can be used to boot up a baremetal node with"},{"line_number":38,"context_line":"the deploy ISO image.  A new class ``IloVirtualMediaAgentDeploy`` can be added"},{"line_number":39,"context_line":"which will setup the machine to be booted with virtual media instead of PXE."}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_28fddb3b","line":36,"updated":"2014-07-31 20:21:09.000000000","message":"I can help guide when you get ready to implement this as an ISO, if you want to use the CoreOS style agent images. Just ping me in IRC. I think it should be fairly trivial and reuse a lot of what we do for the existing ipa-coreos image.","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"39837f195ee211c5d3949a0399c3683b3a7c8c9f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":36,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"},{"line_number":37,"context_line":"introduced in the above review can be used to boot up a baremetal node with"},{"line_number":38,"context_line":"the deploy ISO image.  A new class ``IloVirtualMediaAgentDeploy`` can be added"},{"line_number":39,"context_line":"which will setup the machine to be booted with virtual media instead of PXE."}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_fe938e4b","line":36,"in_reply_to":"baada198_28fddb3b","updated":"2014-08-01 18:07:12.000000000","message":"Done. Thanks.","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"4ae15d5668ba54e79c83d9526b1eed8ddfa0f089","unresolved":false,"context_lines":[{"line_number":111,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":112,"context_line":"Unit tests will be added for all the code."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Tempest tests need to be added for carrying out deploy, but will be decided"},{"line_number":115,"context_line":"later."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_88ba676f","line":114,"updated":"2014-07-31 20:21:09.000000000","message":"This may be difficult/impossible to do first-party as it needs specific hardware, but I\u0027d love seeing third-party CI for this.\n\nAlso we can do a test ISO build on IPA commits like we do currently for the ipa-coreos image.","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"39837f195ee211c5d3949a0399c3683b3a7c8c9f","unresolved":false,"context_lines":[{"line_number":111,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":112,"context_line":"Unit tests will be added for all the code."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Tempest tests need to be added for carrying out deploy, but will be decided"},{"line_number":115,"context_line":"later."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_cb44324a","line":114,"in_reply_to":"baada198_88ba676f","updated":"2014-08-01 18:07:12.000000000","message":"Okay, will understand more from you more regarding building and testing of ISO of coreos images.","commit_id":"0bc2e689fd601febbf141c622c201924ec9afb1a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"fa8bb3de4bd5802dec87d3904ebe55ffdf5c3963","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"iLO IPA Deploy Driver"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":"This spec talks about creating a new driver which will use iLO virtual media"},{"line_number":11,"context_line":"as the boot mechanism and IPA as the deploy engine."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"baada198_e077ba3a","line":9,"updated":"2014-08-06 17:24:06.000000000","message":"This needs a URL to the blueprint. Please ensure the blueprint and spec names are the same.","commit_id":"d74ac125cb327dcbed976be6f01bea97b655fe29"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"24c442fba7ee02f199b1baa5643666f43679a5c4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"iLO IPA Deploy Driver"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":"This spec talks about creating a new driver which will use iLO virtual media"},{"line_number":11,"context_line":"as the boot mechanism and IPA as the deploy engine."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"baada198_e33504dd","line":9,"in_reply_to":"baada198_e077ba3a","updated":"2014-08-06 17:41:16.000000000","message":"Done","commit_id":"d74ac125cb327dcbed976be6f01bea97b655fe29"}],"specs/juno/ilo-virtualmedia-ipa.rst":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":19,"context_line":"IPA project provides a more powerful ramdisk for doing deploy from the"},{"line_number":20,"context_line":"conductor node.  But IPA has the following issues:"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":23,"context_line":"  the following issues:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_ae8bba15","line":22,"updated":"2014-08-08 06:33:55.000000000","message":"s/customer\u0027s/customers/","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":19,"context_line":"IPA project provides a more powerful ramdisk for doing deploy from the"},{"line_number":20,"context_line":"conductor node.  But IPA has the following issues:"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":23,"context_line":"  the following issues:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_a949f467","line":22,"in_reply_to":"baada198_ae8bba15","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":22,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":23,"context_line":"  the following issues:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"},{"line_number":26,"context_line":"    UDP."},{"line_number":27,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":28,"context_line":"    requests might be required in routers to enable PXE working for the whole"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_8ebb56bd","line":25,"updated":"2014-08-08 06:33:55.000000000","message":"s/in/is/","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":22,"context_line":"- Some customer\u0027s don\u0027t prefer PXE protocol in their environment because of"},{"line_number":23,"context_line":"  the following issues:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  + PXE uses TFTP to transfer the files which in unreliable because it uses"},{"line_number":26,"context_line":"    UDP."},{"line_number":27,"context_line":"  + PXE is not suited for some network topologies where the relaying of PXE"},{"line_number":28,"context_line":"    requests might be required in routers to enable PXE working for the whole"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_c94c6878","line":25,"in_reply_to":"baada198_8ebb56bd","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":"The below review introduced a new mechanism for booting proliant machines with"},{"line_number":37,"context_line":"virtual media."},{"line_number":38,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":41,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_8ed276fb","line":38,"updated":"2014-08-08 06:33:55.000000000","message":"Please use link to the approved spec: http://specs.openstack.org/openstack/ironic-specs/specs/juno/ironic-ilo-virtualmedia-driver.html","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":"The below review introduced a new mechanism for booting proliant machines with"},{"line_number":37,"context_line":"virtual media."},{"line_number":38,"context_line":"https://review.openstack.org/#/c/97744"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"IPA build mechanism can be changed to build up an ISO image comprising of the"},{"line_number":41,"context_line":"IPA kernel and IPA agent ramdisk.  The methods ``setup_virtual_media_boot``"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_e953ec13","line":38,"in_reply_to":"baada198_8ed276fb","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":47,"context_line":"deploy and complete it."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"This change will also enable the admin token to be handed off securely to the"},{"line_number":50,"context_line":"baremetal node through OOB channel over virtual media."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_8e17d69c","line":50,"updated":"2014-08-08 06:33:55.000000000","message":"Please rephrase your \"Proposed Changes\" to clearly distinguish between \"will do\" (what\u0027s in the scope of the spec) and \"can do\" aka future possibilities. E.g. reading this section I thought that you\u0027re proposing also a change to IPA build system, while the only work item is adding IloVirtualMediaAgentDeploy.","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":47,"context_line":"deploy and complete it."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"This change will also enable the admin token to be handed off securely to the"},{"line_number":50,"context_line":"baremetal node through OOB channel over virtual media."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_6947dc4c","line":50,"in_reply_to":"baada198_8e17d69c","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Data model impact"},{"line_number":57,"context_line":"-----------------"},{"line_number":58,"context_line":"One new parameter \u0027deploy_iso\u0027 will be used in driver_info to boot up the"},{"line_number":59,"context_line":"node for the deploy."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"REST API impact"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_eecdb231","line":58,"updated":"2014-08-08 06:33:55.000000000","message":"Is it Glance image ID? Please state for the sake of completeness.","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Data model impact"},{"line_number":57,"context_line":"-----------------"},{"line_number":58,"context_line":"One new parameter \u0027deploy_iso\u0027 will be used in driver_info to boot up the"},{"line_number":59,"context_line":"node for the deploy."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"REST API impact"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_89427039","line":58,"in_reply_to":"baada198_eecdb231","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":89,"context_line":"Other deployer impact"},{"line_number":90,"context_line":"---------------------"},{"line_number":91,"context_line":"This method of deploy no longer requires an extra service (like tftp service"},{"line_number":92,"context_line":"incase of pxe driver) to be running on the conductor node."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Developer impact"},{"line_number":95,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_2e508a5e","line":92,"updated":"2014-08-08 06:33:55.000000000","message":"s/incase/in case/","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":89,"context_line":"Other deployer impact"},{"line_number":90,"context_line":"---------------------"},{"line_number":91,"context_line":"This method of deploy no longer requires an extra service (like tftp service"},{"line_number":92,"context_line":"incase of pxe driver) to be running on the conductor node."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Developer impact"},{"line_number":95,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_a97014a8","line":92,"in_reply_to":"baada198_2e508a5e","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24dbab85d9041870f7f8c1e5cc89e40e78d881d8","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Dependencies"},{"line_number":111,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":112,"context_line":"* iLO Virtual Media iSCSI Deploy Driver: https://review.openstack.org/#/c/97744"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_ee56925d","line":112,"updated":"2014-08-08 06:33:55.000000000","message":"That\u0027s no longer a dependency, because it\u0027s merged","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"005113d68ce6358c612e2f9ab7eca9803a8505af","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Dependencies"},{"line_number":111,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":112,"context_line":"* iLO Virtual Media iSCSI Deploy Driver: https://review.openstack.org/#/c/97744"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_c97388b1","line":112,"in_reply_to":"baada198_ee56925d","updated":"2014-08-08 06:47:44.000000000","message":"Done","commit_id":"47c4dbe66a3488abf5d53fd66d832b0d923cdf72"}]}