)]}'
{"tripleo_common/image/image_uploader.py":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"0604768e9949598998c9c16b2b56ef32442a2595","unresolved":false,"context_lines":[{"line_number":400,"context_line":"        if hasattr(session, \u0027reauthenticate\u0027):"},{"line_number":401,"context_line":"            if request.status_code \u003d\u003d 401:"},{"line_number":402,"context_line":"                session.reauthenticate(**session.auth_args)"},{"line_number":403,"context_line":"                if hasattr(request, \u0027text\u0027):"},{"line_number":404,"context_line":"                    raise requests.exceptions.HTTPError(request.text)"},{"line_number":405,"context_line":"                else:"},{"line_number":406,"context_line":"                    raise SystemError()"},{"line_number":407,"context_line":"        request.raise_for_status()"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"7faddb67_c0263869","side":"PARENT","line":406,"range":{"start_line":403,"start_character":0,"end_line":406,"end_character":39},"updated":"2019-08-13 20:26:05.000000000","message":"So this logic triggered the larger retry within this process. It appears this has introduced some other issue where the reauth process continues to fail with 401s.  I think we have another issue.","commit_id":"6f014aa21bca37921db07e97b9006889cb9ecfee"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"8886246119466e46955f965cd48585dbec721216","unresolved":false,"context_lines":[{"line_number":497,"context_line":"                error \u003d None"},{"line_number":498,"context_line":"                if \u0027error\u003d\u0027 in www_auth:"},{"line_number":499,"context_line":"                    error \u003d re.search(\u0027error\u003d\"(.*?)\"\u0027, www_auth).group(1)"},{"line_number":500,"context_line":"                if error \u003d\u003d \u0027invalid_token\u0027:"},{"line_number":501,"context_line":"                    if hasattr(session, \u0027reauthenticate\u0027):"},{"line_number":502,"context_line":"                        reauth \u003d int(session.headers.get(\u0027_TripleOReAuth\u0027, 0))"},{"line_number":503,"context_line":"                        reauth +\u003d 1"}],"source_content_type":"text/x-python","patch_set":24,"id":"7faddb67_96fa6372","line":500,"range":{"start_line":500,"start_character":16,"end_line":500,"end_character":44},"updated":"2019-08-20 17:01:41.000000000","message":"Another kind of error that can be caught here is \"insufficient_scope\":\n\n    www-authenticate Bearer realm\u003d\"https://auth.docker.io/token\",service\u003d\"registry.docker.io\",scope\u003d\"repository:v2/tripleomaster/centos-binary-nova-compute:pull\",error\u003d\"insufficient_scope\"","commit_id":"b2e6db04e58af68b20e4630ac496995d89c84281"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"439f3473d9c98b85772461ad924a455918ea27ac","unresolved":false,"context_lines":[{"line_number":1314,"context_line":"        with session.get("},{"line_number":1315,"context_line":"                source_blob_url, stream\u003dTrue, timeout\u003d30) as blob_req:"},{"line_number":1316,"context_line":"            # TODO(aschultz): unsure if necessary or if only when using .text"},{"line_number":1317,"context_line":"            blob_req.encoding \u003d \u0027utf-8\u0027"},{"line_number":1318,"context_line":"            cls.check_status(session\u003dsession, request\u003dblob_req)"},{"line_number":1319,"context_line":"            for data in blob_req.iter_content(chunk_size):"},{"line_number":1320,"context_line":"                if not data:"}],"source_content_type":"text/x-python","patch_set":37,"id":"7faddb67_dcdd2ade","line":1317,"updated":"2019-08-22 19:14:31.000000000","message":"not sure if this necessary but it doesn\u0027t seem to be breaking anything","commit_id":"e06cf4482da855d63ab21f8183cea66499e6f5f9"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"ee1f9d74bdfe4440abd6350a26cc58b9ebb99db9","unresolved":false,"context_lines":[{"line_number":1314,"context_line":"        with session.get("},{"line_number":1315,"context_line":"                source_blob_url, stream\u003dTrue, timeout\u003d30) as blob_req:"},{"line_number":1316,"context_line":"            # TODO(aschultz): unsure if necessary or if only when using .text"},{"line_number":1317,"context_line":"            blob_req.encoding \u003d \u0027utf-8\u0027"},{"line_number":1318,"context_line":"            cls.check_status(session\u003dsession, request\u003dblob_req)"},{"line_number":1319,"context_line":"            for data in blob_req.iter_content(chunk_size):"},{"line_number":1320,"context_line":"                if not data:"}],"source_content_type":"text/x-python","patch_set":37,"id":"7faddb67_972f8b3d","line":1317,"in_reply_to":"7faddb67_dcdd2ade","updated":"2019-08-22 19:46:50.000000000","message":"+1 I can create a follow on review to pull this out. see if there are any negative effects.","commit_id":"e06cf4482da855d63ab21f8183cea66499e6f5f9"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"4ba1793103d2435bf257fc12f29c0673952759cd","unresolved":false,"context_lines":[{"line_number":1419,"context_line":"                )"},{"line_number":1420,"context_line":""},{"line_number":1421,"context_line":"                r \u003d source_session.get(source_config_url, timeout\u003d30)"},{"line_number":1422,"context_line":"                cls.check_status("},{"line_number":1423,"context_line":"                    session\u003dsource_session,"},{"line_number":1424,"context_line":"                    request\u003dr"},{"line_number":1425,"context_line":"                )"},{"line_number":1426,"context_line":"                config_str \u003d cls._get_response_text(r)"},{"line_number":1427,"context_line":"                manifest[\u0027config\u0027][\u0027size\u0027] \u003d len(config_str)"},{"line_number":1428,"context_line":"                manifest[\u0027config\u0027][\u0027mediaType\u0027] \u003d MEDIA_CONFIG"}],"source_content_type":"text/x-python","patch_set":37,"id":"3fa7e38b_7e248360","line":1425,"range":{"start_line":1422,"start_character":0,"end_line":1425,"end_character":17},"updated":"2019-10-11 20:23:30.000000000","message":"because this code didn\u0027t exist in stein, this part didn\u0027t get backported. I noticed this when trying to apply this code snippet back into stein for clean backports","commit_id":"e06cf4482da855d63ab21f8183cea66499e6f5f9"}]}