)]}'
{"nova/image/glance.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"fda54448962133c14a7ec5e568f419b970b4a1fd","unresolved":false,"context_lines":[{"line_number":192,"context_line":"    for api_server in configured_servers:"},{"line_number":193,"context_line":"        if \u0027//\u0027 not in api_server:"},{"line_number":194,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"},{"line_number":195,"context_line":"            # NOTE(sdague): remove in N."},{"line_number":196,"context_line":"            LOG.warn(_LW(\"No protocol specified in %s, \""},{"line_number":197,"context_line":"                         \"this will be required in the future\"), api_server)"},{"line_number":198,"context_line":"        o \u003d urlparse.urlparse(api_server)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_be4333ae","line":195,"updated":"2015-12-08 17:32:07.000000000","message":"# s/NOTE/TODO\n\nDon\u0027t you have to add a Reno note upgrade for something like that?","commit_id":"3e5daba291e05a32a12a61a09034db63186c29f7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a426b69f78973c7e49e365889c7b3895544093fd","unresolved":false,"context_lines":[{"line_number":192,"context_line":"    for api_server in configured_servers:"},{"line_number":193,"context_line":"        if \u0027//\u0027 not in api_server:"},{"line_number":194,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"},{"line_number":195,"context_line":"            # NOTE(sdague): remove in N."},{"line_number":196,"context_line":"            LOG.warn(_LW(\"No protocol specified in %s, \""},{"line_number":197,"context_line":"                         \"this will be required in the future\"), api_server)"},{"line_number":198,"context_line":"        o \u003d urlparse.urlparse(api_server)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_347e746a","line":195,"in_reply_to":"7a740942_be4333ae","updated":"2015-12-08 22:59:22.000000000","message":"yeh, good point","commit_id":"3e5daba291e05a32a12a61a09034db63186c29f7"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"fda54448962133c14a7ec5e568f419b970b4a1fd","unresolved":false,"context_lines":[{"line_number":194,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"},{"line_number":195,"context_line":"            # NOTE(sdague): remove in N."},{"line_number":196,"context_line":"            LOG.warn(_LW(\"No protocol specified in %s, \""},{"line_number":197,"context_line":"                         \"this will be required in the future\"), api_server)"},{"line_number":198,"context_line":"        o \u003d urlparse.urlparse(api_server)"},{"line_number":199,"context_line":"        port \u003d o.port or 80"},{"line_number":200,"context_line":"        host \u003d o.netloc.rsplit(\u0027:\u0027, 1)[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_19086127","line":197,"updated":"2015-12-08 17:32:07.000000000","message":"So this log message will be formatted with \u0027api_server\u0027 enhanced by protocol. I\u0027m not sure it will be clear for operator to understand it.","commit_id":"3e5daba291e05a32a12a61a09034db63186c29f7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a426b69f78973c7e49e365889c7b3895544093fd","unresolved":false,"context_lines":[{"line_number":194,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"},{"line_number":195,"context_line":"            # NOTE(sdague): remove in N."},{"line_number":196,"context_line":"            LOG.warn(_LW(\"No protocol specified in %s, \""},{"line_number":197,"context_line":"                         \"this will be required in the future\"), api_server)"},{"line_number":198,"context_line":"        o \u003d urlparse.urlparse(api_server)"},{"line_number":199,"context_line":"        port \u003d o.port or 80"},{"line_number":200,"context_line":"        host \u003d o.netloc.rsplit(\u0027:\u0027, 1)[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_d49930c5","line":197,"in_reply_to":"7a740942_19086127","updated":"2015-12-08 22:59:22.000000000","message":"good catch, fixed","commit_id":"3e5daba291e05a32a12a61a09034db63186c29f7"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5e8d1daeb1d1adb913b51c08c08fcb77384049b5","unresolved":false,"context_lines":[{"line_number":190,"context_line":"            # NOTE(sdague): remove in N."},{"line_number":191,"context_line":"            LOG.warn("},{"line_number":192,"context_line":"                _LW(\"No protocol specified in for api_server \u0027%s\u0027, \""},{"line_number":193,"context_line":"                    \"pleased update [glance] api_servers with fully \""},{"line_number":194,"context_line":"                    \"qualified url including protocol (http / https)\"),"},{"line_number":195,"context_line":"                api_server)"},{"line_number":196,"context_line":"        o \u003d urlparse.urlparse(api_server)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_3185a3ac","line":193,"range":{"start_line":193,"start_character":27,"end_line":193,"end_character":28},"updated":"2015-12-10 22:40:31.000000000","message":"Should be \"please\"","commit_id":"55ba2e6e1725aa57847fce022a8c4ae5d3f70e16"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5e8d1daeb1d1adb913b51c08c08fcb77384049b5","unresolved":false,"context_lines":[{"line_number":191,"context_line":"            LOG.warn("},{"line_number":192,"context_line":"                _LW(\"No protocol specified in for api_server \u0027%s\u0027, \""},{"line_number":193,"context_line":"                    \"pleased update [glance] api_servers with fully \""},{"line_number":194,"context_line":"                    \"qualified url including protocol (http / https)\"),"},{"line_number":195,"context_line":"                api_server)"},{"line_number":196,"context_line":"        o \u003d urlparse.urlparse(api_server)"},{"line_number":197,"context_line":"        port \u003d o.port or 80"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_31acc334","line":194,"range":{"start_line":194,"start_character":55,"end_line":194,"end_character":67},"updated":"2015-12-10 22:40:31.000000000","message":"This is the \"scheme\" part of the URL, but what you have is fine if you prefer.","commit_id":"55ba2e6e1725aa57847fce022a8c4ae5d3f70e16"}]}