)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"e826a98eb39caf6f1e269247b6f1963f83c517d2","unresolved":false,"context_lines":[{"line_number":19,"context_line":"I62646b99adca47a9c9fe0f466f7a23ac8fa4553e."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: Ie127f55f2e5bcd383796ff51253cf1091be16c98"},{"line_number":22,"context_line":"Closes-Bug: #1537882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_c4f3069e","line":22,"updated":"2016-07-25 20:05:13.000000000","message":"@gibi: Why not?","commit_id":"e36bc6ee352600d77f4b2cbf352bf2a09933c86d"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"03d027d057ce929df019c736f1ccebf02c7c798e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"I62646b99adca47a9c9fe0f466f7a23ac8fa4553e."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: Ie127f55f2e5bcd383796ff51253cf1091be16c98"},{"line_number":22,"context_line":"Closes-Bug: #1537882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1aa78d24_ee56d25a","line":22,"updated":"2016-07-06 07:27:29.000000000","message":"I think this patch does not fix the bug any more","commit_id":"e36bc6ee352600d77f4b2cbf352bf2a09933c86d"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5acfab2b742329ca8fbbc148871cfb526f146cdc","unresolved":false,"context_lines":[{"line_number":19,"context_line":"I62646b99adca47a9c9fe0f466f7a23ac8fa4553e."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: Ie127f55f2e5bcd383796ff51253cf1091be16c98"},{"line_number":22,"context_line":"Closes-Bug: #1537882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_e75f113f","line":22,"in_reply_to":"dada55a8_c4f3069e","updated":"2016-07-26 09:36:18.000000000","message":"@Jay, sorry for being too short. So the original problem reported in the #1537882 says that test_obj_make_compatible test case will start failing in notification payload ovos because some of those expects parameters in __init__ which is not provided by the test case. \nHowever during this refactoring https://review.openstack.org/#/c/331727/ we moved the notification objects test to a separate test class so the test_obj_make_compatible test case will not run on the notification objects any more and therefor it cannot fail with this reason.","commit_id":"e36bc6ee352600d77f4b2cbf352bf2a09933c86d"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"66a703b4f1103690c403bc84dac1ffbf4209c54f","unresolved":false,"context_lines":[{"line_number":19,"context_line":"I62646b99adca47a9c9fe0f466f7a23ac8fa4553e."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: Ie127f55f2e5bcd383796ff51253cf1091be16c98"},{"line_number":22,"context_line":"Closes-Bug: #1537882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bacf61ea_107c761f","line":22,"in_reply_to":"dada55a8_e75f113f","updated":"2016-08-03 09:17:58.000000000","message":"I guess, but this bug is raised to track this patch, so we can always edit the bug to match in this case.","commit_id":"e36bc6ee352600d77f4b2cbf352bf2a09933c86d"}],"nova/tests/unit/objects/test_objects.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"fb997e2c469f9504f7b4be6056d89d114eb84f4e","unresolved":false,"context_lines":[{"line_number":1239,"context_line":"        # Hold a dictionary of args/kwargs that need to get passed into"},{"line_number":1240,"context_line":"        # __init__() for specific classes. The key in the dictionary is"},{"line_number":1241,"context_line":"        # the obj_class that needs the init args/kwargs."},{"line_number":1242,"context_line":"        init_args \u003d {service_notif.ServiceStatusPayload: [objects.Service()]}"},{"line_number":1243,"context_line":"        init_kwargs \u003d {}"},{"line_number":1244,"context_line":""},{"line_number":1245,"context_line":"        checker \u003d fixture.ObjectVersionChecker("}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_ade19d00","line":1242,"updated":"2016-07-05 12:59:44.000000000","message":"I think it is not needed / not needed here after https://review.openstack.org/#/c/331727/ as this test will not run on ServiceStatusPayload","commit_id":"9d602979829140148653fb4c1ad3d767b32cb21f"},{"author":{"_account_id":11803,"name":"Ryan Rossiter","email":"rlrossit@us.ibm.com","username":"rlrossiter"},"change_message_id":"25fc3551509e8b97570e0434d5a411a410c99df4","unresolved":false,"context_lines":[{"line_number":1239,"context_line":"        # Hold a dictionary of args/kwargs that need to get passed into"},{"line_number":1240,"context_line":"        # __init__() for specific classes. The key in the dictionary is"},{"line_number":1241,"context_line":"        # the obj_class that needs the init args/kwargs."},{"line_number":1242,"context_line":"        init_args \u003d {service_notif.ServiceStatusPayload: [objects.Service()]}"},{"line_number":1243,"context_line":"        init_kwargs \u003d {}"},{"line_number":1244,"context_line":""},{"line_number":1245,"context_line":"        checker \u003d fixture.ObjectVersionChecker("}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_bc6edf93","line":1242,"in_reply_to":"1aa78d24_ade19d00","updated":"2016-07-05 18:44:35.000000000","message":"I was wondering why that wasn\u0027t failing after pulling in the new notifications :). Good catch.","commit_id":"9d602979829140148653fb4c1ad3d767b32cb21f"}]}