)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"e9693485a25b0330b67c606a430cacb595746d7a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Listing alarms based on timestamp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The alarms are ordered by \"Alarm Id\". Ceilometer Client should display it ordered by \"Timestamp\"."},{"line_number":10,"context_line":"This patch is introducing \"Timestamp\" as one more attribute on Alarm-Listing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Fixes Bug #1337838"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_0450be30","line":9,"updated":"2014-07-14 11:43:19.000000000","message":"please fit 80 symbols margin here as in code too :)","commit_id":"2c142c5f3a29ff5ea73712f62fc823874c2e9c62"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"95fd92d2a55aa5909872cc34950b5a45b700b4c8","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The alarms are ordered by \"Alarm Id\". Ceilometer Client should display it ordered by \"Timestamp\"."},{"line_number":10,"context_line":"This patch is introducing \"Timestamp\" as one more attribute on Alarm-Listing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Fixes Bug #1337838"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Icfeba852964c29114da84497aac37afee07dc03f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_8fb8e9c8","line":12,"updated":"2014-07-17 02:14:11.000000000","message":"Please use \u0027Closes-Bug: #1337838\u0027, otherwise the gerrit can\u0027t interpret this to create hyperlink to the launchpad bug.","commit_id":"2c142c5f3a29ff5ea73712f62fc823874c2e9c62"}],"ceilometerclient/v2/shell.py":[{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"95fd92d2a55aa5909872cc34950b5a45b700b4c8","unresolved":false,"context_lines":[{"line_number":278,"context_line":"    field_labels \u003d [\u0027Timestamp\u0027, \u0027Alarm ID\u0027, \u0027Name\u0027, \u0027State\u0027, \u0027Enabled\u0027,"},{"line_number":279,"context_line":"                    \u0027Continuous\u0027, \u0027Alarm condition\u0027, \u0027Time constraints\u0027]"},{"line_number":280,"context_line":"    fields \u003d [\u0027timestamp\u0027, \u0027alarm_id\u0027, \u0027name\u0027, \u0027state\u0027, \u0027enabled\u0027,"},{"line_number":281,"context_line":"              \u0027repeat_actions\u0027, \u0027rule\u0027, \u0027time_constraints\u0027]"},{"line_number":282,"context_line":"    utils.print_list("},{"line_number":283,"context_line":"        alarms, fields, field_labels,"},{"line_number":284,"context_line":"        formatters\u003d{\u0027rule\u0027: alarm_rule_formatter,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_7afef106","line":281,"updated":"2014-07-17 02:14:11.000000000","message":"Can we not put the timestamp as the first column here? it\u0027s a little bit strange. We can leverage the sortby parameter of utils.print_list() if we really want to sort by timestamp.\n\nb.t.w, why do we want to sort by timestamp, why can\u0027t we live with by sorting \u0027alarm id\u0027 or \u0027alarm name\u0027?","commit_id":"2c142c5f3a29ff5ea73712f62fc823874c2e9c62"},{"author":{"_account_id":8052,"name":"Nejc Saje","email":"nejc@saje.info","username":"nsaje"},"change_message_id":"965b2ad66bc709fff5745527f855abf28a45ef72","unresolved":false,"context_lines":[{"line_number":278,"context_line":"    field_labels \u003d [\u0027Timestamp\u0027, \u0027Alarm ID\u0027, \u0027Name\u0027, \u0027State\u0027, \u0027Enabled\u0027,"},{"line_number":279,"context_line":"                    \u0027Continuous\u0027, \u0027Alarm condition\u0027, \u0027Time constraints\u0027]"},{"line_number":280,"context_line":"    fields \u003d [\u0027timestamp\u0027, \u0027alarm_id\u0027, \u0027name\u0027, \u0027state\u0027, \u0027enabled\u0027,"},{"line_number":281,"context_line":"              \u0027repeat_actions\u0027, \u0027rule\u0027, \u0027time_constraints\u0027]"},{"line_number":282,"context_line":"    utils.print_list("},{"line_number":283,"context_line":"        alarms, fields, field_labels,"},{"line_number":284,"context_line":"        formatters\u003d{\u0027rule\u0027: alarm_rule_formatter,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_bfacdf8d","line":281,"in_reply_to":"baada198_7afef106","updated":"2014-07-17 15:00:27.000000000","message":"I agree, sorting by name would make much more sense.","commit_id":"2c142c5f3a29ff5ea73712f62fc823874c2e9c62"},{"author":{"_account_id":12256,"name":"Samta Rangare","email":"samtarangare@gmail.com","username":"samta"},"change_message_id":"22c0739f33a8b1a837160a1374d9dcf591786e31","unresolved":false,"context_lines":[{"line_number":278,"context_line":"    field_labels \u003d [\u0027Timestamp\u0027, \u0027Alarm ID\u0027, \u0027Name\u0027, \u0027State\u0027, \u0027Enabled\u0027,"},{"line_number":279,"context_line":"                    \u0027Continuous\u0027, \u0027Alarm condition\u0027, \u0027Time constraints\u0027]"},{"line_number":280,"context_line":"    fields \u003d [\u0027timestamp\u0027, \u0027alarm_id\u0027, \u0027name\u0027, \u0027state\u0027, \u0027enabled\u0027,"},{"line_number":281,"context_line":"              \u0027repeat_actions\u0027, \u0027rule\u0027, \u0027time_constraints\u0027]"},{"line_number":282,"context_line":"    utils.print_list("},{"line_number":283,"context_line":"        alarms, fields, field_labels,"},{"line_number":284,"context_line":"        formatters\u003d{\u0027rule\u0027: alarm_rule_formatter,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_c6df1c37","line":281,"in_reply_to":"baada198_bfacdf8d","updated":"2014-07-23 14:10:03.000000000","message":"Hi, Sorting by \"Alarm Name\" or \"Alarm Id\" doesn’t really tells anything about Alarms. Whereas \"Timestamp\" helps us to find out the newer or older alarm. Please Refer ceilometer bug - Bug #1287119.","commit_id":"2c142c5f3a29ff5ea73712f62fc823874c2e9c62"}]}
