)]}'
{"nova/api/openstack/placement/microversion.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d2185f79a939e3844d8cc79491b5118d5529a02a","unresolved":false,"context_lines":[{"line_number":225,"context_line":"    # just in case."},{"line_number":226,"context_line":"    method_list \u003d VERSIONED_METHODS.get(qualified_name, [])"},{"line_number":227,"context_line":"    for min_version, max_version, func in method_list:"},{"line_number":228,"context_line":"        if min_version \u003c\u003d version \u003c\u003d max_version:"},{"line_number":229,"context_line":"            return func"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    raise webob.exc.HTTPNotFound()"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a1ced50_30b39399","line":228,"updated":"2017-03-23 13:28:59.000000000","message":"IIUC string comparison still does the wrong thing here, because \u00271.10\u0027 \u003c \u00271.2\u0027.\n\nOr is this using a different type that I cannot see?","commit_id":"b2b366aa4aa2b5199ddbd8cd51019f3d8b7bccc3"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"6d9b6384f628e9627a6ec42ffa375efd8e4b3af9","unresolved":false,"context_lines":[{"line_number":225,"context_line":"    # just in case."},{"line_number":226,"context_line":"    method_list \u003d VERSIONED_METHODS.get(qualified_name, [])"},{"line_number":227,"context_line":"    for min_version, max_version, func in method_list:"},{"line_number":228,"context_line":"        if min_version \u003c\u003d version \u003c\u003d max_version:"},{"line_number":229,"context_line":"            return func"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    raise webob.exc.HTTPNotFound()"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a1ced50_b384ed5d","line":228,"in_reply_to":"1a1ced50_30b39399","updated":"2017-03-23 13:33:35.000000000","message":"This is a tuple here, or more specifically the Version named tuple class define in this file.","commit_id":"b2b366aa4aa2b5199ddbd8cd51019f3d8b7bccc3"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"936d047e61b6d87192b28aad19c2f0bc9852e742","unresolved":false,"context_lines":[{"line_number":225,"context_line":"    # just in case."},{"line_number":226,"context_line":"    method_list \u003d VERSIONED_METHODS.get(qualified_name, [])"},{"line_number":227,"context_line":"    for min_version, max_version, func in method_list:"},{"line_number":228,"context_line":"        if min_version \u003c\u003d version \u003c\u003d max_version:"},{"line_number":229,"context_line":"            return func"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    raise webob.exc.HTTPNotFound()"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a1ced50_f3a3c5fd","line":228,"in_reply_to":"1a1ced50_30b39399","updated":"2017-03-23 13:33:18.000000000","message":"You\u0027re missing L250 and below that transform the string into a tuple like (1, 1)","commit_id":"b2b366aa4aa2b5199ddbd8cd51019f3d8b7bccc3"}]}
