)]}'
{"specs/queens/approved/list-show-all-server-migration-types.rst":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b47d7546b1a1c1818e1eebf5706994d7114eaf01","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7f287b81_85badfbd","line":90,"updated":"2017-08-31 12:57:56.000000000","message":"I know that this spec was just a re-propose from pike but I came across the migration statuses and I feel that it would be good to also document the \u0027failed\u0027 and the \u0027error\u0027 statuses as both exists in the code.","commit_id":"b3646d1f8a890b0bebadd532946637caf0a59c2e"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a32e4aa940cd99f207dcf93c12534ceeb2f484af","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7f287b81_bba3259e","line":90,"in_reply_to":"7f287b81_2fd93356","updated":"2017-09-07 07:56:59.000000000","message":"I guess I put my comment in the wrong place. For example in L50 the spec states that the transition ends with \u0027finished\u0027 state. But it is just the happy case. Actually a migration can end up in \u0027error\u0027 or \u0027failed\u0027 state as well.\n\nWill the proposed API only show migrations in the states defined at L50?","commit_id":"b3646d1f8a890b0bebadd532946637caf0a59c2e"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bcac0020aac99894495132131dee2c26e7d61c96","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7f287b81_2fd93356","line":90,"in_reply_to":"7f287b81_85badfbd","updated":"2017-09-07 04:23:25.000000000","message":"They are in-progress migration states.\nSo the \u0027failed\u0027 and the \u0027error\u0027 should not be added.","commit_id":"b3646d1f8a890b0bebadd532946637caf0a59c2e"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"ea1eb0648c58827944c9284aa53f2b34937dd3ed","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7f287b81_c201593a","line":90,"in_reply_to":"7f287b81_bba3259e","updated":"2017-09-07 13:20:50.000000000","message":"\u003e Will the proposed API only show migrations in the states defined at L50?\n\nNo.\nThe proposed API shows the statues in line 70-88 only (\u003d in-progress status).","commit_id":"b3646d1f8a890b0bebadd532946637caf0a59c2e"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"7c0498c305bd7b2d5bb0d1a7959e91596186452b","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7f287b81_450fb087","line":90,"in_reply_to":"7f287b81_c201593a","updated":"2017-09-07 15:05:12.000000000","message":"OK, then I rest my case. This spec is OK as is.","commit_id":"b3646d1f8a890b0bebadd532946637caf0a59c2e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41496cdcff5e143164a5461f0ff3e449f9df1f9b","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to list server migrations and show"},{"line_number":14,"context_line":"a server migration. But they can list or show live migration(s) only."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_2b9c870d","line":13,"range":{"start_line":13,"start_character":36,"end_line":13,"end_character":53},"updated":"2017-09-14 21:29:51.000000000","message":"in-progress server live migrations and show in-progress server live migrations\n\nI think it\u0027s important to be clear that these APIs are only showing in-progress migrations, which is different from how os-migrations works.","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to list server migrations and show"},{"line_number":14,"context_line":"a server migration. But they can list or show live migration(s) only."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_b577a4a0","line":13,"range":{"start_line":13,"start_character":36,"end_line":13,"end_character":53},"in_reply_to":"5f2577a7_2b9c870d","updated":"2017-09-22 18:36:26.000000000","message":"++","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bc8f54cfc036a2c94687a23fd0538f16bb30f758","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to list server migrations and show"},{"line_number":14,"context_line":"a server migration. But they can list or show live migration(s) only."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f5c4f37_33b6d3db","line":13,"range":{"start_line":13,"start_character":36,"end_line":13,"end_character":53},"in_reply_to":"5f2577a7_2b9c870d","updated":"2017-09-21 05:27:40.000000000","message":"Done","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec32d9a5543f8e6f4d6837a025544035a4aba1ba","unresolved":false,"context_lines":[{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_cba4e3a8","line":25,"range":{"start_line":25,"start_character":16,"end_line":25,"end_character":24},"updated":"2017-09-14 21:20:17.000000000","message":"currently","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41496cdcff5e143164a5461f0ff3e449f9df1f9b","unresolved":false,"context_lines":[{"line_number":21,"context_line":"Problem description"},{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_ab8757b5","line":25,"range":{"start_line":24,"start_character":64,"end_line":25,"end_character":10},"updated":"2017-09-14 21:29:51.000000000","message":"in-progress cold migrations","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41496cdcff5e143164a5461f0ff3e449f9df1f9b","unresolved":false,"context_lines":[{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_4bc0f3e1","line":25,"range":{"start_line":25,"start_character":44,"end_line":25,"end_character":59},"updated":"2017-09-14 21:29:51.000000000","message":"in-progress live migrations","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bc8f54cfc036a2c94687a23fd0538f16bb30f758","unresolved":false,"context_lines":[{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f5c4f37_d34777bc","line":25,"range":{"start_line":25,"start_character":44,"end_line":25,"end_character":59},"in_reply_to":"5f2577a7_4bc0f3e1","updated":"2017-09-21 05:27:40.000000000","message":"Done","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bc8f54cfc036a2c94687a23fd0538f16bb30f758","unresolved":false,"context_lines":[{"line_number":21,"context_line":"Problem description"},{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f5c4f37_133e2f29","line":25,"range":{"start_line":24,"start_character":64,"end_line":25,"end_character":10},"in_reply_to":"5f2577a7_ab8757b5","updated":"2017-09-21 05:27:40.000000000","message":"Done","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bc8f54cfc036a2c94687a23fd0538f16bb30f758","unresolved":false,"context_lines":[{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"To abort cold migrations [1]_, administrators have to list/show cold"},{"line_number":25,"context_line":"migrations. But curently they can list/show live migrations only"},{"line_number":26,"context_line":"in server migrations APIs."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f5c4f37_b34403b9","line":25,"range":{"start_line":25,"start_character":16,"end_line":25,"end_character":24},"in_reply_to":"5f2577a7_cba4e3a8","updated":"2017-09-21 05:27:40.000000000","message":"Done","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41496cdcff5e143164a5461f0ff3e449f9df1f9b","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Use Cases"},{"line_number":29,"context_line":"---------"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Operators want to list all in-progress migrations in the cloud [1]_."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Proposed change"},{"line_number":34,"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":3,"id":"5f2577a7_4b335315","line":31,"updated":"2017-09-14 21:29:51.000000000","message":"One could argue that if we\u0027re just doing this change so we can eventually support aborting a cold migration, then why do we care about showing in-progress resize or evacuate operations?\n\nI\u0027ve always thought it was confusing that these APIs just show in-progress live migrations, since personally I\u0027d want to see all migrations for a given server regardless of type or state and then perform actions on them, and let those actions validate if the type/status is valid for that operation, but it\u0027s probably too late for that now, and that\u0027s what os-migrations is for I guess.","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b96a3ba384e8fe95297a6464571fc10b706aadeb","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_8b237cbd","line":90,"updated":"2017-09-14 20:59:43.000000000","message":"As we discussed at the PTG, could you please simplify the number of states and just mention \"in-progress\" when a migration is in progress ? :-)","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bc8f54cfc036a2c94687a23fd0538f16bb30f758","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f5c4f37_b34be3ae","line":90,"in_reply_to":"5f2577a7_2b692720","updated":"2017-09-21 05:27:40.000000000","message":"Done","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec32d9a5543f8e6f4d6837a025544035a4aba1ba","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Alternatives"},{"line_number":92,"context_line":"------------"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f2577a7_2b692720","line":90,"in_reply_to":"5f2577a7_8b237cbd","updated":"2017-09-14 21:20:17.000000000","message":"One issue that could happen is the force-complete and abort live migration operations require that the migration status is \u0027running\u0027 and that\u0027s what we say is a precondition in the API reference. If we just show the status as \u0027in progress\u0027 now, the user won\u0027t know if they can make the post-copy/cancel call and it would result in a 400. Which is probably OK, but it\u0027s something we should be aware of. I think that\u0027s probably better than exposing all of the various state values out of the REST API.","commit_id":"e92f257c2ea346e945d11d86e36810047236ade5"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to in-progress server live migrations"},{"line_number":14,"context_line":"and show in-progress server live migrations."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_30fa2c42","line":13,"range":{"start_line":13,"start_character":23,"end_line":13,"end_character":31},"updated":"2017-09-22 18:36:26.000000000","message":"s/used to in-progress/used to list in-progress/","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"a91d178906687a00434423ba508359bde58f456e","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to in-progress server live migrations"},{"line_number":14,"context_line":"and show in-progress server live migrations."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_c2977590","line":13,"range":{"start_line":13,"start_character":23,"end_line":13,"end_character":31},"in_reply_to":"7f515b1d_30fa2c42","updated":"2017-09-25 08:05:54.000000000","message":"Done","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to in-progress server live migrations"},{"line_number":14,"context_line":"and show in-progress server live migrations."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_d09e3806","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":44},"updated":"2017-09-22 18:36:26.000000000","message":"I think you meant to say \"and show an in-progress live migration\u0027s details\"","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"a91d178906687a00434423ba508359bde58f456e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to in-progress server live migrations"},{"line_number":14,"context_line":"and show in-progress server live migrations."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_42e94518","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":44},"in_reply_to":"7f515b1d_d09e3806","updated":"2017-09-25 08:05:54.000000000","message":"Done","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* Live migration"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"  - (Skip the definition)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"In-progress migration states are defined as follows:"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_b0bcdc69","line":66,"range":{"start_line":66,"start_character":5,"end_line":66,"end_character":25},"updated":"2017-09-22 18:36:26.000000000","message":"?","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"a91d178906687a00434423ba508359bde58f456e","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* Live migration"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"  - (Skip the definition)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"In-progress migration states are defined as follows:"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_423705b2","line":66,"range":{"start_line":66,"start_character":5,"end_line":66,"end_character":25},"in_reply_to":"7f515b1d_b0bcdc69","updated":"2017-09-25 08:05:54.000000000","message":"This blueprint aims to implements the function except live migration. The function related to live migration remains as it is. So the description is omitted.","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"These in-progress migrations are listed/shown, but the migration status is not"},{"line_number":92,"context_line":"returned in the response."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_50ab681c","line":91,"range":{"start_line":91,"start_character":72,"end_line":91,"end_character":74},"updated":"2017-09-22 18:36:26.000000000","message":"s/is not/will not be/","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"a91d178906687a00434423ba508359bde58f456e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"These in-progress migrations are listed/shown, but the migration status is not"},{"line_number":92,"context_line":"returned in the response."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_a2d91984","line":91,"range":{"start_line":91,"start_character":72,"end_line":91,"end_character":74},"in_reply_to":"7f515b1d_50ab681c","updated":"2017-09-25 08:05:54.000000000","message":"Done","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7c58f6d99387757c63b13b1ff84faa7a59e7c353","unresolved":false,"context_lines":[{"line_number":115,"context_line":"  If \u0027type\u0027 parameter is not specified, all migration types are listed."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"  The valid \u0027type\u0027 parameters are \u0027live-migration\u0027, \u0027migration\u0027,"},{"line_number":118,"context_line":"  \u0027resize\u0027 and \u0027evacuation\u0027."},{"line_number":119,"context_line":"  If \u0027type\u0027 parameter is wrong, nova-api returns 400 error."},{"line_number":120,"context_line":"  So add badRequest(400) to error response codes."},{"line_number":121,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_902440c8","line":118,"updated":"2017-09-22 18:36:26.000000000","message":"Really, I can\u0027t stand that we have a \"resize\" operation that is different than a \"migration\" operation. Both are identical save for the fact that one of them changes resource values -- something that is not germane to the definition of a move operation itself.","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"a91d178906687a00434423ba508359bde58f456e","unresolved":false,"context_lines":[{"line_number":115,"context_line":"  If \u0027type\u0027 parameter is not specified, all migration types are listed."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"  The valid \u0027type\u0027 parameters are \u0027live-migration\u0027, \u0027migration\u0027,"},{"line_number":118,"context_line":"  \u0027resize\u0027 and \u0027evacuation\u0027."},{"line_number":119,"context_line":"  If \u0027type\u0027 parameter is wrong, nova-api returns 400 error."},{"line_number":120,"context_line":"  So add badRequest(400) to error response codes."},{"line_number":121,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_a270f973","line":118,"in_reply_to":"7f515b1d_902440c8","updated":"2017-09-25 08:05:54.000000000","message":"In current implementation, a type of a migration object can be \u0027resize\u0027 or \u0027migration\u0027. This spec will not change the current definition.","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"fc42d67f044751c4fc806e60fe2edfeaa399427d","unresolved":false,"context_lines":[{"line_number":115,"context_line":"  If \u0027type\u0027 parameter is not specified, all migration types are listed."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"  The valid \u0027type\u0027 parameters are \u0027live-migration\u0027, \u0027migration\u0027,"},{"line_number":118,"context_line":"  \u0027resize\u0027 and \u0027evacuation\u0027."},{"line_number":119,"context_line":"  If \u0027type\u0027 parameter is wrong, nova-api returns 400 error."},{"line_number":120,"context_line":"  So add badRequest(400) to error response codes."},{"line_number":121,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7f515b1d_2e7632a6","line":118,"in_reply_to":"7f515b1d_a270f973","updated":"2017-09-25 10:12:14.000000000","message":"Oh, I know :) I wasn\u0027t saying you needed to fix this! Just that I found it annoying :)","commit_id":"7a9f7720131702c4e2f90706c07e1b0e546513cf"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"42ff604c2dd329cd2cdc30c3c9d885ad64479361","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The following APIs are used to list n-progress server live migrations"},{"line_number":14,"context_line":"and show an in-progress live migration\u0027s details."},{"line_number":15,"context_line":"So this blueprint enables us to list and show other migration types"},{"line_number":16,"context_line":"(\u0027evacuation\u0027, \u0027resize\u0027, \u0027migration\u0027)."}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f515b1d_0cbbbb72","line":13,"range":{"start_line":13,"start_character":36,"end_line":13,"end_character":37},"updated":"2017-10-03 14:38:23.000000000","message":"nit: in","commit_id":"7f7b1b78dcbf7c6515b8cf2b0031ae582db938ec"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3c25cefad716843124b337b407ed18d6a309d864","unresolved":false,"context_lines":[{"line_number":88,"context_line":"  - \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027"},{"line_number":89,"context_line":"  - Existing definition. They remains as it is."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"These in-progress migrations are listed/shown, but the migration status will"},{"line_number":92,"context_line":"not be returned in the response."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Alternatives"},{"line_number":95,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f515b1d_75c5520a","line":92,"range":{"start_line":91,"start_character":0,"end_line":92,"end_character":32},"updated":"2017-10-03 16:56:01.000000000","message":"So do you propose to remove the \u0027status\u0027 key from the GET /servers/{server_id}/migrations response, or just report that as \u0027in-progress\u0027 always? Because today it shows the granular in-progress live migration status which is one of \u0027queued\u0027, \u0027preparing\u0027, \u0027running\u0027, \u0027post-migrating\u0027.\n\n\u003clater\u003e\n\nOK I see L123 now which says, \"The migration status is not included in the response.\"","commit_id":"7f7b1b78dcbf7c6515b8cf2b0031ae582db938ec"}]}