)]}'
{"gnocchi/rest/__init__.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"abd465e18689dc230c483ff833bb5bdb4d0d04e2","unresolved":false,"context_lines":[{"line_number":1376,"context_line":"                        except indexer.NamedMetricAlreadyExists as e:"},{"line_number":1377,"context_line":"                            already_exists_names.append(e.metric)"},{"line_number":1378,"context_line":"                        except indexer.NoSuchResource:"},{"line_number":1379,"context_line":"                            unknown_resources[six.text_type(resource_id)] \u003d ("},{"line_number":1380,"context_line":"                                original_resource_id)"},{"line_number":1381,"context_line":"                        except indexer.IndexerException as e:"},{"line_number":1382,"context_line":"                            # This catch NoSuchArchivePolicy, which is unlikely"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba5201f7_710eb233","line":1379,"updated":"2017-01-10 13:37:47.000000000","message":"I\u0027m not sure about the format, it seems… confusing. Maybe something more like:\n\n[\n {\"original_resource_id\": \"foobar\",\n   \"resource_id\": \"29ffabc121abc212b1caba\",\n },\n...\n]","commit_id":"87e9e695f9f7f03aacc1f0680e1fbf00e12fdeec"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"25b316e3fb2719d0b972f94482f49b731934b3e7","unresolved":false,"context_lines":[{"line_number":1376,"context_line":"                        except indexer.NamedMetricAlreadyExists as e:"},{"line_number":1377,"context_line":"                            already_exists_names.append(e.metric)"},{"line_number":1378,"context_line":"                        except indexer.NoSuchResource:"},{"line_number":1379,"context_line":"                            unknown_resources[six.text_type(resource_id)] \u003d ("},{"line_number":1380,"context_line":"                                original_resource_id)"},{"line_number":1381,"context_line":"                        except indexer.IndexerException as e:"},{"line_number":1382,"context_line":"                            # This catch NoSuchArchivePolicy, which is unlikely"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba5201f7_66323a60","line":1379,"in_reply_to":"ba5201f7_35514bd9","updated":"2017-01-11 15:00:58.000000000","message":"if i\u0027m not mistaken, this code has not been released yet so it\u0027s only ceilometer ocata relying on this, no? we can just fix it before releasing anything right?","commit_id":"87e9e695f9f7f03aacc1f0680e1fbf00e12fdeec"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"dca4e4047a5ff192eec28652697b2d8f5a2218dc","unresolved":false,"context_lines":[{"line_number":1376,"context_line":"                        except indexer.NamedMetricAlreadyExists as e:"},{"line_number":1377,"context_line":"                            already_exists_names.append(e.metric)"},{"line_number":1378,"context_line":"                        except indexer.NoSuchResource:"},{"line_number":1379,"context_line":"                            unknown_resources[six.text_type(resource_id)] \u003d ("},{"line_number":1380,"context_line":"                                original_resource_id)"},{"line_number":1381,"context_line":"                        except indexer.IndexerException as e:"},{"line_number":1382,"context_line":"                            # This catch NoSuchArchivePolicy, which is unlikely"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba5201f7_56bb19e5","line":1379,"in_reply_to":"ba5201f7_66323a60","updated":"2017-01-11 17:53:55.000000000","message":"Indeed, let\u0027s break Ceilometer, I will fix it immediatly after.","commit_id":"87e9e695f9f7f03aacc1f0680e1fbf00e12fdeec"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"0cb19b340900347dc81a5b5a7e1f04e9d7fe3778","unresolved":false,"context_lines":[{"line_number":1376,"context_line":"                        except indexer.NamedMetricAlreadyExists as e:"},{"line_number":1377,"context_line":"                            already_exists_names.append(e.metric)"},{"line_number":1378,"context_line":"                        except indexer.NoSuchResource:"},{"line_number":1379,"context_line":"                            unknown_resources[six.text_type(resource_id)] \u003d ("},{"line_number":1380,"context_line":"                                original_resource_id)"},{"line_number":1381,"context_line":"                        except indexer.IndexerException as e:"},{"line_number":1382,"context_line":"                            # This catch NoSuchArchivePolicy, which is unlikely"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba5201f7_35514bd9","line":1379,"in_reply_to":"ba5201f7_710eb233","updated":"2017-01-11 09:26:21.000000000","message":"Been explicit is better for sure. I have done this to not break the current Ceilometer dispatcher.\n\nI prefer this format too. But it means will we break \ncompatibility with Ceilometer \u003c ocata.\n\nMy version is compatible with Ceilometer \u003c ocata but that depends on a python tricks. Ceilometer uses\n\n  for rid in e.message[\u0027detail\u0027]:\n\nand that works for list and dict :)\n\nBut I\u0027m ok break the compat for this better error.","commit_id":"87e9e695f9f7f03aacc1f0680e1fbf00e12fdeec"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7ec24b8e369e92bfdff71f8d6623454972d5de61","unresolved":false,"context_lines":[{"line_number":1343,"context_line":"        unknown_metrics \u003d []"},{"line_number":1344,"context_line":"        unknown_resources \u003d []"},{"line_number":1345,"context_line":"        body_by_rid \u003d {}"},{"line_number":1346,"context_line":"        for original_resource_id, resource_id in body:"},{"line_number":1347,"context_line":"            body_by_rid[resource_id] \u003d body[(original_resource_id,"},{"line_number":1348,"context_line":"                                             resource_id)]"},{"line_number":1349,"context_line":"            names \u003d body[(original_resource_id, resource_id)].keys()"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a3905b3_a62b982f","line":1346,"updated":"2017-01-19 15:47:40.000000000","message":"what\u0027s the diff between resource_id and original_resource_id? i thought we don\u0027t do that translation anymore? is this to support future project-unique id work?","commit_id":"5005d598bc6f8ebf99aeb931583693f429de437a"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f515a4e9c62cb69f0666a00553b6a29aa32927e3","unresolved":false,"context_lines":[{"line_number":1343,"context_line":"        unknown_metrics \u003d []"},{"line_number":1344,"context_line":"        unknown_resources \u003d []"},{"line_number":1345,"context_line":"        body_by_rid \u003d {}"},{"line_number":1346,"context_line":"        for original_resource_id, resource_id in body:"},{"line_number":1347,"context_line":"            body_by_rid[resource_id] \u003d body[(original_resource_id,"},{"line_number":1348,"context_line":"                                             resource_id)]"},{"line_number":1349,"context_line":"            names \u003d body[(original_resource_id, resource_id)].keys()"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a3905b3_9002513d","line":1346,"in_reply_to":"5a3905b3_a62b982f","updated":"2017-01-19 16:39:17.000000000","message":"we always do that translation if original_resource_id is not an uuid","commit_id":"5005d598bc6f8ebf99aeb931583693f429de437a"}]}