)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"83a957a5a867487ab5b021656cfe84d82a36d4d2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix protocol cascade delete migration names"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due the related bug, the names weren\u0027t being correctly processed. So we"},{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_f3379b4a","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":19},"updated":"2017-02-14 15:51:59.000000000","message":"is it a related bug? Isn\u0027t it just \"the\" bug?","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"008f53653af7b19a86039d43ee1fcf731b60a4b0","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix protocol cascade delete migration names"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due the related bug, the names weren\u0027t being correctly processed. So we"},{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_f32dd046","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":19},"in_reply_to":"fa31d9ce_f3379b4a","updated":"2017-02-14 19:19:34.000000000","message":"I meant the \"Related-Bug\", should I rephrase it?","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"83a957a5a867487ab5b021656cfe84d82a36d4d2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix protocol cascade delete migration names"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due the related bug, the names weren\u0027t being correctly processed. So we"},{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"},{"line_number":13,"context_line":"Related-Bug: 1658116"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_9313bfb7","line":10,"range":{"start_line":10,"start_character":42,"end_line":10,"end_character":47},"updated":"2017-02-14 15:51:59.000000000","message":"s/wrong/each/ ??","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"008f53653af7b19a86039d43ee1fcf731b60a4b0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix protocol cascade delete migration names"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due the related bug, the names weren\u0027t being correctly processed. So we"},{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"},{"line_number":13,"context_line":"Related-Bug: 1658116"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_93f814c5","line":10,"range":{"start_line":10,"start_character":42,"end_line":10,"end_character":47},"in_reply_to":"fa31d9ce_9313bfb7","updated":"2017-02-14 19:19:34.000000000","message":"I guess it is \"wrong\", we had to add the exception to the data_migration banned operation tests although the data_migration script were empty.","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"e84ebc359b9210d842cb26adb3f31d09da0a7a67","unresolved":false,"context_lines":[{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"},{"line_number":13,"context_line":"Related-Bug: 1658116"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_96eaa672","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"updated":"2017-02-15 11:07:04.000000000","message":"let me rephrase Henry\u0027s question: should it be Related-Bug or Closes-Bug?","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"58b3f259ced6a64dfe16d954cfe2d63c73f2faca","unresolved":false,"context_lines":[{"line_number":10,"context_line":"needed to add the migration number to the wrong test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2413686cd8c89c88397fd3923fe9491015a8c534"},{"line_number":13,"context_line":"Related-Bug: 1658116"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_4a531d17","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"fa31d9ce_96eaa672","updated":"2017-02-15 12:20:20.000000000","message":"related, the actual fix is to proper enforce the names have to be different.","commit_id":"e84de06f9ac939eefaeb9156be7256304969117f"}],"keystone/tests/unit/test_sql_banned_operations.py":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"83a957a5a867487ab5b021656cfe84d82a36d4d2","unresolved":false,"context_lines":[{"line_number":281,"context_line":"        # timestamp to datetime and updates the initial value in the contract"},{"line_number":282,"context_line":"        # phase. Adding an exception here to pass data migrations banned tests,"},{"line_number":283,"context_line":"        # otherwise fails."},{"line_number":284,"context_line":"        4,"},{"line_number":285,"context_line":"        # Migration 013 updates a foreign key constraint at the federated_user"},{"line_number":286,"context_line":"        # table. It is a composite key pointing to the procotol.id and"},{"line_number":287,"context_line":"        # protocol.idp_id columns. Since we can\u0027t create a new foreign key"},{"line_number":288,"context_line":"        # before dropping the old one and the operations happens in the same"},{"line_number":289,"context_line":"        # upgrade phase, adding an exception here to pass the data migration"},{"line_number":290,"context_line":"        # banned tests."},{"line_number":291,"context_line":"        13"},{"line_number":292,"context_line":"    ]"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_136fcf2b","side":"PARENT","line":291,"range":{"start_line":284,"start_character":9,"end_line":291,"end_character":10},"updated":"2017-02-14 15:51:59.000000000","message":"is removing this part of the same bug/issue?","commit_id":"9c474958b16ca1775fd1f1c09b870817708d1ea2"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"008f53653af7b19a86039d43ee1fcf731b60a4b0","unresolved":false,"context_lines":[{"line_number":281,"context_line":"        # timestamp to datetime and updates the initial value in the contract"},{"line_number":282,"context_line":"        # phase. Adding an exception here to pass data migrations banned tests,"},{"line_number":283,"context_line":"        # otherwise fails."},{"line_number":284,"context_line":"        4,"},{"line_number":285,"context_line":"        # Migration 013 updates a foreign key constraint at the federated_user"},{"line_number":286,"context_line":"        # table. It is a composite key pointing to the procotol.id and"},{"line_number":287,"context_line":"        # protocol.idp_id columns. Since we can\u0027t create a new foreign key"},{"line_number":288,"context_line":"        # before dropping the old one and the operations happens in the same"},{"line_number":289,"context_line":"        # upgrade phase, adding an exception here to pass the data migration"},{"line_number":290,"context_line":"        # banned tests."},{"line_number":291,"context_line":"        13"},{"line_number":292,"context_line":"    ]"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_73748067","side":"PARENT","line":291,"range":{"start_line":284,"start_character":9,"end_line":291,"end_character":10},"in_reply_to":"fa31d9ce_136fcf2b","updated":"2017-02-14 19:19:34.000000000","message":"yes, we had to put this because the bug.","commit_id":"9c474958b16ca1775fd1f1c09b870817708d1ea2"}]}