)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e5a36e368c69bf3b58b30fe20760a2d8d27dc193","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This was approved in Queens at"},{"line_number":13,"context_line":"https://review.openstack.org/#/c/464915"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I23d28acb581447a4cf2cbace63b90852a1ab8c9a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa0c359_ab62b208","line":14,"updated":"2018-02-12 18:52:05.000000000","message":"Should add this line:\n\nPreviously-approved: Queens\n\nhttps://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications","commit_id":"1dd59c048acb9e0d31fdf1302c76ec516ee550e1"}],"specs/rocky/approved/add-zvm-support.rst":[{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"106e031abc50f5cee77f6783dcced6f6c541a964","unresolved":false,"context_lines":[{"line_number":220,"context_line":""},{"line_number":221,"context_line":"* Code submitted in Queens"},{"line_number":222,"context_line":"    * https://review.openstack.org/#/q/topic:bp/add-zvm-driver-queens+(status:open+OR+status:merged)"},{"line_number":223,"context_line":" "},{"line_number":224,"context_line":"History"},{"line_number":225,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":226,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7f96bb07_4a2db944","line":223,"range":{"start_line":223,"start_character":0,"end_line":223,"end_character":1},"updated":"2018-01-22 12:30:18.000000000","message":"White space","commit_id":"e1ab55ea0426bb1bc8867b6f7ffef5fd46f12d11"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"106e031abc50f5cee77f6783dcced6f6c541a964","unresolved":false,"context_lines":[{"line_number":238,"context_line":""},{"line_number":239,"context_line":"   * - Release Name"},{"line_number":240,"context_line":"     - Description"},{"line_number":241,"context_line":"   * - Rocky "},{"line_number":242,"context_line":"     - Introduced"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7f96bb07_0a27b166","line":241,"range":{"start_line":241,"start_character":12,"end_line":241,"end_character":13},"updated":"2018-01-22 12:30:18.000000000","message":"ditto","commit_id":"e1ab55ea0426bb1bc8867b6f7ffef5fd46f12d11"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"73b2aec1c2390d0f49e4989eae4bae17e697647b","unresolved":false,"context_lines":[{"line_number":71,"context_line":"This subset of the driver code (and associated unit tests) would support"},{"line_number":72,"context_line":"features such as:"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"* Basic VM lifecycle tasks (spawn, shutdown, reboot, snapshot, etc)"},{"line_number":75,"context_line":"* Instance status"},{"line_number":76,"context_line":"* console output"},{"line_number":77,"context_line":"* Flat/VLAN networking using the z/VM neutron agent"},{"line_number":78,"context_line":"* Config drive"},{"line_number":79,"context_line":"* Live Migrate"},{"line_number":80,"context_line":"* Cinder Volume Support"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"This phase of the driver is meant to get the net minimum of `mandatory` and"},{"line_number":83,"context_line":"`choice` options from the `support matrix`_."}],"source_content_type":"text/x-rst","patch_set":2,"id":"5f93b717_797b1927","line":80,"range":{"start_line":74,"start_character":0,"end_line":80,"end_character":23},"updated":"2018-02-01 13:43:16.000000000","message":"Based on the PowerVM experience [1], this is probably too ambitious for one release.  You could leave it like it is, but understand that there\u0027s a pretty low probability all of this will get in.\n\nOr you could list the breakdown of all the features you plan to port, in the order in which you plan to port them, and make the goal of the blueprint to get in \"as many as you can\".\n\nOr you could split the list as you\u0027ve done, but more realistically.  Having a realistic concrete list in an approved blueprint would (theoretically) give weight to review requests for those items.\n\n[1] https://review.openstack.org/#/q/status:merged+project:openstack/nova+branch:master+topic:bp/powervm-nova-compute-driver","commit_id":"0f7600263bf418f41d21d8cc9df8845803bebb53"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"c4b9d786dc5a767a4357949e68feeb95cf9cb744","unresolved":false,"context_lines":[{"line_number":71,"context_line":"This subset of the driver code (and associated unit tests) would support"},{"line_number":72,"context_line":"features such as:"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"* Basic VM lifecycle tasks (spawn, shutdown, reboot, snapshot, etc)"},{"line_number":75,"context_line":"* Instance status"},{"line_number":76,"context_line":"* console output"},{"line_number":77,"context_line":"* Flat/VLAN networking using the z/VM neutron agent"},{"line_number":78,"context_line":"* Config drive"},{"line_number":79,"context_line":"* Live Migrate"},{"line_number":80,"context_line":"* Cinder Volume Support"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"This phase of the driver is meant to get the net minimum of `mandatory` and"},{"line_number":83,"context_line":"`choice` options from the `support matrix`_."}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa0c359_268537ae","line":80,"range":{"start_line":74,"start_character":0,"end_line":80,"end_character":23},"in_reply_to":"5f93b717_797b1927","updated":"2018-02-05 07:38:21.000000000","message":"thanks for the sharing\n\nI understood those are too much :) , only purpose is to list as much as we can then hopefully those can be merged at stage 1 of the Rocky release, if not ,from top to down is the priority we want to get it in\n\nanyway, I removed some to make it more reasonable and realistic to the review band.. thanks","commit_id":"0f7600263bf418f41d21d8cc9df8845803bebb53"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"73b2aec1c2390d0f49e4989eae4bae17e697647b","unresolved":false,"context_lines":[{"line_number":241,"context_line":".. list-table:: Revisions"},{"line_number":242,"context_line":"   :header-rows: 1"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"   * - Release Name"},{"line_number":245,"context_line":"     - Description"},{"line_number":246,"context_line":"   * - Rocky"},{"line_number":247,"context_line":"     - Introduced"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5f93b717_793eb9b8","line":247,"range":{"start_line":244,"start_character":0,"end_line":247,"end_character":17},"updated":"2018-02-01 13:43:16.000000000","message":"This should reflect that it was introduced and approved in Queens, and reproposed in Rocky.","commit_id":"0f7600263bf418f41d21d8cc9df8845803bebb53"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"c4b9d786dc5a767a4357949e68feeb95cf9cb744","unresolved":false,"context_lines":[{"line_number":241,"context_line":".. list-table:: Revisions"},{"line_number":242,"context_line":"   :header-rows: 1"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"   * - Release Name"},{"line_number":245,"context_line":"     - Description"},{"line_number":246,"context_line":"   * - Rocky"},{"line_number":247,"context_line":"     - Introduced"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa0c359_c691f3f3","line":247,"range":{"start_line":244,"start_character":0,"end_line":247,"end_character":17},"in_reply_to":"5f93b717_793eb9b8","updated":"2018-02-05 07:38:21.000000000","message":"Thanks , I updated it just now","commit_id":"0f7600263bf418f41d21d8cc9df8845803bebb53"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"92b739ded7be87d5adde9484515c05615f9470e6","unresolved":false,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":" This work is licensed under a Creative Commons Attribution 3.0 Unported"},{"line_number":3,"context_line":" License."},{"line_number":4,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_4eef7c1d","line":1,"updated":"2018-02-12 19:02:40.000000000","message":"The filename is wrong, it needs to match the blueprint name, so add-zvm-driver-rocky.rst.","commit_id":"1dd59c048acb9e0d31fdf1302c76ec516ee550e1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"23d7d68cd921fe0c6310df4f8e5b09cfd23741b5","unresolved":false,"context_lines":[{"line_number":17,"context_line":"These z/VM virtual servers can run Linux, z/OS and more."},{"line_number":18,"context_line":"The detailed information can be found at http://www.vm.ibm.com/"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This spec was previously approved at"},{"line_number":21,"context_line":"https://review.openstack.org/#/c/464915"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_0ea7340d","line":21,"range":{"start_line":20,"start_character":0,"end_line":21,"end_character":39},"updated":"2018-02-12 19:00:37.000000000","message":"You can remove this, it\u0027s part of the History section already.","commit_id":"1dd59c048acb9e0d31fdf1302c76ec516ee550e1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c9e498188e86aa16142d060e6cd0dc979a4b83ab","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Some of the specific functions supported in out-of-tree driver now"},{"line_number":88,"context_line":"that would come as part of future blueprints that are not part of this one:"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"* Resize"},{"line_number":91,"context_line":"* Cold Migrate"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_14f05a90","line":89,"updated":"2018-02-05 13:55:49.000000000","message":"You could/should still put those other items in this part of the list.","commit_id":"1dd59c048acb9e0d31fdf1302c76ec516ee550e1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"23d7d68cd921fe0c6310df4f8e5b09cfd23741b5","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    * Bugs: `\u003chttps://bugs.launchpad.net/ceilometer-zvm/\u003e`_"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"* Code submitted in Queens"},{"line_number":224,"context_line":"    * https://review.openstack.org/#/q/topic:bp/add-zvm-driver-queens+(status:open+OR+status:merged)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"History"},{"line_number":227,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_ceacecee","line":224,"range":{"start_line":224,"start_character":45,"end_line":224,"end_character":69},"updated":"2018-02-12 19:00:37.000000000","message":"If this is the queens blueprint, what is this tracked against for rocky?\n\nI guess the blueprint was renamed?\n\nhttps://blueprints.launchpad.net/nova/+spec/add-zvm-driver-rocky\n\nWhich means the topic branch for those patches will change, so there isn\u0027t much point in linking those in here, so remove this part about code submitted in Queens. It would only be useful to point out what was merged in queens if there was anything merged, but there wasn\u0027t.","commit_id":"1dd59c048acb9e0d31fdf1302c76ec516ee550e1"}]}