)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Co-Authored-By: Jay Lau \u003cliugya@cn.ibm.com\u003e"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"APIImpact"},{"line_number":20,"context_line":"Change-Id: Ic369981d53523f23e82f5145a9cbb705bacdf3e6"},{"line_number":21,"context_line":"Implements: blueprint cold-migration-with-target-ocata"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"1a95cdbc_29b944ea","line":20,"updated":"2016-10-04 17:56:45.000000000","message":"Better to add\n\n Previously-approved: Juno\n\nfrom this history to get a consensus easily.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Co-Authored-By: Jay Lau \u003cliugya@cn.ibm.com\u003e"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"APIImpact"},{"line_number":20,"context_line":"Change-Id: Ic369981d53523f23e82f5145a9cbb705bacdf3e6"},{"line_number":21,"context_line":"Implements: blueprint cold-migration-with-target-ocata"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"1a95cdbc_406e1096","line":20,"in_reply_to":"1a95cdbc_29b944ea","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"}],"specs/ocata/approved/cold-migration-with-target-ocata.rst":[{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"84c4c79e8eee9da997260823cfe9934bd77fd22f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    {"},{"line_number":95,"context_line":"        \"migrate\":"},{"line_number":96,"context_line":"        {"},{"line_number":97,"context_line":"            \"host\": \"target_host\""},{"line_number":98,"context_line":"        }"},{"line_number":99,"context_line":"    }"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1aa78d24_829c5e5e","line":97,"updated":"2016-07-11 13:53:45.000000000","message":"This should be modified to include a force parameter - the following \"check destinations\" version will be the default.","commit_id":"c824c9b04ac8f392b31ec4c82f209c234ef524f4"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"301e767dd7d96f72267e50f0e1d392881f0ed349","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    {"},{"line_number":95,"context_line":"        \"migrate\":"},{"line_number":96,"context_line":"        {"},{"line_number":97,"context_line":"            \"host\": \"target_host\""},{"line_number":98,"context_line":"        }"},{"line_number":99,"context_line":"    }"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1aa78d24_efa242fb","line":97,"in_reply_to":"1aa78d24_829c5e5e","updated":"2016-07-12 06:27:46.000000000","message":"It will be implemented in the subsequent blueprint.\n\nhttps://blueprints.launchpad.net/nova/+spec/check-destination-on-cold-migration","commit_id":"c824c9b04ac8f392b31ec4c82f209c234ef524f4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fd203ba294841823a2e277d98e70dd4a5614dc22","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The aim of this feature is to let operators cold migrate instances with"},{"line_number":14,"context_line":"target host manually."},{"line_number":15,"context_line":"Before the \u0027enable cold migration with target host\u0027 blueprint [1] and spec"},{"line_number":16,"context_line":"were proposed and approved. The blueprint and spec are reproposed."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"},{"line_number":19,"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":12,"id":"5a8bc5a2_b858b28b","line":16,"range":{"start_line":15,"start_character":0,"end_line":16,"end_character":66},"updated":"2016-09-26 11:07:38.000000000","message":"Nit: this is not required.","commit_id":"406b1835d6af1663a623c7751af607a5e41e333a"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"0bb16629bad96dfb17e7fa289027dfefced68d09","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The aim of this feature is to let operators cold migrate instances with"},{"line_number":14,"context_line":"target host manually."},{"line_number":15,"context_line":"Before the \u0027enable cold migration with target host\u0027 blueprint [1] and spec"},{"line_number":16,"context_line":"were proposed and approved. The blueprint and spec are reproposed."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"},{"line_number":19,"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":12,"id":"3a98d1f4_e961743a","line":16,"range":{"start_line":15,"start_character":0,"end_line":16,"end_character":66},"in_reply_to":"5a8bc5a2_b858b28b","updated":"2016-10-02 07:56:31.000000000","message":"Done","commit_id":"406b1835d6af1663a623c7751af607a5e41e333a"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fd203ba294841823a2e277d98e70dd4a5614dc22","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    {"},{"line_number":96,"context_line":"        \"migrate\": {"},{"line_number":97,"context_line":"            \"host\": \"target_host\""},{"line_number":98,"context_line":"        }"},{"line_number":99,"context_line":"    }"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5a8bc5a2_14716ea8","line":97,"updated":"2016-09-26 11:07:38.000000000","message":"We have had to change live-migrate, so its clear when the scheduler should be checking the chosen target, and when it is meant to be a force.\n\nhttp://developer.openstack.org/api-ref/compute/?expanded\u003dlive-migrate-server-os-migratelive-action-detail#live-migrate-server-os-migratelive-action\n\nI think we need to do the same for migrate, when we add the host target.","commit_id":"406b1835d6af1663a623c7751af607a5e41e333a"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"0bb16629bad96dfb17e7fa289027dfefced68d09","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    {"},{"line_number":96,"context_line":"        \"migrate\": {"},{"line_number":97,"context_line":"            \"host\": \"target_host\""},{"line_number":98,"context_line":"        }"},{"line_number":99,"context_line":"    }"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"3a98d1f4_29cfdc29","line":97,"in_reply_to":"5a8bc5a2_14716ea8","updated":"2016-10-02 07:56:31.000000000","message":"I will add \u0027force\u0027 flag in https://review.openstack.org/#/c/334725/ .","commit_id":"406b1835d6af1663a623c7751af607a5e41e333a"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":22,"context_line":"by custom HA plugin."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"It is difficult to migrate a stopped VM instance to the new version"},{"line_number":25,"context_line":"compute node by aiming it when upgrading compute nodes."},{"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":13,"id":"1a95cdbc_ee3042a1","line":25,"updated":"2016-10-04 17:56:45.000000000","message":"+1","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":28,"context_line":"---------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* HA plugin"},{"line_number":31,"context_line":"There is a customized HA plugin which automatically performs migrations"},{"line_number":32,"context_line":"under certain conditions, and the HA plugin is able to intelligently"},{"line_number":33,"context_line":"pick up destinations, but only live migrations support specifying"},{"line_number":34,"context_line":"destinations."}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_8e6c1e9e","line":31,"updated":"2016-10-04 17:56:45.000000000","message":"nit: better to put 2 spaces before line 31 after \"* Foo\".\nCurrent context is a little unreadable according to http://docs-draft.openstack.org/86/334286/13/check/gate-nova-specs-docs-ubuntu-xenial/96ed38d//doc/build/html/specs/ocata/approved/cold-migration-with-target-ocata.html\n\nThe following lies also the same pointed.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":28,"context_line":"---------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* HA plugin"},{"line_number":31,"context_line":"There is a customized HA plugin which automatically performs migrations"},{"line_number":32,"context_line":"under certain conditions, and the HA plugin is able to intelligently"},{"line_number":33,"context_line":"pick up destinations, but only live migrations support specifying"},{"line_number":34,"context_line":"destinations."}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_203ca42e","line":31,"in_reply_to":"1a95cdbc_8e6c1e9e","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1b41483e4e2da5a93486bc87fcc8437266ef8efb","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":"* HA plugin"},{"line_number":31,"context_line":"There is a customized HA plugin which automatically performs migrations"},{"line_number":32,"context_line":"under certain conditions, and the HA plugin is able to intelligently"},{"line_number":33,"context_line":"pick up destinations, but only live migrations support specifying"},{"line_number":34,"context_line":"destinations."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"At the moment cold migration do not support migrate a VM instance with target"},{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_10d82853","line":34,"range":{"start_line":30,"start_character":0,"end_line":34,"end_character":13},"updated":"2016-10-04 19:04:20.000000000","message":"What\u0027s this now?","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","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":"* HA plugin"},{"line_number":31,"context_line":"There is a customized HA plugin which automatically performs migrations"},{"line_number":32,"context_line":"under certain conditions, and the HA plugin is able to intelligently"},{"line_number":33,"context_line":"pick up destinations, but only live migrations support specifying"},{"line_number":34,"context_line":"destinations."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"At the moment cold migration do not support migrate a VM instance with target"},{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_e0163c54","line":34,"range":{"start_line":30,"start_character":0,"end_line":34,"end_character":13},"in_reply_to":"1a95cdbc_10d82853","updated":"2016-10-06 10:54:39.000000000","message":"I added these sentences because they existed in previously approved spec.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":34,"context_line":"destinations."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"At the moment cold migration do not support migrate a VM instance with target"},{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"},{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_eea5a2b5","line":37,"range":{"start_line":37,"start_character":21,"end_line":37,"end_character":32},"updated":"2016-10-04 17:56:45.000000000","message":"nit: s/want to add/wants to add/\n\nor just s/want to add/adds/","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":34,"context_line":"destinations."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"At the moment cold migration do not support migrate a VM instance with target"},{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"},{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_e010fcaa","line":37,"range":{"start_line":37,"start_character":21,"end_line":37,"end_character":32},"in_reply_to":"1a95cdbc_eea5a2b5","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"},{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"},{"line_number":41,"context_line":"migration support migration with and w/o target host."},{"line_number":42,"context_line":"(This use case is transcribed from the blueprint [1] spec.)"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_0e82ce00","line":40,"range":{"start_line":40,"start_character":8,"end_line":40,"end_character":12},"updated":"2016-10-04 17:56:45.000000000","message":"nit: s/make/makes/","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":37,"context_line":"host, this blueprint want to add this feature to nova so that above scenario"},{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"},{"line_number":41,"context_line":"migration support migration with and w/o target host."},{"line_number":42,"context_line":"(This use case is transcribed from the blueprint [1] spec.)"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_2023c406","line":40,"range":{"start_line":40,"start_character":8,"end_line":40,"end_character":12},"in_reply_to":"1a95cdbc_0e82ce00","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1b41483e4e2da5a93486bc87fcc8437266ef8efb","unresolved":false,"context_lines":[{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"},{"line_number":41,"context_line":"migration support migration with and w/o target host."},{"line_number":42,"context_line":"(This use case is transcribed from the blueprint [1] spec.)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Upgrading compute nodes"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_10bf48d1","line":41,"range":{"start_line":41,"start_character":37,"end_line":41,"end_character":40},"updated":"2016-10-04 19:04:20.000000000","message":"\"without\"","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":38,"context_line":"can be satisified."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"It also make cold migration consistent with live-migrate operations as live"},{"line_number":41,"context_line":"migration support migration with and w/o target host."},{"line_number":42,"context_line":"(This use case is transcribed from the blueprint [1] spec.)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Upgrading compute nodes"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_807ad8ef","line":41,"range":{"start_line":41,"start_character":37,"end_line":41,"end_character":40},"in_reply_to":"1a95cdbc_10bf48d1","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":48,"context_line":"another compute node."},{"line_number":49,"context_line":"And in case of rolling upgrade, old and new version compute nodes coexists."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Making old version compute nodes be disable and migrating VM instances"},{"line_number":52,"context_line":"to new version compute nodes."},{"line_number":53,"context_line":"If migrating VM instances via scheduler, they may be migrated to old version"},{"line_number":54,"context_line":"compute node. In that case, the migration to new version compute node could be"},{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_ee4e8295","line":58,"range":{"start_line":51,"start_character":0,"end_line":58,"end_character":18},"updated":"2016-10-04 17:56:45.000000000","message":"I am not sure this block\u0027s meaning, especially line 51-52.\n\nHere just means:\n\n To block scheduling a new VM to old compute nodes, we need\n to make the nodes disable. After that, we just need to\n migrate the existing running/stopped VMs from old compute\n nodes to new compute nodes with specifying a target host.\n\nright?","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1b41483e4e2da5a93486bc87fcc8437266ef8efb","unresolved":false,"context_lines":[{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_70ca4c51","line":58,"updated":"2016-10-04 19:04:20.000000000","message":"I would replace all of the above with just:\n\n \"As an admin, during maintenance I need to be able to move workloads to specific machines to clear off machines needing attention\"","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_abd7b521","line":58,"in_reply_to":"1a95cdbc_70ca4c51","updated":"2016-10-06 10:54:39.000000000","message":"@Dan, it is cold migration use case, so VMs are stopped. IMO, \"move workloads\" is not suitable.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"59dfaad09286a901909d7b1b54483496d5b30a7b","unresolved":false,"context_lines":[{"line_number":48,"context_line":"another compute node."},{"line_number":49,"context_line":"And in case of rolling upgrade, old and new version compute nodes coexists."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Making old version compute nodes be disable and migrating VM instances"},{"line_number":52,"context_line":"to new version compute nodes."},{"line_number":53,"context_line":"If migrating VM instances via scheduler, they may be migrated to old version"},{"line_number":54,"context_line":"compute node. In that case, the migration to new version compute node could be"},{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_f9734ee5","line":58,"range":{"start_line":51,"start_character":0,"end_line":58,"end_character":18},"in_reply_to":"1a95cdbc_eb5c5def","updated":"2016-10-06 13:07:20.000000000","message":"@Ohmichi-san, sorry.\nIt is a little different. I amended the description.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":48,"context_line":"another compute node."},{"line_number":49,"context_line":"And in case of rolling upgrade, old and new version compute nodes coexists."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Making old version compute nodes be disable and migrating VM instances"},{"line_number":52,"context_line":"to new version compute nodes."},{"line_number":53,"context_line":"If migrating VM instances via scheduler, they may be migrated to old version"},{"line_number":54,"context_line":"compute node. In that case, the migration to new version compute node could be"},{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_eb5c5def","line":58,"range":{"start_line":51,"start_character":0,"end_line":58,"end_character":18},"in_reply_to":"1a95cdbc_ee4e8295","updated":"2016-10-06 10:54:39.000000000","message":"@Ohmichi-san, Yes.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"7d9ac3a53b79d6b9fbee9e3d817d810ac2464b40","unresolved":false,"context_lines":[{"line_number":48,"context_line":"another compute node."},{"line_number":49,"context_line":"And in case of rolling upgrade, old and new version compute nodes coexists."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Making old version compute nodes be disable and migrating VM instances"},{"line_number":52,"context_line":"to new version compute nodes."},{"line_number":53,"context_line":"If migrating VM instances via scheduler, they may be migrated to old version"},{"line_number":54,"context_line":"compute node. In that case, the migration to new version compute node could be"},{"line_number":55,"context_line":"occurred again."},{"line_number":56,"context_line":"In order to migrate a stopped VM instance to new version destination host"},{"line_number":57,"context_line":"at one time, add the function to specify a target host (compute node)"},{"line_number":58,"context_line":"in cold migration."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Proposed change"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_948cd136","line":58,"range":{"start_line":51,"start_character":0,"end_line":58,"end_character":18},"in_reply_to":"1a95cdbc_f9734ee5","updated":"2016-10-06 13:15:39.000000000","message":"I assume \"one by one\" upgrade.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    {"},{"line_number":94,"context_line":"        \"migrate\": {"},{"line_number":95,"context_line":"            \"host\": \"target_host\""},{"line_number":96,"context_line":"        }"},{"line_number":97,"context_line":"    }"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_c97380b3","line":95,"range":{"start_line":95,"start_character":12,"end_line":95,"end_character":33},"updated":"2016-10-04 17:56:45.000000000","message":"+1, this parameter seems consistent with os-migrateLive.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaaf3b7a9dfc3ab85a532dd5c4aea19cd95e93c9","unresolved":false,"context_lines":[{"line_number":145,"context_line":"----------"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Add logic to select target host for cold migration"},{"line_number":148,"context_line":"* Add API"},{"line_number":149,"context_line":"* Set target host optional on nova-client"},{"line_number":150,"context_line":"* Add nova functional tests"},{"line_number":151,"context_line":"* Add tempest tests"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_49c4d091","line":148,"range":{"start_line":148,"start_character":0,"end_line":148,"end_character":9},"updated":"2016-10-04 17:56:45.000000000","message":"better to add \"with bumping a new microversion\"","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":145,"context_line":"----------"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Add logic to select target host for cold migration"},{"line_number":148,"context_line":"* Add API"},{"line_number":149,"context_line":"* Set target host optional on nova-client"},{"line_number":150,"context_line":"* Add nova functional tests"},{"line_number":151,"context_line":"* Add tempest tests"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_c0e94082","line":148,"range":{"start_line":148,"start_character":0,"end_line":148,"end_character":9},"in_reply_to":"1a95cdbc_49c4d091","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1b41483e4e2da5a93486bc87fcc8437266ef8efb","unresolved":false,"context_lines":[{"line_number":158,"context_line":""},{"line_number":159,"context_line":"But the subsequent blueprint [2] will be implemented."},{"line_number":160,"context_line":"It will add a function to check whether a destination host is"},{"line_number":161,"context_line":"in accordance with scheduler rules and a \u0027force\u0027 parameter"},{"line_number":162,"context_line":"to the API."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_d3524add","line":161,"updated":"2016-10-04 19:04:20.000000000","message":"I think these should be squashed. This one is not valid without the other, IMHO.","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"1eb2c8244795ea60d64abfdf81545ee130e37492","unresolved":false,"context_lines":[{"line_number":158,"context_line":""},{"line_number":159,"context_line":"But the subsequent blueprint [2] will be implemented."},{"line_number":160,"context_line":"It will add a function to check whether a destination host is"},{"line_number":161,"context_line":"in accordance with scheduler rules and a \u0027force\u0027 parameter"},{"line_number":162,"context_line":"to the API."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":13,"id":"1a95cdbc_208664b9","line":161,"in_reply_to":"1a95cdbc_d3524add","updated":"2016-10-06 10:54:39.000000000","message":"Done","commit_id":"63454dc25caa3ca89205c1fe2726d56e8516f967"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fdfb96e8527dec7b04d8bedcd26253aeec5a8104","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    {"},{"line_number":90,"context_line":"        \"migrate\": {"},{"line_number":91,"context_line":"            \"host\": \"target_host\""},{"line_number":92,"context_line":"        }"},{"line_number":93,"context_line":"    }"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-rst","patch_set":16,"id":"fa6399be_106e2301","line":91,"updated":"2016-10-12 14:49:36.000000000","message":"I am tempted not to add this version, and only add the target+host combo.","commit_id":"3a4f78575248e4432ecb046529e816a44fc886bc"}]}