)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Base PowerVM support has been added to nova [1]. This updates the"},{"line_number":10,"context_line":"support matrix to include PowerVM features."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[1] https://review.openstack.org/#/q/topic:bp/powervm-nova-compute-driver+status:merged+project:openstack/nova"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1693083"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df140735_6e97e549","line":12,"updated":"2017-06-05 15:18:08.000000000","message":"This link is going to be a moving target, so not useful for posterity.  Better to include an explicit list of reviews/Change-Ids/commits.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"d697bd5afceac4b659b1e14ebd5f1e67c17b25ad","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[1] https://review.openstack.org/#/q/topic:bp/powervm-nova-compute-driver+status:merged+project:openstack/nova"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1693083"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I0583400b546d3cb32f2fe828711df8272c5ab5af"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df140735_3947f9be","line":14,"updated":"2017-06-05 15:35:18.000000000","message":"I was gonna say add \"Partially-Implements: blueprint powervm-nova-compute-driver\" on this, but if we\u0027re linking to a bug I guess that\u0027s probably not necessary. Noting it just to see if anyone else wants to chime in and say we should add it.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"4af1b2120569fcc50ff8acb1ca350d0377191404","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Base PowerVM support has been added to nova [1]. This updates the"},{"line_number":10,"context_line":"support matrix to include PowerVM features."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[1] https://review.openstack.org/#/q/topic:bp/powervm-nova-compute-driver+status:merged+project:openstack/nova"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1693083"},{"line_number":15,"context_line":"Partially-Implements: blueprint powervm-nova-compute-driver"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"df140735_93dfdd64","line":12,"updated":"2017-06-06 15:36:53.000000000","message":"[copied from PS2]\nThis link is going to be a moving target, so not useful for posterity.  Better to include an explicit list of reviews/Change-Ids/commits.","commit_id":"0327c8fcce9b4e150d2cab619189476bad9b2290"}],"doc/source/support-matrix.ini":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":109,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":110,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":111,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":112,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"[operation.detach-volume]"},{"line_number":115,"context_line":"title\u003dDetach block volume from instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_8e859909","line":112,"updated":"2017-06-05 15:18:08.000000000","message":"Nope, this is for volumes; we just have ephemeral.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"d5fb2e8a6fd0ba26a74a6d8b0045b1b13dd47b19","unresolved":false,"context_lines":[{"line_number":109,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":110,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":111,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":112,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"[operation.detach-volume]"},{"line_number":115,"context_line":"title\u003dDetach block volume from instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_10062853","line":112,"in_reply_to":"df140735_8e859909","updated":"2017-06-06 13:31:31.000000000","message":"Done","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":128,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":129,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":130,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":131,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"[operation.attach-interface]"},{"line_number":134,"context_line":"title\u003dAttach virtual network interface to instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_6e8005f6","line":131,"updated":"2017-06-05 15:18:08.000000000","message":"ditto","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"d5fb2e8a6fd0ba26a74a6d8b0045b1b13dd47b19","unresolved":false,"context_lines":[{"line_number":128,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":129,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":130,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":131,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"[operation.attach-interface]"},{"line_number":134,"context_line":"title\u003dAttach virtual network interface to instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_d0fb1044","line":131,"in_reply_to":"df140735_6e8005f6","updated":"2017-06-06 13:31:31.000000000","message":"Done","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":259,"context_line":"status\u003dmandatory"},{"line_number":260,"context_line":"notes\u003dProvides a quick report on information about the guest instance,"},{"line_number":261,"context_line":"  including the power state, memory allocation, CPU allocation, number"},{"line_number":262,"context_line":"  of vCPUs and cummulative CPU execution time. As well as being"},{"line_number":263,"context_line":"  informational, the power state is used by the compute manager for"},{"line_number":264,"context_line":"  tracking changes in guests. Therefore this operation is considered"},{"line_number":265,"context_line":"  mandatory to support."}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_4ece615a","line":262,"range":{"start_line":262,"start_character":17,"end_line":262,"end_character":19},"updated":"2017-06-05 15:18:08.000000000","message":"typo (unrelated).\n\nWe don\u0027t actually have cumulative CPU execution time supported, even OOT.  We should raise a bug for that.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":8190,"name":"Drew Thorstensen (thorst)","email":"thorst@us.ibm.com","username":"thorst"},"change_message_id":"2564444f8b35d81414d2da5b471f68f37d9f03ce","unresolved":false,"context_lines":[{"line_number":259,"context_line":"status\u003dmandatory"},{"line_number":260,"context_line":"notes\u003dProvides a quick report on information about the guest instance,"},{"line_number":261,"context_line":"  including the power state, memory allocation, CPU allocation, number"},{"line_number":262,"context_line":"  of vCPUs and cummulative CPU execution time. As well as being"},{"line_number":263,"context_line":"  informational, the power state is used by the compute manager for"},{"line_number":264,"context_line":"  tracking changes in guests. Therefore this operation is considered"},{"line_number":265,"context_line":"  mandatory to support."}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_99a6050f","line":262,"range":{"start_line":262,"start_character":17,"end_line":262,"end_character":19},"in_reply_to":"df140735_4ece615a","updated":"2017-06-05 15:28:10.000000000","message":"typo should be fixed in separate change set?","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5c84382805e04d5540d34180f46bf19bd0fd4dd6","unresolved":false,"context_lines":[{"line_number":259,"context_line":"status\u003dmandatory"},{"line_number":260,"context_line":"notes\u003dProvides a quick report on information about the guest instance,"},{"line_number":261,"context_line":"  including the power state, memory allocation, CPU allocation, number"},{"line_number":262,"context_line":"  of vCPUs and cummulative CPU execution time. As well as being"},{"line_number":263,"context_line":"  informational, the power state is used by the compute manager for"},{"line_number":264,"context_line":"  tracking changes in guests. Therefore this operation is considered"},{"line_number":265,"context_line":"  mandatory to support."}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_19a675ac","line":262,"range":{"start_line":262,"start_character":17,"end_line":262,"end_character":19},"in_reply_to":"df140735_99a6050f","updated":"2017-06-05 15:30:18.000000000","message":"Probably, but it\u0027s small enough they might let it slide in this one.  May be worth doing it here and then split it out if someone complains.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_ce67114d","line":279,"range":{"start_line":279,"start_character":20,"end_line":279,"end_character":28},"updated":"2017-06-05 15:18:08.000000000","message":"Looking again, this is currently \u0027partial\u0027.  We definitely need another change set (in pike) to fill out the rest of nova.virt.powervm.vm.InstanceInfo.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"d5fb2e8a6fd0ba26a74a6d8b0045b1b13dd47b19","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_30a00c42","line":279,"range":{"start_line":279,"start_character":20,"end_line":279,"end_character":28},"in_reply_to":"df140735_ce67114d","updated":"2017-06-06 13:31:31.000000000","message":"Done","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":8190,"name":"Drew Thorstensen (thorst)","email":"thorst@us.ibm.com","username":"thorst"},"change_message_id":"2564444f8b35d81414d2da5b471f68f37d9f03ce","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_b90d49e9","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":28},"updated":"2017-06-05 15:28:10.000000000","message":"I think this is missing?\n\nEric - please verify.  We would need:\nhttps://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L1091-L1172","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"d5fb2e8a6fd0ba26a74a6d8b0045b1b13dd47b19","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_d090b0ef","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":28},"in_reply_to":"df140735_19bf95aa","updated":"2017-06-06 13:31:31.000000000","message":"Done","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5c84382805e04d5540d34180f46bf19bd0fd4dd6","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_19bf95aa","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":28},"in_reply_to":"df140735_b90d49e9","updated":"2017-06-05 15:30:18.000000000","message":"Riiight, cause resize is a migration.  So yeah, missing.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"d697bd5afceac4b659b1e14ebd5f1e67c17b25ad","unresolved":false,"context_lines":[{"line_number":1004,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":1005,"context_line":"driver-impl-libvirt-vz-vm\u003dpartial"},{"line_number":1006,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":1007,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":1008,"context_line":""},{"line_number":1009,"context_line":"[storage.block.backend.fibrechannel]"},{"line_number":1010,"context_line":"title\u003dBlock storage over fibre channel"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_8e47d977","line":1007,"updated":"2017-06-05 15:35:18.000000000","message":"I don\u0027t think this is talking about swift/ceph... it talks about swift as an alternative to this, the way I read it. That said, I don\u0027t think this is ephemeral either, so it\u0027s missing.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"481867dcde9c38f7da0e44d024ab0596b903f34c","unresolved":false,"context_lines":[{"line_number":1004,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":1005,"context_line":"driver-impl-libvirt-vz-vm\u003dpartial"},{"line_number":1006,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":1007,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":1008,"context_line":""},{"line_number":1009,"context_line":"[storage.block.backend.fibrechannel]"},{"line_number":1010,"context_line":"title\u003dBlock storage over fibre channel"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_eec095e3","line":1007,"updated":"2017-06-05 15:18:08.000000000","message":"I don\u0027t think this refers to ephemeral boot disk - I think it\u0027s talking about ceph or swift or whatever - so I think \u0027missing\u0027, but verify with thorst.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"d5fb2e8a6fd0ba26a74a6d8b0045b1b13dd47b19","unresolved":false,"context_lines":[{"line_number":1004,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":1005,"context_line":"driver-impl-libvirt-vz-vm\u003dpartial"},{"line_number":1006,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":1007,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":1008,"context_line":""},{"line_number":1009,"context_line":"[storage.block.backend.fibrechannel]"},{"line_number":1010,"context_line":"title\u003dBlock storage over fibre channel"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_70c324ee","line":1007,"in_reply_to":"df140735_8e47d977","updated":"2017-06-06 13:31:31.000000000","message":"Done","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":8190,"name":"Drew Thorstensen (thorst)","email":"thorst@us.ibm.com","username":"thorst"},"change_message_id":"2564444f8b35d81414d2da5b471f68f37d9f03ce","unresolved":false,"context_lines":[{"line_number":1004,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":1005,"context_line":"driver-impl-libvirt-vz-vm\u003dpartial"},{"line_number":1006,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":1007,"context_line":"driver-impl-powervm\u003dcomplete"},{"line_number":1008,"context_line":""},{"line_number":1009,"context_line":"[storage.block.backend.fibrechannel]"},{"line_number":1010,"context_line":"title\u003dBlock storage over fibre channel"}],"source_content_type":"text/x-properties","patch_set":2,"id":"df140735_b9b429f3","line":1007,"in_reply_to":"df140735_eec095e3","updated":"2017-06-05 15:28:10.000000000","message":"Yeah, I think this is missing.  Even OOT.","commit_id":"3f4e7362c534f7be3f6cd75bb3792aef09589264"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"d52716fdacf0b7080f05597624cc0cb85371b6ef","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dpartial"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":3,"id":"df140735_58e282d6","line":279,"updated":"2017-06-06 15:27:41.000000000","message":"This will need to be changed to \u0027complete\u0027 either here or https://review.openstack.org/#/c/471146/ depending which merges first.","commit_id":"677cde2834e38ee63f84e5073b1e4a05329432f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":229,"context_line":"driver-impl-ironic\u003dunknown"},{"line_number":230,"context_line":"driver-impl-libvirt-vz-vm\u003dmissing"},{"line_number":231,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":232,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"[operation.rebuild]"},{"line_number":235,"context_line":"title\u003dRebuild instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_e9f6baf9","line":232,"range":{"start_line":232,"start_character":20,"end_line":232,"end_character":27},"updated":"2017-06-13 21:09:03.000000000","message":"Hmm, are we sure about this one? Evacuate has a default implementation:\n\nhttps://github.com/openstack/nova/blob/bbbb043b992c465f75ebcfbc1d2c653936d1a6e7/nova/compute/manager.py#L2677\n\nWhich is just spawn the instance on another host. Maybe this should be \"unknown\" instead of \"missing\"? Or do you actually know that evacuate doesn\u0027t work with powervm?","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"709f5033d961a60a692cb78640c8ef47c8466212","unresolved":false,"context_lines":[{"line_number":229,"context_line":"driver-impl-ironic\u003dunknown"},{"line_number":230,"context_line":"driver-impl-libvirt-vz-vm\u003dmissing"},{"line_number":231,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":232,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"[operation.rebuild]"},{"line_number":235,"context_line":"title\u003dRebuild instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_197fcb15","line":232,"range":{"start_line":232,"start_character":20,"end_line":232,"end_character":27},"in_reply_to":"7f231b9d_09369653","updated":"2017-06-15 13:39:09.000000000","message":"Done","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"2e2a27b7a2e32d4bce25de6612b104d0de652786","unresolved":false,"context_lines":[{"line_number":229,"context_line":"driver-impl-ironic\u003dunknown"},{"line_number":230,"context_line":"driver-impl-libvirt-vz-vm\u003dmissing"},{"line_number":231,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":232,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"[operation.rebuild]"},{"line_number":235,"context_line":"title\u003dRebuild instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_09369653","line":232,"range":{"start_line":232,"start_character":20,"end_line":232,"end_character":27},"in_reply_to":"7f231b9d_e9f6baf9","updated":"2017-06-13 21:20:02.000000000","message":"@esberglu, let\u0027s change this to \u0027unknown\u0027 and add a to-do in our etherpad to try it out.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":252,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":253,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":254,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":255,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"[operation.get-guest-info]"},{"line_number":258,"context_line":"title\u003dGuest instance status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_c9dd3e71","line":255,"range":{"start_line":255,"start_character":20,"end_line":255,"end_character":27},"updated":"2017-06-13 21:09:03.000000000","message":"same as evacuate above","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"709f5033d961a60a692cb78640c8ef47c8466212","unresolved":false,"context_lines":[{"line_number":252,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":253,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":254,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":255,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"[operation.get-guest-info]"},{"line_number":258,"context_line":"title\u003dGuest instance status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_f9a08f92","line":255,"range":{"start_line":255,"start_character":20,"end_line":255,"end_character":27},"in_reply_to":"7f231b9d_c93f7e23","updated":"2017-06-15 13:39:09.000000000","message":"Done","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"2e2a27b7a2e32d4bce25de6612b104d0de652786","unresolved":false,"context_lines":[{"line_number":252,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":253,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":254,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":255,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"[operation.get-guest-info]"},{"line_number":258,"context_line":"title\u003dGuest instance status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_c93f7e23","line":255,"range":{"start_line":255,"start_character":20,"end_line":255,"end_character":27},"in_reply_to":"7f231b9d_c9dd3e71","updated":"2017-06-13 21:20:02.000000000","message":"Ditto","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dpartial"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_89c0667d","line":279,"range":{"start_line":279,"start_character":20,"end_line":279,"end_character":27},"updated":"2017-06-13 21:09:03.000000000","message":"Why wouldn\u0027t this be complete? The only thing we care about is the power_state, as noted here:\n\nhttps://review.openstack.org/#/c/471146/","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"2e2a27b7a2e32d4bce25de6612b104d0de652786","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dpartial"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_e9035a72","line":279,"range":{"start_line":279,"start_character":20,"end_line":279,"end_character":27},"in_reply_to":"7f231b9d_89c0667d","updated":"2017-06-13 21:20:02.000000000","message":"Yeah, I think we were looking at making this \u0027complete\u0027 as soon as that change merged.  But I\u0027m good changing it now.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"709f5033d961a60a692cb78640c8ef47c8466212","unresolved":false,"context_lines":[{"line_number":276,"context_line":"driver-impl-ironic\u003dcomplete"},{"line_number":277,"context_line":"driver-impl-libvirt-vz-vm\u003dcomplete"},{"line_number":278,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":279,"context_line":"driver-impl-powervm\u003dpartial"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"[operation.get-host-info]"},{"line_number":282,"context_line":"title\u003dGuest host status"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_99075b83","line":279,"range":{"start_line":279,"start_character":20,"end_line":279,"end_character":27},"in_reply_to":"7f231b9d_e9035a72","updated":"2017-06-15 13:39:09.000000000","message":"Done","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_c9741e6b","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":27},"updated":"2017-06-13 21:09:03.000000000","message":"I\u0027m surprised by this one too - why wouldn\u0027t resize work for a basic flavor where you\u0027re just increasing cpu/ram/disk?","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"709f5033d961a60a692cb78640c8ef47c8466212","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_992cfb02","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":27},"in_reply_to":"7f231b9d_49636e43","updated":"2017-06-15 13:39:09.000000000","message":"Done","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"2e2a27b7a2e32d4bce25de6612b104d0de652786","unresolved":false,"context_lines":[{"line_number":482,"context_line":"driver-notes-vz-vm\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":483,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":484,"context_line":"driver-notes-vz-ct\u003dResizing Virtuozzo instances implies guest filesystem resize also"},{"line_number":485,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":"[operation.resume]"},{"line_number":488,"context_line":"title\u003dRestore instance"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_49636e43","line":485,"range":{"start_line":485,"start_character":20,"end_line":485,"end_character":27},"in_reply_to":"7f231b9d_c9741e6b","updated":"2017-06-13 21:20:02.000000000","message":"I thought there was a reason, but I\u0027m probably misremembering.  @esberglu - same \u0027unknown\u0027 + to-do as mentioned above.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":503,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":504,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"[operation.service-control.wtf.com]"},{"line_number":507,"context_line":"title\u003dService control"},{"line_number":508,"context_line":"status\u003doptional"},{"line_number":509,"context_line":"notes\u003dSomething something, dark side, something something."}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_290952dc","line":506,"updated":"2017-06-13 21:09:03.000000000","message":"I don\u0027t even know what this is referring to, or why it\u0027s in here. Separate patch could probably remove this if we don\u0027t know what it is.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ce63c667f3f9d1d2c13672758f278587d737d989","unresolved":false,"context_lines":[{"line_number":503,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":504,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"[operation.service-control.wtf.com]"},{"line_number":507,"context_line":"title\u003dService control"},{"line_number":508,"context_line":"status\u003doptional"},{"line_number":509,"context_line":"notes\u003dSomething something, dark side, something something."}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_c9777ed4","line":506,"in_reply_to":"7f231b9d_290952dc","updated":"2017-06-13 21:19:27.000000000","message":"https://review.openstack.org/#/c/473989/","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"709f5033d961a60a692cb78640c8ef47c8466212","unresolved":false,"context_lines":[{"line_number":503,"context_line":"driver-impl-libvirt-vz-ct\u003dcomplete"},{"line_number":504,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"[operation.service-control.wtf.com]"},{"line_number":507,"context_line":"title\u003dService control"},{"line_number":508,"context_line":"status\u003doptional"},{"line_number":509,"context_line":"notes\u003dSomething something, dark side, something something."}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_19d10b24","line":506,"in_reply_to":"7f231b9d_c9777ed4","updated":"2017-06-15 13:39:09.000000000","message":"Done","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec93e34a2ae5119c3820e2b473f086d7a0dc280c","unresolved":false,"context_lines":[{"line_number":757,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":758,"context_line":"driver-impl-libvirt-vz-vm\u003dmissing"},{"line_number":759,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":760,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":761,"context_line":""},{"line_number":762,"context_line":"[guest.setup.configdrive]"},{"line_number":763,"context_line":"title\u003dConfig drive support"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_c922de52","line":760,"range":{"start_line":760,"start_character":20,"end_line":760,"end_character":27},"updated":"2017-06-13 21:09:03.000000000","message":"I figured disk i/o tuning would be part of this one:\n\nhttps://review.openstack.org/#/c/391288/57/nova/virt/powervm/vm.py\n\nBut doesn\u0027t look like it.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"2e2a27b7a2e32d4bce25de6612b104d0de652786","unresolved":false,"context_lines":[{"line_number":757,"context_line":"driver-impl-ironic\u003dmissing"},{"line_number":758,"context_line":"driver-impl-libvirt-vz-vm\u003dmissing"},{"line_number":759,"context_line":"driver-impl-libvirt-vz-ct\u003dmissing"},{"line_number":760,"context_line":"driver-impl-powervm\u003dmissing"},{"line_number":761,"context_line":""},{"line_number":762,"context_line":"[guest.setup.configdrive]"},{"line_number":763,"context_line":"title\u003dConfig drive support"}],"source_content_type":"text/x-properties","patch_set":6,"id":"7f231b9d_49648e22","line":760,"range":{"start_line":760,"start_character":20,"end_line":760,"end_character":27},"in_reply_to":"7f231b9d_c922de52","updated":"2017-06-13 21:20:02.000000000","message":"No; in fact, assuming the platform even supports it, I\u0027m pretty sure we would have to make changes well beyond openstack to expose it.","commit_id":"d722490fbab2464f6f36d038cbeb95e865f2ed3d"}]}