)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14162,"name":"Daisuke Fujita","email":"fuzita.daisuke@jp.fujitsu.com","username":"daisukefujita"},"change_message_id":"8475a95615e8e34583434d3105ecf5e100abc4bf","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit proposes changes to Nova Ironic virt driver"},{"line_number":10,"context_line":"to support booting from Cinder volume (in tandem with"},{"line_number":11,"context_line":"other changes proposed in Ironic)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa80f949_d7ebbd11","line":11,"updated":"2015-11-06 12:20:17.000000000","message":"Could you add a link of blueprint?[1]\n\n[1] https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"2c34ff407156eab0d367c0c5dec6baf5a76f10a6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit proposes changes to Nova Ironic virt driver"},{"line_number":10,"context_line":"to support booting from Cinder volume (in tandem with"},{"line_number":11,"context_line":"other changes proposed in Ironic)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da85f559_137ec7e4","line":11,"in_reply_to":"fa80f949_d7ebbd11","updated":"2015-11-07 11:28:03.000000000","message":"Done","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":14162,"name":"Daisuke Fujita","email":"fuzita.daisuke@jp.fujitsu.com","username":"daisukefujita"},"change_message_id":"cf2ba778f3554b4daf387fdf902c2b8ce7fc1137","unresolved":false,"context_lines":[{"line_number":10,"context_line":"to support booting from Cinder volume (in tandem with"},{"line_number":11,"context_line":"other changes proposed in Ironic)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a8ffd7b_add6d3ee","line":13,"updated":"2015-11-27 08:48:52.000000000","message":"@Ramakrishnan,\n\nCould you add a following line?\n\nImplements: blueprint ironic-boot-from-volume","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"bcd96d6832888ecf18d4790e7f5c34cb249f4a6d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"to support booting from Cinder volume (in tandem with"},{"line_number":11,"context_line":"other changes proposed in Ironic)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a8ffd7b_10ce8783","line":13,"in_reply_to":"9a8ffd7b_add6d3ee","updated":"2015-12-01 12:19:43.000000000","message":"Done","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"eb8dbe0d33f4153c13c1b796bdc53826e2f1a452","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Co-Authored-By: Satoru Moriya \u003csatoru.moriya.br@htiachi.com\u003e"},{"line_number":14,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"},{"line_number":15,"context_line":"Implements: blueprint ironic-boot-from-volume"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"7a740942_e487b3d5","line":15,"updated":"2015-12-03 18:02:14.000000000","message":"we need Depends-on: Ibf50be735905f868e0d523ab16353a6b5cc300af adding here.\n\nIts hard to review this without ironic agreement.\n\nIf its just adding support for the ironic spec, we can probably do that in a spec-less blueprint actually.","commit_id":"673b7dda39e3c067ba0b41400a9d27078ee02f25"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"e2698a3f4646a1e7ef2faea7c3b4d80f89d0f45e","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Co-Authored-By: Satoru Moriya \u003csatoru.moriya.br@htiachi.com\u003e"},{"line_number":14,"context_line":"Change-Id: I11d37595d11b536bbe0f986a3a814bed2b3a2650"},{"line_number":15,"context_line":"Implements: blueprint ironic-boot-from-volume"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"7a740942_07de686d","line":15,"in_reply_to":"7a740942_e487b3d5","updated":"2015-12-04 05:27:29.000000000","message":"Done","commit_id":"673b7dda39e3c067ba0b41400a9d27078ee02f25"}],"specs/mitaka/approved/ironic-boot-from-volume.rst":[{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_4e108559","line":20,"range":{"start_line":20,"start_character":38,"end_line":20,"end_character":40},"updated":"2015-09-17 12:45:39.000000000","message":"nit: s/an/a/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_7af2589c","line":20,"in_reply_to":"ba15a1d1_4e108559","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_fae748aa","line":21,"updated":"2015-10-06 04:58:23.000000000","message":"nit: s/Ironic doesn\u0027t support it too/Ironic also doesn\u0027t support it/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_2e0c3140","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":7},"updated":"2015-09-17 12:45:39.000000000","message":"nit: s/to boot/boots/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_55796d92","line":21,"in_reply_to":"7a2fa921_fae748aa","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_9aef34b0","line":21,"in_reply_to":"ba15a1d1_2e0c3140","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1aed648b","line":23,"updated":"2015-10-06 04:58:23.000000000","message":"s/from remote/from a remote/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Currently, there is no way to request an bare metal compute resource from Nova"},{"line_number":21,"context_line":"to boot from a Cinder volume.  Ironic doesn\u0027t support it too.  But with"},{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_3574f19a","line":23,"in_reply_to":"7a2fa921_1aed648b","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"cbff4456893354f7859c1d2171b311496e49db9d","unresolved":false,"context_lines":[{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"da20952f_d4c63a67","line":25,"updated":"2015-09-05 13:58:00.000000000","message":"s/requesed/requested/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_ba02303d","line":25,"updated":"2015-10-06 04:58:23.000000000","message":"s/requessed/requesting it/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_9548655d","line":25,"in_reply_to":"7a2fa921_ba02303d","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"some changes proposed in Ironic ([1] and [2]), it will enable Ironic to boot"},{"line_number":23,"context_line":"bare metal servers from remote volume.  Ironic virt driver needs to pass the"},{"line_number":24,"context_line":"remote volume connection information down to Ironic when spawning a new"},{"line_number":25,"context_line":"bare metal instance requesed to boot from a Cinder volume."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_3a2b80ed","line":25,"in_reply_to":"da20952f_d4c63a67","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"A tenant can request a bare metal server that boots from a Cinder volume."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Project Priority"},{"line_number":33,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_ba89d0cc","line":30,"updated":"2015-10-06 04:58:23.000000000","message":"Maybe something could be mentioned about better security in multi-tenant environments? Don\u0027t have to worry about someone modifying hard drive firmware of the boot device.\n\nNot sure if that is one of the reasons or not for this?","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"A tenant can request a bare metal server that boots from a Cinder volume."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Project Priority"},{"line_number":33,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_f57bd979","line":30,"in_reply_to":"7a2fa921_ba89d0cc","updated":"2015-10-06 05:08:28.000000000","message":"Makes sense. It\u0027s secure and reliable.  But not sure if worth capturing in use cases.  I can capture in problem description though.","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* Whenever an instance that boots from Cinder volume is requested in Nova, it"},{"line_number":41,"context_line":"  queries the virt driver to provide the volume connectors for the hypervisor."},{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_7a9338fb","line":40,"updated":"2015-10-06 04:58:23.000000000","message":"s/from Cinder/from a Cinder/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* Whenever an instance that boots from Cinder volume is requested in Nova, it"},{"line_number":41,"context_line":"  queries the virt driver to provide the volume connectors for the hypervisor."},{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_d53eddb7","line":40,"in_reply_to":"7a2fa921_7a9338fb","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":41,"context_line":"  queries the virt driver to provide the volume connectors for the hypervisor."},{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"},{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_169d9bbb","line":44,"updated":"2015-09-17 12:45:39.000000000","message":"nit: missing a closing \u003e","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"  queries the virt driver to provide the volume connectors for the hypervisor."},{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"},{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_5a30bc56","line":44,"in_reply_to":"ba15a1d1_169d9bbb","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"},{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"},{"line_number":48,"context_line":"  virt driver should query Neutron to get the private IP assigned to the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_fab5087e","line":45,"updated":"2015-10-06 04:58:23.000000000","message":"s/Ironic API/The Ironic API/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":42,"context_line":"  Virt drivers should implement the method ``get_volume_connector`` in the Nova"},{"line_number":43,"context_line":"  driver API.  Ironic virt driver should implement this method which hits the"},{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"},{"line_number":48,"context_line":"  virt driver should query Neutron to get the private IP assigned to the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_f56d19cd","line":45,"in_reply_to":"7a2fa921_fab5087e","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"},{"line_number":48,"context_line":"  virt driver should query Neutron to get the private IP assigned to the"},{"line_number":49,"context_line":"  MAC address ``mac`` (which is the MAC address of the iSCSI initiator). If"},{"line_number":50,"context_line":"  ``ip`` is not returned, then it means that storage network is managed by"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_b6ae2fe4","line":47,"range":{"start_line":47,"start_character":51,"end_line":47,"end_character":58},"updated":"2015-09-17 12:45:39.000000000","message":"nit: returned","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"  Ironic endpoint ``GET /v1/volume_connectors?node\u003d\u003cnode_uuid`` in the Ironic"},{"line_number":45,"context_line":"  API. Ironic API will provide the volume connectors for the bare metal."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returns from the API, Ironic"},{"line_number":48,"context_line":"  virt driver should query Neutron to get the private IP assigned to the"},{"line_number":49,"context_line":"  MAC address ``mac`` (which is the MAC address of the iSCSI initiator). If"},{"line_number":50,"context_line":"  ``ip`` is not returned, then it means that storage network is managed by"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_fa34284b","line":47,"in_reply_to":"ba15a1d1_b6ae2fe4","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  Neutron."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_3a63801c","line":54,"updated":"2015-10-06 04:58:23.000000000","message":"s/passed to/passed to the/\n\ns/Nova Ironic/The Nova Ironic/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  Neutron."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1573f5b2","line":54,"in_reply_to":"7a2fa921_3a63801c","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1a7244e6","line":55,"updated":"2015-10-06 04:58:23.000000000","message":"s/Ironic node\u0027s/the Ironic node\u0027s/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_b56801bc","line":55,"in_reply_to":"7a2fa921_1a7244e6","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"},{"line_number":59,"context_line":"  to Ironic node\u0027s instance_info."}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_da74ccd0","line":56,"updated":"2015-10-06 04:58:23.000000000","message":"s/Nova Ironic/the Nova Ironic/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":53,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":54,"context_line":"  device passed to ``spawn()`` method to Ironic.  Nova Ironic virt driver"},{"line_number":55,"context_line":"  should add an attribute in Ironic node\u0027s instance_info for each attribute in"},{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"},{"line_number":59,"context_line":"  to Ironic node\u0027s instance_info."}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_756269da","line":56,"in_reply_to":"7a2fa921_da74ccd0","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"},{"line_number":59,"context_line":"  to Ironic node\u0027s instance_info."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":".. note::"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_9a7e54ad","line":59,"updated":"2015-10-06 04:58:23.000000000","message":"s/to Ironic/to the Ironic/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  ``connection_info``.  For example, for an iSCSI Cinder volume, Nova Ironic"},{"line_number":57,"context_line":"  virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":58,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"},{"line_number":59,"context_line":"  to Ironic node\u0027s instance_info."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":".. note::"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_955f050e","line":59,"in_reply_to":"7a2fa921_9a7e54ad","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":61,"context_line":".. note::"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"  The scheduler changes to pick up a suitable bare metal node which can support"},{"line_number":64,"context_line":"  booting from a particular volume should be taken care in the spec which"},{"line_number":65,"context_line":"  covers the changes in scheduling mechanism for Ironic virt driver ([3] and"},{"line_number":66,"context_line":"  [4])."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_d6c72396","line":64,"range":{"start_line":64,"start_character":51,"end_line":64,"end_character":55},"updated":"2015-09-17 12:45:39.000000000","message":"nit: care of","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":61,"context_line":".. note::"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"  The scheduler changes to pick up a suitable bare metal node which can support"},{"line_number":64,"context_line":"  booting from a particular volume should be taken care in the spec which"},{"line_number":65,"context_line":"  covers the changes in scheduling mechanism for Ironic virt driver ([3] and"},{"line_number":66,"context_line":"  [4])."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1a3a4437","line":64,"in_reply_to":"ba15a1d1_d6c72396","updated":"2015-10-06 04:40:02.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"16bfcb4055422efb978b037a8a653ebc62246af9","unresolved":false,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"  The scheduler changes to pick up a suitable bare metal node which can support"},{"line_number":64,"context_line":"  booting from a particular volume should be taken care in the spec which"},{"line_number":65,"context_line":"  covers the changes in scheduling mechanism for Ironic virt driver ([3] and"},{"line_number":66,"context_line":"  [4])."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_da5b6c53","line":65,"updated":"2015-10-06 04:58:23.000000000","message":"s/for Ironic/for the Ironic/","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"f992bb31e63492623abceb7ad0109c45cb77dd53","unresolved":false,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"  The scheduler changes to pick up a suitable bare metal node which can support"},{"line_number":64,"context_line":"  booting from a particular volume should be taken care in the spec which"},{"line_number":65,"context_line":"  covers the changes in scheduling mechanism for Ironic virt driver ([3] and"},{"line_number":66,"context_line":"  [4])."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_359b91cb","line":65,"in_reply_to":"7a2fa921_da5b6c53","updated":"2015-10-06 05:08:28.000000000","message":"Done","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"5da4a9871b3c063fb751a1ef9c09e2a918afd19c","unresolved":false,"context_lines":[{"line_number":94,"context_line":"Other end user impact"},{"line_number":95,"context_line":"---------------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"End users can request a bare metal that boots from Cinder volume."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Performance Impact"},{"line_number":100,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba15a1d1_b6c38f88","line":97,"range":{"start_line":97,"start_character":0,"end_line":97,"end_character":65},"updated":"2015-09-17 12:45:39.000000000","message":"You can just put None.","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"576675a26225f4878398346423ed7f063e899b9e","unresolved":false,"context_lines":[{"line_number":94,"context_line":"Other end user impact"},{"line_number":95,"context_line":"---------------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"End users can request a bare metal that boots from Cinder volume."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Performance Impact"},{"line_number":100,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_da3ccc31","line":97,"in_reply_to":"ba15a1d1_b6c38f88","updated":"2015-10-06 04:40:02.000000000","message":"Done.","commit_id":"490fe712a6afc531e80b5d8d146b694b2f1544dc"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6cbe525f4112cbcebaeb2ab606eeb2659671b240","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  API. The Ironic API will provide the volume connectors for the bare metal."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returned from the API,"},{"line_number":54,"context_line":"  Ironic virt driver should query Neutron to get the private IP assigned to the"},{"line_number":55,"context_line":"  MAC address ``mac`` (which is the MAC address of the iSCSI initiator). If"},{"line_number":56,"context_line":"  ``ip`` is not returned, then it means that storage network is managed by"},{"line_number":57,"context_line":"  Neutron."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a2ca52d_ddf51fed","line":54,"updated":"2015-10-16 08:34:32.000000000","message":"This should be querying the instance network info cache, rather than hitting Neutron.","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"08ae5a22e451eaa1bbd450db2b924289ce626e80","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  API. The Ironic API will provide the volume connectors for the bare metal."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returned from the API,"},{"line_number":54,"context_line":"  Ironic virt driver should query Neutron to get the private IP assigned to the"},{"line_number":55,"context_line":"  MAC address ``mac`` (which is the MAC address of the iSCSI initiator). If"},{"line_number":56,"context_line":"  ``ip`` is not returned, then it means that storage network is managed by"},{"line_number":57,"context_line":"  Neutron."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_d7973dea","line":54,"in_reply_to":"5a2ca52d_ddf51fed","updated":"2015-11-06 12:25:16.000000000","message":"We have found that when an instance is booted, in the get_volume_connector method of the virt driver, the instance info cache has not yet been populated, and the network allocation may not yet have been completed. Additionally, the asychronous handle for waiting on the network allocation is not made available inside this method. There will need to be some refactoring around the compute driver in order to make this information available.","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"2c34ff407156eab0d367c0c5dec6baf5a76f10a6","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  API. The Ironic API will provide the volume connectors for the bare metal."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"  If ``ip`` doesn\u0027t exist in the volume connectors returned from the API,"},{"line_number":54,"context_line":"  Ironic virt driver should query Neutron to get the private IP assigned to the"},{"line_number":55,"context_line":"  MAC address ``mac`` (which is the MAC address of the iSCSI initiator). If"},{"line_number":56,"context_line":"  ``ip`` is not returned, then it means that storage network is managed by"},{"line_number":57,"context_line":"  Neutron."}],"source_content_type":"text/x-rst","patch_set":3,"id":"da85f559_f35f8373","line":54,"in_reply_to":"fa80f949_d7973dea","updated":"2015-11-07 11:28:03.000000000","message":"I will correct this.\n\nFrom the patch https://review.openstack.org/#/c/215385/4/nova/virt/ironic/driver.py which implements this (honestly not implemented by me :)), it seems to do the job without much refactoring.","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"20174ced623e890f15ca59ee7083d82e297df1ca","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  Neutron."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":60,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":61,"context_line":"  driver should add an attribute in Ironic node\u0027s instance_info for each"},{"line_number":62,"context_line":"  attribute in ``connection_info``.  For example, for an iSCSI Cinder volume,"},{"line_number":63,"context_line":"  the Nova Ironic virt driver should write the ``target_iqn``, ``target_lun``,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1920f248","line":60,"updated":"2015-10-13 09:09:18.000000000","message":"Do you intend to support passing more than one block device here?","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"2c34ff407156eab0d367c0c5dec6baf5a76f10a6","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  Neutron."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":60,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":61,"context_line":"  driver should add an attribute in Ironic node\u0027s instance_info for each"},{"line_number":62,"context_line":"  attribute in ``connection_info``.  For example, for an iSCSI Cinder volume,"},{"line_number":63,"context_line":"  the Nova Ironic virt driver should write the ``target_iqn``, ``target_lun``,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"da85f559_d35eff75","line":60,"in_reply_to":"5a2ca52d_fd5fbbe2","updated":"2015-11-07 11:28:03.000000000","message":"I didn\u0027t get the query for multiple block device. Did you mean multiple block devices out of which one will be boot device and others will be Cinder volumes that will just \"attached\" ? If not, I would like to make it out of scope of this spec, may be do *something* else that is required to make that happen later.  Thoughts ?","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6cbe525f4112cbcebaeb2ab606eeb2659671b240","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  Neutron."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":60,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":61,"context_line":"  driver should add an attribute in Ironic node\u0027s instance_info for each"},{"line_number":62,"context_line":"  attribute in ``connection_info``.  For example, for an iSCSI Cinder volume,"},{"line_number":63,"context_line":"  the Nova Ironic virt driver should write the ``target_iqn``, ``target_lun``,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a2ca52d_fd5fbbe2","line":60,"in_reply_to":"7a2fa921_1920f248","updated":"2015-10-16 08:34:32.000000000","message":"+1 for multiple block device attachments. At least at boot time, if not dynamic.","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6cbe525f4112cbcebaeb2ab606eeb2659671b240","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":60,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":61,"context_line":"  driver should add an attribute in Ironic node\u0027s instance_info for each"},{"line_number":62,"context_line":"  attribute in ``connection_info``.  For example, for an iSCSI Cinder volume,"},{"line_number":63,"context_line":"  the Nova Ironic virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":64,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a2ca52d_bd0ce3ec","line":61,"updated":"2015-10-16 08:34:32.000000000","message":"This isn\u0027t quite how the Ironic spec describes it - there\u0027s a top level target_info attribute containing the connection info and the boot index.","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"2c34ff407156eab0d367c0c5dec6baf5a76f10a6","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":60,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":61,"context_line":"  driver should add an attribute in Ironic node\u0027s instance_info for each"},{"line_number":62,"context_line":"  attribute in ``connection_info``.  For example, for an iSCSI Cinder volume,"},{"line_number":63,"context_line":"  the Nova Ironic virt driver should write the ``target_iqn``, ``target_lun``,"},{"line_number":64,"context_line":"  ``target_portal``, ``auth_method``, ``auth_username`` and ``auth_password``"}],"source_content_type":"text/x-rst","patch_set":3,"id":"da85f559_b3825bc7","line":61,"in_reply_to":"5a2ca52d_bd0ce3ec","updated":"2015-11-07 11:28:03.000000000","message":"Done","commit_id":"c7d4872dbac3bbc2ff5a96da6e1e10d5a25545f6"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"51528b5e449123eaf3e24278a5c5c796b707abcc","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"A bare metal hardware using a Cinder volume is more secure in a multi-tenant"},{"line_number":34,"context_line":"environment, as tenants don\u0027t get to see other\u0027s data by doing something"},{"line_number":35,"context_line":"malicious on the system\u0027s or disk\u0027s firmware. Booting from SAN also provides"},{"line_number":36,"context_line":"better data protection at lower cost including support for snapshots and"},{"line_number":37,"context_line":"replication.  A tenant can request a bare metal server that boots from a"},{"line_number":38,"context_line":"Cinder volume."}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_1fe681b4","line":35,"updated":"2015-11-25 15:09:32.000000000","message":"I\u0027m not sure this is true - the hardware is still prone to firmware-based attacks.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"A bare metal hardware using a Cinder volume is more secure in a multi-tenant"},{"line_number":34,"context_line":"environment, as tenants don\u0027t get to see other\u0027s data by doing something"},{"line_number":35,"context_line":"malicious on the system\u0027s or disk\u0027s firmware. Booting from SAN also provides"},{"line_number":36,"context_line":"better data protection at lower cost including support for snapshots and"},{"line_number":37,"context_line":"replication.  A tenant can request a bare metal server that boots from a"},{"line_number":38,"context_line":"Cinder volume."}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_8175d8fa","line":35,"in_reply_to":"9a8ffd7b_1fe681b4","updated":"2015-11-27 11:19:14.000000000","message":"Agreed. But in some cases, it may be more secure.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"51528b5e449123eaf3e24278a5c5c796b707abcc","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  is managed by Neutron.  In that case, Ironic virt driver should use the fixed"},{"line_number":57,"context_line":"  IP assigned to the MAC address specified in the ``mac`` parameter (which"},{"line_number":58,"context_line":"  is the MAC address of the iSCSI initiator).  If ``mac`` parameter also"},{"line_number":59,"context_line":"  doesn\u0027t exist, then the first fixed IP for the instance is chosen."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":62,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_1fabe11e","line":59,"updated":"2015-11-25 15:09:32.000000000","message":"I wonder if the operator should be able to optionally specify the Neutron network that cinder is available on, so in the case of multiple instance IPs, we have a better chance at guessing the right one.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  is managed by Neutron.  In that case, Ironic virt driver should use the fixed"},{"line_number":57,"context_line":"  IP assigned to the MAC address specified in the ``mac`` parameter (which"},{"line_number":58,"context_line":"  is the MAC address of the iSCSI initiator).  If ``mac`` parameter also"},{"line_number":59,"context_line":"  doesn\u0027t exist, then the first fixed IP for the instance is chosen."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":62,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_02332663","line":59,"in_reply_to":"9a8ffd7b_1fabe11e","updated":"2015-11-27 11:19:14.000000000","message":"Make sense. I\u0027ll update the ironic spec for operators to store network uuid in DB. This driver can get it from Ironic API and use it to choose Neutron network.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"6c34457c18b231851b5456f4b883a6388d32474c","unresolved":false,"context_lines":[{"line_number":61,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":62,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":63,"context_line":"  driver should add an attribute ``target_info`` in Ironic node\u0027s"},{"line_number":64,"context_line":"  ``instance_info``.  ``target_info`` will contain the ``driver_volume_type``,"},{"line_number":65,"context_line":"  ``boot_index`` and the connection information for the remote volume within"},{"line_number":66,"context_line":"  ``data`` attribute."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_bf01756c","line":64,"updated":"2015-11-25 15:03:17.000000000","message":"According to the ironic spec, we aren\u0027t using instance_info for this.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":61,"context_line":"* Nova Ironic virt driver should pass the ``connection_info`` of the block"},{"line_number":62,"context_line":"  device passed to the ``spawn()`` method to Ironic.  The Nova Ironic virt"},{"line_number":63,"context_line":"  driver should add an attribute ``target_info`` in Ironic node\u0027s"},{"line_number":64,"context_line":"  ``instance_info``.  ``target_info`` will contain the ``driver_volume_type``,"},{"line_number":65,"context_line":"  ``boot_index`` and the connection information for the remote volume within"},{"line_number":66,"context_line":"  ``data`` attribute."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_d6395fd8","line":64,"in_reply_to":"9a8ffd7b_bf01756c","updated":"2015-11-27 11:19:14.000000000","message":"Done","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"51528b5e449123eaf3e24278a5c5c796b707abcc","unresolved":false,"context_lines":[{"line_number":114,"context_line":"---------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Deployers need to prepare bare metal servers that can boot from remote volume"},{"line_number":117,"context_line":"in Ironic."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Developer impact"},{"line_number":120,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_bf3e55d8","line":117,"updated":"2015-11-25 15:09:32.000000000","message":"Can you add more info on what this entails? There\u0027s also presumably work to do to make sure cinder is available from networks that the instance lives on, no?","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":114,"context_line":"---------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Deployers need to prepare bare metal servers that can boot from remote volume"},{"line_number":117,"context_line":"in Ironic."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Developer impact"},{"line_number":120,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_d6e63f14","line":117,"in_reply_to":"9a8ffd7b_bf3e55d8","updated":"2015-11-27 11:19:14.000000000","message":"Done","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"c24d6b78ba3f601626e7a14fc403bfbc4913ade8","unresolved":false,"context_lines":[{"line_number":128,"context_line":"-----------"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Primary assignee:"},{"line_number":131,"context_line":"  Tomoki Sekiyama"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Work Items"},{"line_number":134,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"da85f559_5972e012","line":131,"updated":"2015-11-16 00:33:34.000000000","message":"Please add \"Satoru Moriya\".\n\nRecently, he can\u0027t get enough time for the patch and so I\u0027ll take over it.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"002ead7c4c0a0daa69f917a945302b99ba79af04","unresolved":false,"context_lines":[{"line_number":128,"context_line":"-----------"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Primary assignee:"},{"line_number":131,"context_line":"  Tomoki Sekiyama"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Work Items"},{"line_number":134,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"da85f559_c5225fa3","line":131,"in_reply_to":"da85f559_5972e012","updated":"2015-11-16 03:58:33.000000000","message":"Please feel free to post a new patch with your name in it.  You are welcome to take control of this spec patch as you own the code for it.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":128,"context_line":"-----------"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Primary assignee:"},{"line_number":131,"context_line":"  Tomoki Sekiyama"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Work Items"},{"line_number":134,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_f606e31a","line":131,"in_reply_to":"da85f559_c5225fa3","updated":"2015-11-27 11:19:14.000000000","message":"OK, I\u0027ll update the spec.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"51528b5e449123eaf3e24278a5c5c796b707abcc","unresolved":false,"context_lines":[{"line_number":147,"context_line":"Testing"},{"line_number":148,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"Unit tests will be added."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"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":4,"id":"9a8ffd7b_df18f986","line":150,"updated":"2015-11-25 15:09:32.000000000","message":"I\u0027d love to also have integration tests for this.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":147,"context_line":"Testing"},{"line_number":148,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"Unit tests will be added."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"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":4,"id":"9a8ffd7b_e25dc247","line":150,"in_reply_to":"9a8ffd7b_4d2de7fc","updated":"2015-11-27 11:19:14.000000000","message":"@Daisuke,\n\nThank you. I\u0027ll add you to \"Assignee\" section. If you don\u0027t want to add, please let me know.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":14162,"name":"Daisuke Fujita","email":"fuzita.daisuke@jp.fujitsu.com","username":"daisukefujita"},"change_message_id":"cf2ba778f3554b4daf387fdf902c2b8ce7fc1137","unresolved":false,"context_lines":[{"line_number":147,"context_line":"Testing"},{"line_number":148,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"Unit tests will be added."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"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":4,"id":"9a8ffd7b_4d2de7fc","line":150,"in_reply_to":"9a8ffd7b_df18f986","updated":"2015-11-27 08:48:52.000000000","message":"@Jim,\n\nShould devstack/tempest be updated? Like Ironic/Neutron integration test[1].\n\nWe, Fujitsu, would like to work for integration test.\n\n[1] http://lists.openstack.org/pipermail/openstack-dev/2015-November/080492.html","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"51528b5e449123eaf3e24278a5c5c796b707abcc","unresolved":false,"context_lines":[{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"None."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"References"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_22356ed9","line":155,"updated":"2015-11-25 15:09:32.000000000","message":"Are there docs that say Ironic does not support boot-from-volume that need to be changed?\n\nAre there operator docs that need details on how to configure this?","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":14162,"name":"Daisuke Fujita","email":"fuzita.daisuke@jp.fujitsu.com","username":"daisukefujita"},"change_message_id":"cf2ba778f3554b4daf387fdf902c2b8ce7fc1137","unresolved":false,"context_lines":[{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"None."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"References"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_cd28f7e4","line":155,"in_reply_to":"9a8ffd7b_22356ed9","updated":"2015-11-27 08:48:52.000000000","message":"@Jim,\n\nI think that the following documents should be updated, is it right?\n\n\u003e Are there docs that say Ironic does not support boot-from-volume that need to be changed?\n\nhttp://docs.openstack.org/developer/ironic/deploy/install-guide.html\n\n*Local boot with partition images\n\n\u003e Are there operator docs that need details on how to configure this?\n\nhttp://docs.openstack.org/user-guide/cli_launch_instances.html\n\n*Launch instances","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"bcd96d6832888ecf18d4790e7f5c34cb249f4a6d","unresolved":false,"context_lines":[{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"None."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"References"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_bccbda1d","line":155,"in_reply_to":"9a8ffd7b_969af76c","updated":"2015-12-01 12:19:43.000000000","message":"@Daisuke\n\nI have second thought.\n\nFirst one is \"Ironic\" developer document and so it should be covered in Ironic spec. Also, though I may miss something... I can\u0027t find anything that we need to update in the second documentation.\n\nInstead, we need to update following nova\u0027s developers documentation.\n\nhttp://docs.openstack.org/developer/nova/project_scope.html\n\nhttp://docs.openstack.org/developer/nova/support-matrix.html","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":14162,"name":"Daisuke Fujita","email":"fuzita.daisuke@jp.fujitsu.com","username":"daisukefujita"},"change_message_id":"3094ae8f0eb765ab263b912406612abc6d8cd948","unresolved":false,"context_lines":[{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"None."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"References"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_100522c1","line":155,"in_reply_to":"9a8ffd7b_bccbda1d","updated":"2015-12-01 13:35:00.000000000","message":"@Satoru,\n\nI agree.\n\nI read carefully again the second documentation, it seems that there is not the operation which a user minds for booting bare metals from Cinder volume in CLI.\n\n\u003e Instead, we need to update following nova\u0027s developers documentation.\n\n+1","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"8f0f01cc22bd051d11bc2286149ab0339eb2a842","unresolved":false,"context_lines":[{"line_number":152,"context_line":"Documentation Impact"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"None."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"References"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a8ffd7b_969af76c","line":155,"in_reply_to":"9a8ffd7b_cd28f7e4","updated":"2015-11-27 11:19:14.000000000","message":"@Daisuke,\n\nThanks. I\u0027ll add them here.","commit_id":"8e0f123d8193e8bbf21a836ba0afa34cce3556ed"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"9d1f53d0615ae410e4847439b1685ebb72653423","unresolved":false,"context_lines":[{"line_number":162,"context_line":"Testing"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"Unit tests will be added."},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Documentation Impact"},{"line_number":168,"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":5,"id":"9a8ffd7b_f497ee8d","line":165,"updated":"2015-12-02 13:25:30.000000000","message":"Again, I think this needs integration tests in the gate via tempest/devstack","commit_id":"bbf4b3f9e5ee2f379eb07c3ee58166c762cc94c2"},{"author":{"_account_id":9066,"name":"Satoru Moriya","email":"satoru.moriya.br@hitachi.com","username":"satoru-moriya-br"},"change_message_id":"100b9ddc51027d7e9d241cb6644aa06c74d22df6","unresolved":false,"context_lines":[{"line_number":162,"context_line":"Testing"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"Unit tests will be added."},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Documentation Impact"},{"line_number":168,"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":5,"id":"9a8ffd7b_e92b70e9","line":165,"in_reply_to":"9a8ffd7b_f497ee8d","updated":"2015-12-03 01:35:55.000000000","message":"Thanks, I\u0027ll update soon.","commit_id":"bbf4b3f9e5ee2f379eb07c3ee58166c762cc94c2"},{"author":{"_account_id":14427,"name":"Blake Caldwell","email":"blakec@ornl.gov","username":"caldweb"},"change_message_id":"be53c6a06ef2833bbf391e1dea4b503fe7743d05","unresolved":false,"context_lines":[{"line_number":36,"context_line":"better data protection at lower cost including support for snapshots and"},{"line_number":37,"context_line":"replication.  A tenant can request a bare metal server that boots from a"},{"line_number":38,"context_line":"Cinder volume."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Project Priority"},{"line_number":41,"context_line":"-----------------"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a122d0e_aa9f44b2","line":39,"updated":"2016-04-29 16:46:53.000000000","message":"Some environments want to boot bare metal hardware using a network image that is external to OpenStack (even Cinder), yet manage the node with Ironic  for IPMI, console, and re-provisionining capabilities.","commit_id":"67f1fe8234a51f65324159723aec226991b263a6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"cbe89495852e19f48faac4e46d84d08567654192","unresolved":false,"context_lines":[{"line_number":67,"context_line":"  ``POST /v1/volume_target?node\u003d\u003cnode_uuid\u003e`` in the Ironic API. The post"},{"line_number":68,"context_line":"  data will contain the ``boot_index``, ``serial``, ``type`` and the connection"},{"line_number":69,"context_line":"  information for the remote volume within ``data`` attribute in"},{"line_number":70,"context_line":"  ``connection_info``."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"  The scheduler changes to pick up a suitable bare metal node which"},{"line_number":73,"context_line":"  can support booting from a particular volume should be taken care of in the"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1af94dfe_230b73d7","line":70,"updated":"2016-03-18 20:57:00.000000000","message":"I would like to see the information to be posted to the API prior to the ironic node.validate call being performed \"validate we are ready to do the deploy\", and upon failure of that call the driver needs to cleanup the data it posted to the API.","commit_id":"67f1fe8234a51f65324159723aec226991b263a6"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"6330caddb869c059bcc12600b0889e705d042dd3","unresolved":false,"context_lines":[{"line_number":167,"context_line":"Documentation Impact"},{"line_number":168,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Following documents will be update to explain boot from volume with Ironic."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"* Nova developer Documentation"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"fa69d971_854098a6","line":170,"updated":"2016-01-11 12:18:11.000000000","message":"Typo: \"updated\"","commit_id":"67f1fe8234a51f65324159723aec226991b263a6"}]}