)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"e3baf7456e807ae257ac0926416c740af43800ff","unresolved":false,"context_lines":[{"line_number":8016,"context_line":"        \"\"\"Used only for cleanup in case migrate_disk_and_power_off fails.\"\"\""},{"line_number":8017,"context_line":"        try:"},{"line_number":8018,"context_line":"            if os.path.exists(inst_base_resize):"},{"line_number":8019,"context_line":"                shutil.rmtree(inst_base)"},{"line_number":8020,"context_line":"                utils.execute(\u0027mv\u0027, inst_base_resize, inst_base)"},{"line_number":8021,"context_line":"                if not shared_storage:"},{"line_number":8022,"context_line":"                    self._remotefs.remove_dir(dest, inst_base)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df7087c5_907bc1bc","line":8019,"updated":"2018-03-21 21:21:39.000000000","message":"tsk, no UT for this path.","commit_id":"9ed0e56b798b30d6eb9d8c77a28d30aece27aeb3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0064424818c04396200c4161abd6df0bc73d87b3","unresolved":false,"context_lines":[{"line_number":8016,"context_line":"        \"\"\"Used only for cleanup in case migrate_disk_and_power_off fails.\"\"\""},{"line_number":8017,"context_line":"        try:"},{"line_number":8018,"context_line":"            if os.path.exists(inst_base_resize):"},{"line_number":8019,"context_line":"                shutil.rmtree(inst_base)"},{"line_number":8020,"context_line":"                utils.execute(\u0027mv\u0027, inst_base_resize, inst_base)"},{"line_number":8021,"context_line":"                if not shared_storage:"},{"line_number":8022,"context_line":"                    self._remotefs.remove_dir(dest, inst_base)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df7087c5_484fb033","line":8019,"in_reply_to":"df7087c5_8b45d045","updated":"2018-03-22 09:38:16.000000000","message":"It does, actually","commit_id":"9ed0e56b798b30d6eb9d8c77a28d30aece27aeb3"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"503e22557493cba50a3f9ad70b6f2b837c6329d7","unresolved":false,"context_lines":[{"line_number":8016,"context_line":"        \"\"\"Used only for cleanup in case migrate_disk_and_power_off fails.\"\"\""},{"line_number":8017,"context_line":"        try:"},{"line_number":8018,"context_line":"            if os.path.exists(inst_base_resize):"},{"line_number":8019,"context_line":"                shutil.rmtree(inst_base)"},{"line_number":8020,"context_line":"                utils.execute(\u0027mv\u0027, inst_base_resize, inst_base)"},{"line_number":8021,"context_line":"                if not shared_storage:"},{"line_number":8022,"context_line":"                    self._remotefs.remove_dir(dest, inst_base)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df7087c5_8b45d045","line":8019,"in_reply_to":"df7087c5_907bc1bc","updated":"2018-03-21 21:24:45.000000000","message":"Why doesn\u0027t [1] also apply here, stephenfin?\n\n[1] https://review.openstack.org/#/c/554793/1/nova/virt/libvirt/driver.py@1223","commit_id":"9ed0e56b798b30d6eb9d8c77a28d30aece27aeb3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4df8e30d5bbd0f6f9d2133c2014b8960d22dfe99","unresolved":false,"context_lines":[{"line_number":8038,"context_line":"        \"\"\"Used only for cleanup in case migrate_disk_and_power_off fails.\"\"\""},{"line_number":8039,"context_line":"        try:"},{"line_number":8040,"context_line":"            if os.path.exists(inst_base_resize):"},{"line_number":8041,"context_line":"                shutil.rmtree(inst_base)"},{"line_number":8042,"context_line":"                utils.execute(\u0027mv\u0027, inst_base_resize, inst_base)"},{"line_number":8043,"context_line":"                if not shared_storage:"},{"line_number":8044,"context_line":"                    self._remotefs.remove_dir(dest, inst_base)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_68e63419","line":8041,"updated":"2018-03-22 09:41:26.000000000","message":"If you pass the \u0027ignore_errors\u0027 argument here, it should mitigate any potential issues\n\n[1] https://docs.python.org/3/library/shutil.html#shutil.rmtree","commit_id":"a49dbf87f04415fcaa51127e832559b2f2aeddeb"}]}
