)]}'
{"cinder/api/common.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9033678346b2420927ba23a0e1f32868ab41d1d6","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    Given: \u0027http://cinder.example.com/v1.1\u0027"},{"line_number":196,"context_line":"    Returns: \u0027http://cinder.example.com\u0027"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    Given: \u0027http://cinder.example.com/volume/drivers/v1.1/flashsystem\u0027"},{"line_number":199,"context_line":"    Returns: \u0027http://cinder.example.com/volume/drivers/flashsystem\u0027"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    \"\"\""},{"line_number":202,"context_line":"    parsed_url \u003d urllib.parse.urlsplit(href)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_8b5d9316","line":199,"range":{"start_line":198,"start_character":4,"end_line":199,"end_character":67},"updated":"2019-02-12 18:57:27.000000000","message":"This case was never true?","commit_id":"ffc2d3263007eb747494201c4fdb9af6d3c3435e"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"c61aa22756f35d85bcac32f2797ef58c13094b07","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    Given: \u0027http://cinder.example.com/v1.1\u0027"},{"line_number":196,"context_line":"    Returns: \u0027http://cinder.example.com\u0027"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    Given: \u0027http://cinder.example.com/volume/drivers/v1.1/flashsystem\u0027"},{"line_number":199,"context_line":"    Returns: \u0027http://cinder.example.com/volume/drivers/flashsystem\u0027"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    \"\"\""},{"line_number":202,"context_line":"    parsed_url \u003d urllib.parse.urlsplit(href)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_ddb27c94","line":199,"range":{"start_line":198,"start_character":4,"end_line":199,"end_character":67},"in_reply_to":"9fdfeff1_46d9e254","updated":"2019-02-12 21:45:28.000000000","message":"Exactly yes, the only piece that the commit was missing to have this working was this one that I included now... plus some tests.","commit_id":"ffc2d3263007eb747494201c4fdb9af6d3c3435e"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"4a65e7c846cb963984bbef6d3d4f451acc341fea","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    Given: \u0027http://cinder.example.com/v1.1\u0027"},{"line_number":196,"context_line":"    Returns: \u0027http://cinder.example.com\u0027"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    Given: \u0027http://cinder.example.com/volume/drivers/v1.1/flashsystem\u0027"},{"line_number":199,"context_line":"    Returns: \u0027http://cinder.example.com/volume/drivers/flashsystem\u0027"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    \"\"\""},{"line_number":202,"context_line":"    parsed_url \u003d urllib.parse.urlsplit(href)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_8bdef3d7","line":199,"range":{"start_line":198,"start_character":4,"end_line":199,"end_character":67},"in_reply_to":"9fdfeff1_8b5d9316","updated":"2019-02-12 19:29:20.000000000","message":"\u003e\u003e\u003e from six.moves import urllib\n\u003e\u003e\u003e import re\n\u003e\u003e\u003e \n\u003e\u003e\u003e href \u003d \"http://cinder.example.com/volume/drivers/v1.1/flashsystem\"\n\u003e\u003e\u003e parsed_url \u003d urllib.parse.urlsplit(href)\n\u003e\u003e\u003e parsed_url\nSplitResult(scheme\u003d\u0027http\u0027, netloc\u003d\u0027cinder.example.com\u0027, path\u003d\u0027/volume/drivers/v1.1/flashsystem\u0027, query\u003d\u0027\u0027, fragment\u003d\u0027\u0027)\n\u003e\u003e\u003e url_parts \u003d parsed_url.path.split(\u0027/\u0027, 2)\n\u003e\u003e\u003e url_parts\n[\u0027\u0027, \u0027volume\u0027, \u0027drivers/v1.1/flashsystem\u0027]\n\u003e\u003e\u003e expression \u003d re.compile(r\u0027^v([0-9]+|[0-9]+\\.[0-9]+)(/.*|$)\u0027)\n\u003e\u003e\u003e expression.match(url_parts[2])\n\u003e\u003e\u003e expression.match(url_parts[1])\n\u003e\u003e\u003e expression.match(url_parts[0])","commit_id":"ffc2d3263007eb747494201c4fdb9af6d3c3435e"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8215776d60dff4c8be883cf59c1a44e87d152c44","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    Given: \u0027http://cinder.example.com/v1.1\u0027"},{"line_number":196,"context_line":"    Returns: \u0027http://cinder.example.com\u0027"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    Given: \u0027http://cinder.example.com/volume/drivers/v1.1/flashsystem\u0027"},{"line_number":199,"context_line":"    Returns: \u0027http://cinder.example.com/volume/drivers/flashsystem\u0027"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    \"\"\""},{"line_number":202,"context_line":"    parsed_url \u003d urllib.parse.urlsplit(href)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_46d9e254","line":199,"range":{"start_line":198,"start_character":4,"end_line":199,"end_character":67},"in_reply_to":"9fdfeff1_8bdef3d7","updated":"2019-02-12 20:01:01.000000000","message":"Yes, that\u0027s what I mean :) This wasn\u0027t a valid scenario even before your change.. The commit that introduced this [1] added this to the docstring.. but it wasn\u0027t working as intended even then... \n\n[1] https://github.com/openstack/cinder/commit/41981d4803c9cced59a62376c7ffd6ea43f35547","commit_id":"ffc2d3263007eb747494201c4fdb9af6d3c3435e"}]}