)]}'
{"nova/virt/powervm/driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"691df58f13344eb592a4d04776fbca60e1ca782c","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # capabilities on the instance rather than on the class."},{"line_number":57,"context_line":"        self.capabilities \u003d {"},{"line_number":58,"context_line":"            \u0027has_imagecache\u0027: False,"},{"line_number":59,"context_line":"            \u0027supports_recreate\u0027: False,"},{"line_number":60,"context_line":"            \u0027supports_migrate_to_same_host\u0027: False,"},{"line_number":61,"context_line":"            \u0027supports_attach_interface\u0027: False,"},{"line_number":62,"context_line":"            \u0027supports_device_tagging\u0027: False,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_1f708472","line":59,"range":{"start_line":59,"start_character":13,"end_line":59,"end_character":30},"updated":"2018-03-23 21:16:55.000000000","message":"I suspect that the powervm driver supports rebuild / evacuate but no one has tested it yet. It should be easy enough to find out just by enabling a rebuild tempest test in our CI and seeing if it works. Tempest doesn\u0027t test evacuate so that would be a manual test.","commit_id":"1f44fde35e56863e1dd71a0b454ad3ea1ce9e00e"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"1c9bcb303fd0cbbae1e9ffd1ca39a86769ac22a9","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # capabilities on the instance rather than on the class."},{"line_number":57,"context_line":"        self.capabilities \u003d {"},{"line_number":58,"context_line":"            \u0027has_imagecache\u0027: False,"},{"line_number":59,"context_line":"            \u0027supports_recreate\u0027: False,"},{"line_number":60,"context_line":"            \u0027supports_migrate_to_same_host\u0027: False,"},{"line_number":61,"context_line":"            \u0027supports_attach_interface\u0027: False,"},{"line_number":62,"context_line":"            \u0027supports_device_tagging\u0027: False,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_2b54b520","line":59,"range":{"start_line":59,"start_character":13,"end_line":59,"end_character":30},"in_reply_to":"bf659307_1f708472","updated":"2018-03-28 19:14:29.000000000","message":"Note: this is actually specific to evacuate, not rebuild [1].\n\nI looked at the code where this is used and it does look like evacuate should just work. It\u0027s odd, then, that hyperv, xen, vmware, and ironic all have this set to False. Am I missing something there? Should we just remove the capability across the board if there\u0027s nothing for a driver to do here?\n\n[1] https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2993-L2994","commit_id":"1f44fde35e56863e1dd71a0b454ad3ea1ce9e00e"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"f9230179416128cfb61f1852eaef360cf5321581","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # capabilities on the instance rather than on the class."},{"line_number":57,"context_line":"        self.capabilities \u003d {"},{"line_number":58,"context_line":"            \u0027has_imagecache\u0027: False,"},{"line_number":59,"context_line":"            \u0027supports_recreate\u0027: False,"},{"line_number":60,"context_line":"            \u0027supports_migrate_to_same_host\u0027: False,"},{"line_number":61,"context_line":"            \u0027supports_attach_interface\u0027: False,"},{"line_number":62,"context_line":"            \u0027supports_device_tagging\u0027: False,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_ab709e09","line":59,"range":{"start_line":59,"start_character":13,"end_line":59,"end_character":30},"in_reply_to":"bf659307_2b54b520","updated":"2018-03-30 06:10:07.000000000","message":"logic like it might indicate that by default, the hypervisor listed above don\u0027t have implementation ,but yes, default method destroy then create should work\n\n\n3108         try:\n3109             with instance.mutated_migration_context():\n3110                 self.driver.rebuild(**kwargs)\n3111         except NotImplementedError:\n3112             # NOTE(rpodolyaka): driver doesn\u0027t provide specialized version\n3113             # of rebuild, fall back to the default implementation\n3114             self._rebuild_default_impl(**kwargs)","commit_id":"1f44fde35e56863e1dd71a0b454ad3ea1ce9e00e"}]}
