)]}'
{"ansible/roles/swift-rings/files/swift-ring-builder.py":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"9d347fb50ff2b91372f85e1207924cc16c0761ea","unresolved":true,"context_lines":[{"line_number":120,"context_line":"    build_path \u003d sys.argv[2]"},{"line_number":121,"context_line":"    service_name \u003d sys.argv[3]"},{"line_number":122,"context_line":"    with open(config_path) as f:"},{"line_number":123,"context_line":"        config \u003d yaml.safe_load(f)"},{"line_number":124,"context_line":"    build_rings(config, build_path, service_name)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baabe58a_e209e884","line":123,"updated":"2020-12-18 12:47:03.000000000","message":"Seems sensible, but is this required for the change to work? Seems like a potential security fix. Anyway, probably not something to get hung up over as I guess this change will be backported anyway.","commit_id":"c847ef91572afa87a6d76b265c88db2d9e50ccb8"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6881eb6e90a092fbbd78ae5b6bcc3982b65ee77d","unresolved":true,"context_lines":[{"line_number":120,"context_line":"    build_path \u003d sys.argv[2]"},{"line_number":121,"context_line":"    service_name \u003d sys.argv[3]"},{"line_number":122,"context_line":"    with open(config_path) as f:"},{"line_number":123,"context_line":"        config \u003d yaml.safe_load(f)"},{"line_number":124,"context_line":"    build_rings(config, build_path, service_name)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"239880db_70cd0382","line":123,"in_reply_to":"baabe58a_e209e884","updated":"2020-12-21 10:12:50.000000000","message":"You are right in that it\u0027s a separate issue. I just noticed as I was debugging. Like you say, we\u0027ll backport this, so no need to add extra work.","commit_id":"c847ef91572afa87a6d76b265c88db2d9e50ccb8"}]}