)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"ce786e2389ab03715f437e932c9a1c3cb01a2ec9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"When listing images there are several optional parameters that"},{"line_number":10,"context_line":"can be used to filter the list of images retrieved by the API. The"},{"line_number":11,"context_line":"following two parameters are not working properly: the created_at and"},{"line_number":12,"context_line":"the updated_at. Before the Mitaka release it was possible to use these"},{"line_number":13,"context_line":"filters just using a datetime in the format ISO 8601, starting on"},{"line_number":14,"context_line":"Mitaka, you need to add an operator along with the datetime stamp or"},{"line_number":15,"context_line":"the API call fails. This commit adds backwards compatibility so it is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_f4442275","line":12,"updated":"2016-06-06 21:55:23.000000000","message":"Listing images by image name is also broken (pl check the bug update). We need to fix that as well in a subsequent patch.","commit_id":"f85b514ae567bef650dac072f70833c9b4f5e205"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"4283ea4795435014796620ee2806c7dfc4c75b60","unresolved":false,"context_lines":[{"line_number":9,"context_line":"When listing images there are several optional parameters that"},{"line_number":10,"context_line":"can be used to filter the list of images retrieved by the API. The"},{"line_number":11,"context_line":"following two parameters are not working properly: the created_at and"},{"line_number":12,"context_line":"the updated_at. Before the Mitaka release it was possible to use these"},{"line_number":13,"context_line":"filters just using a datetime in the format ISO 8601, starting on"},{"line_number":14,"context_line":"Mitaka, you need to add an operator along with the datetime stamp or"},{"line_number":15,"context_line":"the API call fails. This commit adds backwards compatibility so it is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_db25ceae","line":12,"in_reply_to":"7aa08908_f4442275","updated":"2016-06-10 16:05:07.000000000","message":"I didn\u0027t know image names could contain the \":\" character. Good catch.","commit_id":"f85b514ae567bef650dac072f70833c9b4f5e205"},{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"ce786e2389ab03715f437e932c9a1c3cb01a2ec9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"assumed."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Id5d5455e77637e0dc7baec25c8163b21634d72c4"},{"line_number":21,"context_line":"Closes-Bug: 1584415"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_b463ba19","line":21,"updated":"2016-06-06 21:55:23.000000000","message":"So, can this be Partial-Bug: ?","commit_id":"f85b514ae567bef650dac072f70833c9b4f5e205"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"4283ea4795435014796620ee2806c7dfc4c75b60","unresolved":false,"context_lines":[{"line_number":18,"context_line":"assumed."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Id5d5455e77637e0dc7baec25c8163b21634d72c4"},{"line_number":21,"context_line":"Closes-Bug: 1584415"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_69fcd33e","line":21,"in_reply_to":"7aa08908_b463ba19","updated":"2016-06-10 16:05:07.000000000","message":"Done","commit_id":"f85b514ae567bef650dac072f70833c9b4f5e205"}],"glance/common/utils.py":[{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"ce786e2389ab03715f437e932c9a1c3cb01a2ec9","unresolved":false,"context_lines":[{"line_number":601,"context_line":""},{"line_number":602,"context_line":"    :returns: a tuple (operator, threshold) parsed from expression"},{"line_number":603,"context_line":"    \"\"\""},{"line_number":604,"context_line":"    left, sep, right \u003d expression.partition(\u0027:\u0027)"},{"line_number":605,"context_line":"    if sep:"},{"line_number":606,"context_line":"        # If the expression is a date of the format ISO 8601 like"},{"line_number":607,"context_line":"        # CCYY-MM-DDThh:mm:ss+hh:mm and has no operator, it should"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_f7ca6719","line":604,"updated":"2016-06-06 21:55:23.000000000","message":"I feel this naive partition should be replaced with something like \n\n parts \u003d re.compile(\u0027^(gt|gte|lt|lte|neq|eq):\u0027).split(expression)\n\nThen check the parts to get the operator \u0026 threshold.\n\nThis way when any filter property key contains a \u0027:\u0027, we parse appropriately. I am fine with doing this change in a subsequent patch.","commit_id":"f85b514ae567bef650dac072f70833c9b4f5e205"}]}