)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"19e1ece85c87e5fef52f8396d93afeb4a2a595e5","unresolved":false,"context_lines":[{"line_number":9,"context_line":"this is a patch until we properly separate polling logic. this"},{"line_number":10,"context_line":"patch adds history so if a resource is defined in separate pipelines"},{"line_number":11,"context_line":"it will only be processed in one. by doing this, we send only a"},{"line_number":12,"context_line":"single copy of each sample and it can be processed appropriatedly"},{"line_number":13,"context_line":"by the notification agent"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I9c1d3e25740fff5281d796c7bfea2f0e105bb5c5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1a4dcd0f_1ba865da","line":12,"updated":"2015-08-06 12:00:26.000000000","message":"appropriately","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"b835c43857308635b053657f00cb42abb43e6dd2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"this is a patch until we properly separate polling logic. this"},{"line_number":10,"context_line":"patch adds history so if a resource is defined in separate pipelines"},{"line_number":11,"context_line":"it will only be processed in one. by doing this, we send only a"},{"line_number":12,"context_line":"single copy of each sample and it can be processed appropriatedly"},{"line_number":13,"context_line":"by the notification agent"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I9c1d3e25740fff5281d796c7bfea2f0e105bb5c5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1a4dcd0f_46a2d14c","line":12,"in_reply_to":"1a4dcd0f_1ba865da","updated":"2015-08-11 13:13:11.000000000","message":"Done","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"}],"ceilometer/agent/base.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"f72010adeaf146ed7dbd5093a49165b28d72ee96","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"                # Remove duplicated resources and black resources. Using"},{"line_number":146,"context_line":"                # set() requires well defined __hash__ for each resource."},{"line_number":147,"context_line":"                # Since __eq__ is defined, \u0027not in\u0027 is safe here."},{"line_number":148,"context_line":"                seen \u003d []"},{"line_number":149,"context_line":"                duplicated \u003d []"},{"line_number":150,"context_line":"                polling_resources \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_c39fb85d","line":147,"updated":"2015-08-11 11:35:58.000000000","message":"I\u0027m not sure but can\u0027t we remove \u0027seen\u0027?","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"b835c43857308635b053657f00cb42abb43e6dd2","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"                # Remove duplicated resources and black resources. Using"},{"line_number":146,"context_line":"                # set() requires well defined __hash__ for each resource."},{"line_number":147,"context_line":"                # Since __eq__ is defined, \u0027not in\u0027 is safe here."},{"line_number":148,"context_line":"                seen \u003d []"},{"line_number":149,"context_line":"                duplicated \u003d []"},{"line_number":150,"context_line":"                polling_resources \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_268305ce","line":147,"in_reply_to":"1a4dcd0f_c39fb85d","updated":"2015-08-11 13:13:11.000000000","message":"hmm. i think i need to remove Duplicate error message... which i guess is fine.","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"f72010adeaf146ed7dbd5093a49165b28d72ee96","unresolved":false,"context_lines":[{"line_number":178,"context_line":"                # If no resources, skip for this pollster"},{"line_number":179,"context_line":"                if not polling_resources:"},{"line_number":180,"context_line":"                    context \u003d \u0027new \u0027 if history else \u0027\u0027"},{"line_number":181,"context_line":"                    LOG.info(_(\"Skip polling pollster %(name)s, no %(context)s\""},{"line_number":182,"context_line":"                               \"resources found\"), {\u0027name\u0027: pollster.name,"},{"line_number":183,"context_line":"                                                    \u0027context\u0027: context})"},{"line_number":184,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_03b04042","line":181,"updated":"2015-08-11 11:35:58.000000000","message":"could you please add a space before \u0027resources\u0027?","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"b835c43857308635b053657f00cb42abb43e6dd2","unresolved":false,"context_lines":[{"line_number":178,"context_line":"                # If no resources, skip for this pollster"},{"line_number":179,"context_line":"                if not polling_resources:"},{"line_number":180,"context_line":"                    context \u003d \u0027new \u0027 if history else \u0027\u0027"},{"line_number":181,"context_line":"                    LOG.info(_(\"Skip polling pollster %(name)s, no %(context)s\""},{"line_number":182,"context_line":"                               \"resources found\"), {\u0027name\u0027: pollster.name,"},{"line_number":183,"context_line":"                                                    \u0027context\u0027: context})"},{"line_number":184,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_66b6eddd","line":181,"in_reply_to":"1a4dcd0f_03b04042","updated":"2015-08-11 13:13:11.000000000","message":"the space isn\u0027t needed. it either uses the space after \u0027no\u0027 if history doesn\u0027t exist, or it uses space after \u0027new \u0027 if space does exists.","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"}],"ceilometer/tests/unit/agent/agentbase.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"f72010adeaf146ed7dbd5093a49165b28d72ee96","unresolved":false,"context_lines":[{"line_number":678,"context_line":"                         len(p_coord.extract_my_subset.call_args_list))"},{"line_number":679,"context_line":"        for c in expected:"},{"line_number":680,"context_line":"            self.assertIn(c, p_coord.extract_my_subset.call_args_list)"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"    @mock.patch(\u0027ceilometer.agent.base.LOG\u0027)"},{"line_number":683,"context_line":"    def test_skip_polling_and_notify_with_no_resources(self, LOG):"},{"line_number":684,"context_line":"        self.pipeline_cfg[\u0027sources\u0027][0][\u0027resources\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_a32e2cf8","line":681,"updated":"2015-08-11 11:35:58.000000000","message":"we don\u0027t have any positive tests. But I guess that without poll_and_notify simplification, it cannot be done.","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"b835c43857308635b053657f00cb42abb43e6dd2","unresolved":false,"context_lines":[{"line_number":678,"context_line":"                         len(p_coord.extract_my_subset.call_args_list))"},{"line_number":679,"context_line":"        for c in expected:"},{"line_number":680,"context_line":"            self.assertIn(c, p_coord.extract_my_subset.call_args_list)"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"    @mock.patch(\u0027ceilometer.agent.base.LOG\u0027)"},{"line_number":683,"context_line":"    def test_skip_polling_and_notify_with_no_resources(self, LOG):"},{"line_number":684,"context_line":"        self.pipeline_cfg[\u0027sources\u0027][0][\u0027resources\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_269f8504","line":681,"in_reply_to":"1a4dcd0f_a32e2cf8","updated":"2015-08-11 13:13:11.000000000","message":"i think i have a plan!","commit_id":"f8dcc7fb0fe09cf615ca65f7c738d6b5459ad1a8"}]}