)]}'
{"tripleo_common/tests/utils/test_stack.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                stack.deploy_stack(swift, heat, \u0027overcloud\u0027)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        # verify parameters are as expected"},{"line_number":156,"context_line":"        expected_defaults \u003d {\u0027DeployIdentifier\u0027: 1473366264,"},{"line_number":157,"context_line":"                             \u0027StackAction\u0027: \u0027CREATE\u0027,"},{"line_number":158,"context_line":"                             \u0027random_existing_data\u0027: \u0027a_value\u0027}"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_6c8243c9","line":156,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F841 local variable \u0027expected_defaults\u0027 is assigned to but never used","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                stack.deploy_stack(swift, heat, \u0027overcloud\u0027)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        # verify parameters are as expected"},{"line_number":156,"context_line":"        expected_defaults \u003d {\u0027DeployIdentifier\u0027: 1473366264,"},{"line_number":157,"context_line":"                             \u0027StackAction\u0027: \u0027CREATE\u0027,"},{"line_number":158,"context_line":"                             \u0027random_existing_data\u0027: \u0027a_value\u0027}"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_4c601f36","line":156,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F841 local variable \u0027expected_defaults\u0027 is assigned to but never used","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":252,"context_line":"                mock_get_template_contents.return_value \u003d ({}, {"},{"line_number":253,"context_line":"                    \u0027heat_template_version\u0027: \u00272016-04-30\u0027"},{"line_number":254,"context_line":"                })"},{"line_number":255,"context_line":"                mock_process_multiple_environments_and_files.return_value \u003d ({}, {})"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"                # freeze time at datetime.datetime(2016, 9, 8, 16, 24, 24)"},{"line_number":258,"context_line":"                mock_time.time.return_value \u003d 1473366264"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_cc6a6f19","line":255,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E501 line too long (84 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":252,"context_line":"                mock_get_template_contents.return_value \u003d ({}, {"},{"line_number":253,"context_line":"                    \u0027heat_template_version\u0027: \u00272016-04-30\u0027"},{"line_number":254,"context_line":"                })"},{"line_number":255,"context_line":"                mock_process_multiple_environments_and_files.return_value \u003d ({}, {})"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"                # freeze time at datetime.datetime(2016, 9, 8, 16, 24, 24)"},{"line_number":258,"context_line":"                mock_time.time.return_value \u003d 1473366264"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_2c65eb45","line":255,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E501 line too long (84 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":258,"context_line":"                mock_time.time.return_value \u003d 1473366264"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"                self.assertRaises(RuntimeError, stack.deploy_stack,"},{"line_number":261,"context_line":"                                swift, heat, \u0027overcloud\u0027)"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"    @mock.patch(\u0027tripleo_common.update.check_neutron_mechanism_drivers\u0027)"},{"line_number":264,"context_line":"    @mock.patch(\u0027tripleo_common.utils.stack.time\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ac673b51","line":261,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E128 continuation line under-indented for visual indent","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":258,"context_line":"                mock_time.time.return_value \u003d 1473366264"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"                self.assertRaises(RuntimeError, stack.deploy_stack,"},{"line_number":261,"context_line":"                                swift, heat, \u0027overcloud\u0027)"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"    @mock.patch(\u0027tripleo_common.update.check_neutron_mechanism_drivers\u0027)"},{"line_number":264,"context_line":"    @mock.patch(\u0027tripleo_common.utils.stack.time\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_8c56b75d","line":261,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":294,"context_line":"                mock_get_template_contents.return_value \u003d ({}, {"},{"line_number":295,"context_line":"                    \u0027heat_template_version\u0027: \u00272016-04-30\u0027"},{"line_number":296,"context_line":"                })"},{"line_number":297,"context_line":"                mock_process_multiple_environments_and_files.return_value \u003d ({}, {})"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"                # freeze time at datetime.datetime(2016, 9, 8, 16, 24, 24)"},{"line_number":300,"context_line":"                mock_time.time.return_value \u003d 1473366264"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_0c71070e","line":297,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E501 line too long (84 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":294,"context_line":"                mock_get_template_contents.return_value \u003d ({}, {"},{"line_number":295,"context_line":"                    \u0027heat_template_version\u0027: \u00272016-04-30\u0027"},{"line_number":296,"context_line":"                })"},{"line_number":297,"context_line":"                mock_process_multiple_environments_and_files.return_value \u003d ({}, {})"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"                # freeze time at datetime.datetime(2016, 9, 8, 16, 24, 24)"},{"line_number":300,"context_line":"                mock_time.time.return_value \u003d 1473366264"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_6c5b6385","line":297,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E501 line too long (84 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":301,"context_line":"                mock_check_neutron_drivers.return_value \u003d None"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"                self.assertRaises(RuntimeError, stack.deploy_stack,"},{"line_number":304,"context_line":"                                swift, heat, \u0027overcloud\u0027)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    def test_set_tls_parameters_no_ca_found(self):"},{"line_number":307,"context_line":"        my_params \u003d {}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ec6db32f","line":304,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E128 continuation line under-indented for visual indent","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":301,"context_line":"                mock_check_neutron_drivers.return_value \u003d None"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"                self.assertRaises(RuntimeError, stack.deploy_stack,"},{"line_number":304,"context_line":"                                swift, heat, \u0027overcloud\u0027)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    def test_set_tls_parameters_no_ca_found(self):"},{"line_number":307,"context_line":"        my_params \u003d {}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_cc538f6b","line":304,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"}],"tripleo_common/tests/utils/test_swift.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        )"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_delete_container_success(self):"},{"line_number":42,"context_line":"        swift_utils.empty_container(self.swiftclient, self.container_name)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        mock_calls \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_6cffe365","line":41,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E303 too many blank lines (2)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        )"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_delete_container_success(self):"},{"line_number":42,"context_line":"        swift_utils.empty_container(self.swiftclient, self.container_name)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        mock_calls \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_acf45b88","line":41,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E303 too many blank lines (2)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    def test_put_object_string_from_bytes(self):"},{"line_number":86,"context_line":"        with mock.patch(\u0027builtins.open\u0027, mock.mock_open()):"},{"line_number":87,"context_line":"            swift_utils.put_object_string(self.swiftclient, \u0027foo\u0027, \u0027bar\u0027, b\u0027foo\u0027)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    @mock.patch(\u0027time.time\u0027)"},{"line_number":90,"context_line":"    @mock.patch(\u0027uuid.uuid4\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ccf70f8c","line":87,"updated":"2020-09-23 02:59:21.000000000","message":"linters: E501 line too long (81 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    def test_put_object_string_from_bytes(self):"},{"line_number":86,"context_line":"        with mock.patch(\u0027builtins.open\u0027, mock.mock_open()):"},{"line_number":87,"context_line":"            swift_utils.put_object_string(self.swiftclient, \u0027foo\u0027, \u0027bar\u0027, b\u0027foo\u0027)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    @mock.patch(\u0027time.time\u0027)"},{"line_number":90,"context_line":"    @mock.patch(\u0027uuid.uuid4\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_0cee27b4","line":87,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: E501 line too long (81 \u003e 79 characters)","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"}],"tripleo_common/utils/artifact_store.py":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"546f9771b4acc3e3d058b2875ae6fd67175252c3","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        raise RuntimeError("},{"line_number":32,"context_line":"            \u0027The store to delete was marked as the root data store.\u0027"},{"line_number":33,"context_line":"            \u0027This is wrong. Check config and try again.\u0027"},{"line_number":34,"context_line":"        )"},{"line_number":35,"context_line":"    shutil.rmtree(store)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_7b502e0a","line":34,"updated":"2020-10-09 11:40:00.000000000","message":"what happens if, for #reasons, the store doesn\u0027t exist? Might be good calling \"os.path.exists\"?","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"9e4de19329137a33bdbe22e841730ac6500d67ea","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        raise RuntimeError("},{"line_number":32,"context_line":"            \u0027The store to delete was marked as the root data store.\u0027"},{"line_number":33,"context_line":"            \u0027This is wrong. Check config and try again.\u0027"},{"line_number":34,"context_line":"        )"},{"line_number":35,"context_line":"    shutil.rmtree(store)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_1b676dec","line":34,"in_reply_to":"9f560f44_7b502e0a","updated":"2020-10-09 17:37:35.000000000","message":"Done","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"546f9771b4acc3e3d058b2875ae6fd67175252c3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"            \u0027The store to delete was marked as the root data store.\u0027"},{"line_number":33,"context_line":"            \u0027This is wrong. Check config and try again.\u0027"},{"line_number":34,"context_line":"        )"},{"line_number":35,"context_line":"    shutil.rmtree(store)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"def create_directory(directory):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_5b556af8","line":35,"updated":"2020-10-09 11:40:00.000000000","message":"no catching any error, such as \"permission denied\" or anything? Might be good doing some clean fallback in case there\u0027s something wrong","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"9e4de19329137a33bdbe22e841730ac6500d67ea","unresolved":false,"context_lines":[{"line_number":32,"context_line":"            \u0027The store to delete was marked as the root data store.\u0027"},{"line_number":33,"context_line":"            \u0027This is wrong. Check config and try again.\u0027"},{"line_number":34,"context_line":"        )"},{"line_number":35,"context_line":"    shutil.rmtree(store)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"def create_directory(directory):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_bb6e41c5","line":35,"in_reply_to":"9f560f44_5b556af8","updated":"2020-10-09 17:37:35.000000000","message":"I don\u0027t think so. I personally believe we shouldn\u0027t over engineer it by having fall back methods.","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"546f9771b4acc3e3d058b2875ae6fd67175252c3","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":42,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":43,"context_line":"        os.makedirs(directory_path)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def get_object_string(directory, object_name):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_9b3a62c1","line":43,"updated":"2020-10-09 11:40:00.000000000","message":"would there be any reasons for a \"permission denied\"?","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"9e4de19329137a33bdbe22e841730ac6500d67ea","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":42,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":43,"context_line":"        os.makedirs(directory_path)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def get_object_string(directory, object_name):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_db6975c0","line":43,"in_reply_to":"9f560f44_9b3a62c1","updated":"2020-10-09 17:37:35.000000000","message":"I don\u0027t think so, the py stack should show what the failure is, if that issue comes up.","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"546f9771b4acc3e3d058b2875ae6fd67175252c3","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    container_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":50,"context_line":"    with open(os.path.join(container_path, object_name)) as f:"},{"line_number":51,"context_line":"        return f.read()"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def put_object_string(directory, object_name, contents):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_3b7356b6","line":51,"updated":"2020-10-09 11:40:00.000000000","message":"what if object doesn\u0027t exist for #reason (race condition or whatever)?","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"9e4de19329137a33bdbe22e841730ac6500d67ea","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    container_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":50,"context_line":"    with open(os.path.join(container_path, object_name)) as f:"},{"line_number":51,"context_line":"        return f.read()"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def put_object_string(directory, object_name, contents):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_3b419137","line":51,"in_reply_to":"9f560f44_3b7356b6","updated":"2020-10-09 17:37:35.000000000","message":"we\u0027d get a stacktrace which indicates the failure.","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"546f9771b4acc3e3d058b2875ae6fd67175252c3","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    container_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":58,"context_line":"    with open(os.path.join(container_path, object_name), \u0027w\u0027) as f:"},{"line_number":59,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_5b6e0ac8","line":59,"updated":"2020-10-09 11:40:00.000000000","message":"might be good ensuring the directory exists before trying to write in it?","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"9e4de19329137a33bdbe22e841730ac6500d67ea","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    container_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":58,"context_line":"    with open(os.path.join(container_path, object_name), \u0027w\u0027) as f:"},{"line_number":59,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_fb3a79c1","line":59,"in_reply_to":"9f560f44_5b6e0ac8","updated":"2020-10-09 17:37:35.000000000","message":"Done","commit_id":"71266ebdba878a121808f4fb8b9a4a0186f7f5e5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"02dd2ab8f73187970a999e43d505e2e68420c28b","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        create_directory(directory\u003dcontainer_path)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    file_path \u003d os.path.join(container_path, object_name)"},{"line_number":64,"context_line":"    create_directory(os.path.dirname(file_path))"},{"line_number":65,"context_line":"    with open(file_path, \u0027w\u0027) as f:"},{"line_number":66,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7f6b1bfe_5c39953f","line":64,"updated":"2020-10-13 14:21:05.000000000","message":"isn\u0027t it duplicated with L60-61?","commit_id":"d7d9ec7dd559f91bfb5893b35bab349719ba1d04"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"1e40318ce7ab51322d554f214d8907b7e82418e5","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        create_directory(directory\u003dcontainer_path)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    file_path \u003d os.path.join(container_path, object_name)"},{"line_number":64,"context_line":"    create_directory(os.path.dirname(file_path))"},{"line_number":65,"context_line":"    with open(file_path, \u0027w\u0027) as f:"},{"line_number":66,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7f6b1bfe_5cd515b9","line":64,"in_reply_to":"7f6b1bfe_5c39953f","updated":"2020-10-13 14:25:48.000000000","message":"in some cases yes, this will result in duplicate code, however, the way that the file is processed from the old swift assumptions, the \"object_name\" may contain an additional path.","commit_id":"d7d9ec7dd559f91bfb5893b35bab349719ba1d04"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0c02f1f1ff27c88e41a485c9b0589289efc5d568","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        create_directory(directory\u003dcontainer_path)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    file_path \u003d os.path.join(container_path, object_name)"},{"line_number":64,"context_line":"    create_directory(os.path.dirname(file_path))"},{"line_number":65,"context_line":"    with open(file_path, \u0027w\u0027) as f:"},{"line_number":66,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7f6b1bfe_5f0d0736","line":64,"in_reply_to":"7f6b1bfe_5cd515b9","updated":"2020-10-13 14:37:52.000000000","message":"oh. damn. ok. better on the safe side then!","commit_id":"d7d9ec7dd559f91bfb5893b35bab349719ba1d04"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"b9c3f4f91de571f19506581180737eb9ea3b5b4c","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        create_directory(directory\u003dcontainer_path)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    file_path \u003d os.path.join(container_path, object_name)"},{"line_number":64,"context_line":"    create_directory(os.path.dirname(file_path))"},{"line_number":65,"context_line":"    with open(file_path, \u0027w\u0027) as f:"},{"line_number":66,"context_line":"        return f.write(contents)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7f6b1bfe_5fba27de","line":64,"in_reply_to":"7f6b1bfe_5f0d0736","updated":"2020-10-13 14:54:37.000000000","message":"++ its super redundant, but something we can cleanup once swift is no longer present.","commit_id":"d7d9ec7dd559f91bfb5893b35bab349719ba1d04"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"80b26523d7cdc2f582338223f6904925a563f317","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"TRIPLEO_STORE \u003d \u0027/var/lib/tripleo/stacks\u0027"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def delete_directory(directory):"}],"source_content_type":"text/x-python","patch_set":18,"id":"46544533_bf7b6347","line":23,"updated":"2021-01-29 11:51:11.000000000","message":"adding this new path for keeping persistent state probably also adds impact to the Uc backup and restore ansible role","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"e04b7de400ee6119935065a21ca0a54c9a4d46a8","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"TRIPLEO_STORE \u003d \u0027/var/lib/tripleo/stacks\u0027"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def delete_directory(directory):"}],"source_content_type":"text/x-python","patch_set":18,"id":"0c9b6ca3_ed89bdd9","line":23,"in_reply_to":"46544533_bf7b6347","updated":"2021-01-29 15:18:31.000000000","message":"the path /var/lib/tripleo already exists, which I believe our backup tools is aware of; however, I will confirm.","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"c506bf3745a5cc77b3e8eeb6d1070a1e428ad1ae","unresolved":true,"context_lines":[{"line_number":41,"context_line":"    \"\"\"Create a directory.\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":44,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":45,"context_line":"        os.makedirs(directory_path)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"712fed13_68ea218b","line":44,"range":{"start_line":44,"start_character":2,"end_line":44,"end_character":42},"updated":"2021-01-29 11:54:07.000000000","message":"no need to check this, there is exist_ok for os.makedirs()","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"04c4df62423303ab1e3d9d10b5e41bea783bdde2","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    \"\"\"Create a directory.\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":44,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":45,"context_line":"        os.makedirs(directory_path)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"b6cdac8b_5f6a540d","line":44,"range":{"start_line":44,"start_character":2,"end_line":44,"end_character":42},"in_reply_to":"3c7e9cf3_8c523f48","updated":"2021-01-29 18:53:58.000000000","message":"I missed this commit, fixing now.","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"8b26cb7064ba8185c213462de8fb227998d30819","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    \"\"\"Create a directory.\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":44,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":45,"context_line":"        os.makedirs(directory_path)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"3c7e9cf3_8c523f48","line":44,"range":{"start_line":44,"start_character":2,"end_line":44,"end_character":42},"in_reply_to":"6f28f88e_27666141","updated":"2021-01-29 16:34:05.000000000","message":"please note that exist_ok\u003dFalse by default, we should have it as True","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"e04b7de400ee6119935065a21ca0a54c9a4d46a8","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    \"\"\"Create a directory.\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    directory_path \u003d os.path.join(TRIPLEO_STORE, directory)"},{"line_number":44,"context_line":"    if not os.path.exists(directory_path):"},{"line_number":45,"context_line":"        os.makedirs(directory_path)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"6f28f88e_27666141","line":44,"range":{"start_line":44,"start_character":2,"end_line":44,"end_character":42},"in_reply_to":"712fed13_68ea218b","updated":"2021-01-29 15:18:31.000000000","message":"Done","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"}],"tripleo_common/utils/plan.py":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"b3dcef9239865b5f43b27b85f24477b21c677a16","unresolved":true,"context_lines":[{"line_number":112,"context_line":"            method, url, headers\u003d{\u0027X-Auth-Token\u0027: token})"},{"line_number":113,"context_line":"        response.raise_for_status()"},{"line_number":114,"context_line":"        return response.content"},{"line_number":115,"context_line":"    except Exception:"},{"line_number":116,"context_line":"        with open(url) as f:"},{"line_number":117,"context_line":"            return f.read()"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"def process_environments_and_files(swift, env_paths):"}],"source_content_type":"text/x-python","patch_set":18,"id":"fff8bbbf_e629d802","line":117,"range":{"start_line":115,"start_character":4,"end_line":117,"end_character":27},"updated":"2021-01-29 11:58:46.000000000","message":"please use urlparse to ifentify the local file case (see https://opendev.org/openstack/tripleo-common/src/branch/master/tripleo_common/image/image_uploader.py#L1988-L1991) and omitting unnecessary response.raise_for_status() call?","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"13394059fc9631426009ac06b814808c6c420807","unresolved":true,"context_lines":[{"line_number":112,"context_line":"            method, url, headers\u003d{\u0027X-Auth-Token\u0027: token})"},{"line_number":113,"context_line":"        response.raise_for_status()"},{"line_number":114,"context_line":"        return response.content"},{"line_number":115,"context_line":"    except Exception:"},{"line_number":116,"context_line":"        with open(url) as f:"},{"line_number":117,"context_line":"            return f.read()"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"def process_environments_and_files(swift, env_paths):"}],"source_content_type":"text/x-python","patch_set":18,"id":"1c928264_0ab651bf","line":117,"range":{"start_line":115,"start_character":4,"end_line":117,"end_character":27},"in_reply_to":"fff8bbbf_e629d802","updated":"2021-01-29 15:33:40.000000000","message":"I don\u0027t know if there\u0027s a scheme here we can identify and scope, like we are in the example, however, I\u0027ll give it a look.","commit_id":"432ad42b0fb9abd5dcce9b55e0f053afe0f9717c"}],"tripleo_common/utils/swift.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":15,"context_line":"# limitations under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import dateutil.parser"},{"line_number":18,"context_line":"import logging"},{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ecead39d","line":17,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027dateutil.parser\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":15,"context_line":"# limitations under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import dateutil.parser"},{"line_number":18,"context_line":"import logging"},{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_2c17aba5","line":17,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027dateutil.parser\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import logging"},{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_4c293ff0","line":21,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027tempfile\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import logging"},{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_8c087780","line":21,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027tempfile\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_2c2e8bf9","line":22,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027uuid\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import os"},{"line_number":20,"context_line":"import shutil"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_6c0d238e","line":22,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027uuid\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_8c2fd7f3","line":24,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027six\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_cc254f0c","line":24,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027six\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_6c34034a","line":25,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027six.moves.urllib\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":22,"context_line":"import uuid"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ac229bff","line":25,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027six.moves.urllib\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_cc3c2f32","line":26,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027swiftclient.exceptions as swiftexceptions\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_0c1c67c0","line":26,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027swiftclient.exceptions as swiftexceptions\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ac39fb3f","line":27,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027swiftclient.service.SwiftError\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":24,"context_line":"import six"},{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ec1813ce","line":27,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027swiftclient.service.SwiftError\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_0c43c7b1","line":28,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027swiftclient.service.SwiftUploadObject\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from six.moves import urllib"},{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_4c777f09","line":28,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027swiftclient.service.SwiftUploadObject\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ec3f731f","line":29,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027swiftclient.utils.generate_temp_url\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"from swiftclient import exceptions as swiftexceptions"},{"line_number":27,"context_line":"from swiftclient.service import SwiftError"},{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_2c7ccbe9","line":29,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027swiftclient.utils.generate_temp_url\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_4c12dfb5","line":31,"updated":"2020-09-23 02:59:21.000000000","message":"linters: F401 \u0027tripleo_common.utils.tarball\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8e6843615e0e72d9023b30e07ab006e4db48240d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from swiftclient.service import SwiftUploadObject"},{"line_number":29,"context_line":"from swiftclient.utils import generate_temp_url"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from tripleo_common.utils import tarball"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_8c7d17e8","line":31,"updated":"2020-09-23 02:59:21.000000000","message":"pep8: F401 \u0027tripleo_common.utils.tarball\u0027 imported but unused","commit_id":"c71f666498c56b203bc2952077a20cbda9663682"}]}