)]}'
{"specs/mitaka/approved/consistent-nova-instance-progress.rst":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"00951eaaea1b4f2ba037a268d9dc5e80b6b67b86","unresolved":false,"context_lines":[{"line_number":16,"context_line":"http://developer.openstack.org/api-ref-compute-v2.1.html#listDetailServers"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"this is correct semantics for vmwaareapi and xenapi driver, but in libvirt"},{"line_number":19,"context_line":"driver, this field is assigned as progress of live migration."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This BP will make the progress field as a percentage value of the build"},{"line_number":22,"context_line":"progress for libvirt driver."}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba8a016a_2753096d","line":19,"updated":"2015-11-24 10:47:15.000000000","message":"No, the other drivers use it for progress of resize operations as well. This progress field is for providing completion progress of whatever long running background job is currently active on an instance.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"00951eaaea1b4f2ba037a268d9dc5e80b6b67b86","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"In libvirt driver, the progress field of instance is assigned as progress of"},{"line_number":28,"context_line":"live migration, this is not correct semantics."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This is confused."},{"line_number":31,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba8a016a_e7df41d4","line":28,"updated":"2015-11-24 10:47:15.000000000","message":"It is absolutely fine. The bug here is the API doc saying the progress is only for the build operation. It applies to any long running operation.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"},{"author":{"_account_id":11189,"name":"yuntongjin","email":"yuntong.jin@intel.com","username":"yuntong"},"change_message_id":"8c9f5ceaa4d82e65ae8d3a0035c301e24ac7c71c","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"In libvirt driver, the progress field of instance is assigned as progress of"},{"line_number":28,"context_line":"live migration, this is not correct semantics."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This is confused."},{"line_number":31,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_2ca60ac2","line":28,"in_reply_to":"9a8ffd7b_c93f35d5","updated":"2015-11-24 14:16:27.000000000","message":"Thanks for the suggestion, will update this one, and propose another one to add migration progress to migration DB.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"7605f56ecad9124ee2d76cce56c80f392e87af81","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"In libvirt driver, the progress field of instance is assigned as progress of"},{"line_number":28,"context_line":"live migration, this is not correct semantics."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This is confused."},{"line_number":31,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_c93f35d5","line":28,"in_reply_to":"9a8ffd7b_de5fe973","updated":"2015-11-24 12:48:27.000000000","message":"1) Any operation which can report progress, should report progress.\n2) Whether we add migration progress in os-migrations API or not is irrelevant. We already report progress via the instance.progress attribute and expose that via the API. For compatibility we must continue to do that, regardless of what else we do.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"},{"author":{"_account_id":11189,"name":"yuntongjin","email":"yuntong.jin@intel.com","username":"yuntong"},"change_message_id":"e877e98c027b4ebe84ef31084dd4c6115d351080","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"In libvirt driver, the progress field of instance is assigned as progress of"},{"line_number":28,"context_line":"live migration, this is not correct semantics."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This is confused."},{"line_number":31,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_de5fe973","line":28,"in_reply_to":"ba8a016a_e7df41d4","updated":"2015-11-24 12:30:20.000000000","message":"1. If so, do you think it\u0027d be valuable to implement build progress in libvirt driver like vmareapi/xenapi does?　\n2. the whole plan is to move migration progress to migration DB, then it will show in os-migrations API, which makes more sense, and\nthink it will be helpful to have it in os-migrations API.\notherwise, with hundreds migration in process, user will have to list os-migrations API, get the instance id from the API, and show instance to get each migration progress.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"},{"author":{"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},"change_message_id":"602f7f71841dbf335695f354e53e9afc7574812f","unresolved":false,"context_lines":[{"line_number":28,"context_line":"live migration, this is not correct semantics."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This is confused."},{"line_number":31,"context_line":" "},{"line_number":32,"context_line":"Use Cases"},{"line_number":33,"context_line":"----------"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba8a016a_4fb8a975","line":31,"updated":"2015-11-24 09:53:19.000000000","message":"white space.","commit_id":"96a26cc07afa9efb6f3d4798cfc7b0dd295f8f06"}]}