)]}'
{"swiftclient/service.py":[{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"9d5198f35d31f09ee7892f40c0df60e24de639fa","unresolved":false,"context_lines":[{"line_number":1454,"context_line":"        # to HEAD the first container"},{"line_number":1455,"context_line":"        for r in interruptable_as_completed(create_containers):"},{"line_number":1456,"context_line":"            res \u003d r.result()"},{"line_number":1457,"context_line":"            # Explicitly ignore failure to create container because of missing permission"},{"line_number":1458,"context_line":"            if \u0027error\u0027 in res and res[\u0027error\u0027].http_status \u003d\u003d 405:"},{"line_number":1459,"context_line":"                continue"},{"line_number":1460,"context_line":"            yield res"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_1f592628","line":1457,"updated":"2018-07-25 08:58:08.000000000","message":"pep8 failing due to -line too long (89 \u003e 79 characters)","commit_id":"33d3e664efba862f0f2a46034a1fafa64f9fba45"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d72ad08f7bca0810cf8d207dab9c54462e2bfa45","unresolved":false,"context_lines":[{"line_number":1456,"context_line":"            res \u003d r.result()"},{"line_number":1457,"context_line":"            # Explicitly ignore failure to create container because of"},{"line_number":1458,"context_line":"            # missing permission"},{"line_number":1459,"context_line":"            if \u0027error\u0027 in res and res[\u0027error\u0027].http_status \u003d\u003d 405:"},{"line_number":1460,"context_line":"                continue"},{"line_number":1461,"context_line":"            yield res"},{"line_number":1462,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5f7c97a3_c46ad6b3","line":1459,"range":{"start_line":1459,"start_character":62,"end_line":1459,"end_character":65},"updated":"2018-07-26 18:57:41.000000000","message":"We should be looking for a 403 here, yeah? Maybe even a 409, in case we hit a conflict on X-Storage-Policy?","commit_id":"29eaeb475ff47ef746f9c707272b4b835549a41a"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"de4658e58b70f6d21c78c802406ededd2325e648","unresolved":false,"context_lines":[{"line_number":1456,"context_line":"            res \u003d r.result()"},{"line_number":1457,"context_line":"            # Explicitly ignore failure to create container because of"},{"line_number":1458,"context_line":"            # missing permission"},{"line_number":1459,"context_line":"            if \u0027error\u0027 in res and res[\u0027error\u0027].http_status \u003d\u003d 405:"},{"line_number":1460,"context_line":"                continue"},{"line_number":1461,"context_line":"            yield res"},{"line_number":1462,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5f7c97a3_2482aa52","line":1459,"updated":"2018-07-26 18:58:24.000000000","message":"according to the bug report [1] this should be 403?  Permission?\n\nWhich makes sense to me.\n\n1. https://bugs.launchpad.net/python-swiftclient/+bug/1204558","commit_id":"29eaeb475ff47ef746f9c707272b4b835549a41a"},{"author":{"_account_id":28746,"name":"Alexander Bolshakov","email":"lextiz@gmail.com","username":"lextiz"},"change_message_id":"eb9d338ceb08dffd26b13d58332cd7807d8cbb1e","unresolved":false,"context_lines":[{"line_number":1456,"context_line":"            res \u003d r.result()"},{"line_number":1457,"context_line":"            # Explicitly ignore failure to create container because of"},{"line_number":1458,"context_line":"            # missing permission"},{"line_number":1459,"context_line":"            if \u0027error\u0027 in res and res[\u0027error\u0027].http_status \u003d\u003d 405:"},{"line_number":1460,"context_line":"                continue"},{"line_number":1461,"context_line":"            yield res"},{"line_number":1462,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5f7c97a3_ac240e0f","line":1459,"in_reply_to":"5f7c97a3_2482aa52","updated":"2018-07-31 15:05:53.000000000","message":"Yes 403 should be there as well. I will add it. 405 is a valid use case as well \"Method not allowed\" might be result of missing permission.","commit_id":"29eaeb475ff47ef746f9c707272b4b835549a41a"}]}