)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"8ca4811abe29963bd24a5a66a36ef08671c9c5b8","unresolved":false,"context_lines":[{"line_number":17,"context_line":"following patches."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I3be3f7fb8300e8853ebd35986cdf159422aa247a"},{"line_number":20,"context_line":"Part of blueprint unify-timestamp-of-polled-data"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"1a122d0e_30f728d4","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":48},"updated":"2016-04-21 08:57:22.000000000","message":"add bug id as well","commit_id":"4fc302a232ec85f93c9e0c212a9d503a8431171e"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a58e4d90c60fdc984b15d2271f1f9baf351a242a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"following patches."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I3be3f7fb8300e8853ebd35986cdf159422aa247a"},{"line_number":20,"context_line":"Related-Bug: #1491509"},{"line_number":21,"context_line":"Part of blueprint unify-timestamp-of-polled-data"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"1a122d0e_4390e0ec","line":20,"updated":"2016-04-22 12:22:17.000000000","message":"Closes-Bug:","commit_id":"77d64e702e2d24e38d1965153b96dd7134ae48d4"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a58e4d90c60fdc984b15d2271f1f9baf351a242a","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I3be3f7fb8300e8853ebd35986cdf159422aa247a"},{"line_number":20,"context_line":"Related-Bug: #1491509"},{"line_number":21,"context_line":"Part of blueprint unify-timestamp-of-polled-data"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"1a122d0e_03ad4831","line":21,"updated":"2016-04-22 12:22:17.000000000","message":"Implements: blueprint unify-timestamp-of-polled-data","commit_id":"77d64e702e2d24e38d1965153b96dd7134ae48d4"}],"ceilometer/agent/manager.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"1449117dd536c63238d1fb48b0adbd5a91ce9607","unresolved":false,"context_lines":[{"line_number":227,"context_line":"            {\u0027samples\u0027: samples}"},{"line_number":228,"context_line":"        )"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def _get_current_timestamp(self):"},{"line_number":231,"context_line":"        return timeutils.utcnow().isoformat()"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"class AgentManager(service_base.BaseService):"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_6f93955b","line":231,"range":{"start_line":230,"start_character":4,"end_line":231,"end_character":45},"updated":"2016-04-20 21:18:16.000000000","message":"i think it is not worth to define a new method for a single short expression","commit_id":"2969aba2d36d59869544100359bcf6729d181d6e"}],"ceilometer/agent/plugin_base.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"2228df75b2318c1b3b5f428b540a671ef2c7dc77","unresolved":false,"context_lines":[{"line_number":227,"context_line":"            raise ExtensionLoadError(err)"},{"line_number":228,"context_line":"        return extensions"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def get_current_timestamp(self):"},{"line_number":231,"context_line":"        return timeutils.utcnow().isoformat()"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_75fb8647","line":230,"updated":"2015-10-09 16:48:40.000000000","message":"can be static","commit_id":"d130c919765e30dd896a6ae990eb03f3e7c7f00b"}],"ceilometer/compute/pollsters/instance.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"2228df75b2318c1b3b5f428b540a671ef2c7dc77","unresolved":false,"context_lines":[{"line_number":23,"context_line":"class InstancePollster(pollsters.BaseComputePollster):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    @staticmethod"},{"line_number":26,"context_line":"    def _get_current_timestamp():"},{"line_number":27,"context_line":"        return timeutils.utcnow().isoformat()"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_78b85d2a","line":26,"updated":"2015-10-09 16:48:40.000000000","message":"-1: I think there already has a method get_current_timestamp() in parent class","commit_id":"d130c919765e30dd896a6ae990eb03f3e7c7f00b"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"2228df75b2318c1b3b5f428b540a671ef2c7dc77","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @staticmethod"},{"line_number":30,"context_line":"    def get_samples(manager, cache, resources):"},{"line_number":31,"context_line":"        polling_timestamp \u003d InstancePollster._get_current_timestamp()"},{"line_number":32,"context_line":"        for instance in resources:"},{"line_number":33,"context_line":"            yield util.make_sample_from_instance("},{"line_number":34,"context_line":"                instance,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_38850559","line":31,"updated":"2015-10-09 16:48:40.000000000","message":"can be self.xxx","commit_id":"d130c919765e30dd896a6ae990eb03f3e7c7f00b"}],"ceilometer/compute/pollsters/util.py":[{"author":{"_account_id":18137,"name":"Xia Linjuan","email":"ljxiash@cn.ibm.com","username":"xialinjuan"},"change_message_id":"0800d150e6bb73d290748e9a7314d7945ee95097","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"def make_sample_from_instance(instance, name, type, unit, volume,"},{"line_number":78,"context_line":"                              resource_id\u003dNone, additional_metadata\u003dNone,"},{"line_number":79,"context_line":"                              timestamp\u003dNone):"},{"line_number":80,"context_line":"    additional_metadata \u003d additional_metadata or {}"},{"line_number":81,"context_line":"    resource_metadata \u003d _get_metadata_from_object(instance)"},{"line_number":82,"context_line":"    resource_metadata.update(additional_metadata)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a2ca52d_188f252b","line":79,"updated":"2015-10-16 08:59:28.000000000","message":"For every metric, you design it to get the value of timestamp when it starts pull. So all the metrics must have \"timestamp\" value, why not set \"timestamp\" instead of \"timestamp\u003dNone\"?","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"def make_sample_from_instance(instance, name, type, unit, volume,"},{"line_number":78,"context_line":"                              resource_id\u003dNone, additional_metadata\u003dNone,"},{"line_number":79,"context_line":"                              timestamp\u003dNone):"},{"line_number":80,"context_line":"    additional_metadata \u003d additional_metadata or {}"},{"line_number":81,"context_line":"    resource_metadata \u003d _get_metadata_from_object(instance)"},{"line_number":82,"context_line":"    resource_metadata.update(additional_metadata)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_625fe372","line":79,"in_reply_to":"5a2ca52d_188f252b","updated":"2016-04-12 15:37:52.000000000","message":"I think coding it as an optional parameter will minimize any breaks with existing code that doesn\u0027t get updated right away.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":18137,"name":"Xia Linjuan","email":"ljxiash@cn.ibm.com","username":"xialinjuan"},"change_message_id":"0800d150e6bb73d290748e9a7314d7945ee95097","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        user_id\u003dinstance.user_id,"},{"line_number":89,"context_line":"        project_id\u003dinstance.tenant_id,"},{"line_number":90,"context_line":"        resource_id\u003dresource_id or instance.id,"},{"line_number":91,"context_line":"        timestamp\u003dtimestamp or timeutils.utcnow().isoformat(),"},{"line_number":92,"context_line":"        resource_metadata\u003dresource_metadata,"},{"line_number":93,"context_line":"    )"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a2ca52d_78cc196a","line":91,"updated":"2015-10-16 08:59:28.000000000","message":"I think you can just say \"timestamp\u003dtimestamp\" like unit/name/type","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        user_id\u003dinstance.user_id,"},{"line_number":89,"context_line":"        project_id\u003dinstance.tenant_id,"},{"line_number":90,"context_line":"        resource_id\u003dresource_id or instance.id,"},{"line_number":91,"context_line":"        timestamp\u003dtimestamp or timeutils.utcnow().isoformat(),"},{"line_number":92,"context_line":"        resource_metadata\u003dresource_metadata,"},{"line_number":93,"context_line":"    )"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_22f32b75","line":91,"in_reply_to":"5a2ca52d_78cc196a","updated":"2016-04-12 15:37:52.000000000","message":"Those are not optional params, so they will always have a value. Coding it this way preserves the behavior for any metrics that are not updated.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}],"ceilometer/sample.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"ecc87ff4c190b7dd0c9619af8dc8987a0df8e174","unresolved":false,"context_lines":[{"line_number":57,"context_line":"class Sample(object):"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def __init__(self, name, type, unit, volume, user_id, project_id,"},{"line_number":60,"context_line":"                 resource_id, resource_metadata, timestamp\u003dNone, source\u003dNone,"},{"line_number":61,"context_line":"                 id\u003dNone):"},{"line_number":62,"context_line":"        self.name \u003d name"},{"line_number":63,"context_line":"        self.type \u003d type"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_edadc99c","line":60,"range":{"start_line":60,"start_character":30,"end_line":60,"end_character":64},"updated":"2016-04-30 06:53:30.000000000","message":"it is not good to switch arguments\u0027 position, and sometimes it is danger\n\nmake resource_metadata to be None as well, and if it is not set, we should give it a default {}","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"},{"author":{"_account_id":13248,"name":"Wenzhi Yu","email":"wenzhi_yu@163.com","username":"yuywz"},"change_message_id":"9a9e4654af55dca35315da2e55544791497e2be6","unresolved":false,"context_lines":[{"line_number":57,"context_line":"class Sample(object):"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def __init__(self, name, type, unit, volume, user_id, project_id,"},{"line_number":60,"context_line":"                 resource_id, resource_metadata, timestamp\u003dNone, source\u003dNone,"},{"line_number":61,"context_line":"                 id\u003dNone):"},{"line_number":62,"context_line":"        self.name \u003d name"},{"line_number":63,"context_line":"        self.type \u003d type"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_3c24c3c4","line":60,"range":{"start_line":60,"start_character":30,"end_line":60,"end_character":64},"in_reply_to":"1a122d0e_edadc99c","updated":"2016-05-04 01:44:07.000000000","message":"Done","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"}],"ceilometer/tests/unit/agent/agentbase.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"ecc87ff4c190b7dd0c9619af8dc8987a0df8e174","unresolved":false,"context_lines":[{"line_number":274,"context_line":"                \u0027transformers\u0027: [],"},{"line_number":275,"context_line":"                \u0027publishers\u0027: [\"test\"]}]"},{"line_number":276,"context_line":"        }"},{"line_number":277,"context_line":"        self.timestamp \u003d \u00272222-11-22T00:11:22.333333\u0027"},{"line_number":278,"context_line":"        self.setup_polling()"},{"line_number":279,"context_line":"        self.useFixture(mockpatch.PatchObject("},{"line_number":280,"context_line":"            publisher, \u0027get_publisher\u0027, side_effect\u003dself.get_publisher))"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_8d1ed5b9","line":277,"range":{"start_line":277,"start_character":8,"end_line":277,"end_character":53},"updated":"2016-04-30 06:53:30.000000000","message":"only one place use this value, why should it be object attribute rather than local variable?","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"},{"author":{"_account_id":13248,"name":"Wenzhi Yu","email":"wenzhi_yu@163.com","username":"yuywz"},"change_message_id":"9a9e4654af55dca35315da2e55544791497e2be6","unresolved":false,"context_lines":[{"line_number":274,"context_line":"                \u0027transformers\u0027: [],"},{"line_number":275,"context_line":"                \u0027publishers\u0027: [\"test\"]}]"},{"line_number":276,"context_line":"        }"},{"line_number":277,"context_line":"        self.timestamp \u003d \u00272222-11-22T00:11:22.333333\u0027"},{"line_number":278,"context_line":"        self.setup_polling()"},{"line_number":279,"context_line":"        self.useFixture(mockpatch.PatchObject("},{"line_number":280,"context_line":"            publisher, \u0027get_publisher\u0027, side_effect\u003dself.get_publisher))"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_5c29bf9a","line":277,"range":{"start_line":277,"start_character":8,"end_line":277,"end_character":53},"in_reply_to":"1a122d0e_8d1ed5b9","updated":"2016-05-04 01:44:07.000000000","message":"Done","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"ecc87ff4c190b7dd0c9619af8dc8987a0df8e174","unresolved":false,"context_lines":[{"line_number":729,"context_line":""},{"line_number":730,"context_line":"        self.setup_polling()"},{"line_number":731,"context_line":"        polling_task \u003d list(self.mgr.setup_polling_tasks().values())[0]"},{"line_number":732,"context_line":"        polling_task._send_notification \u003d \\"},{"line_number":733,"context_line":"            mock.Mock(side_effect\u003dfake_send_notification)"},{"line_number":734,"context_line":"        polling_task.poll_and_notify()"},{"line_number":735,"context_line":"        self.assertEqual(self.timestamp, polled_samples[0][\u0027timestamp\u0027])"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_ad14719f","line":732,"range":{"start_line":732,"start_character":42,"end_line":732,"end_character":43},"updated":"2016-04-30 06:53:30.000000000","message":"polling_task._send_notification \u003d mock.Mock(\n    side_effect\u003dfake_send_notification)","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"},{"author":{"_account_id":13248,"name":"Wenzhi Yu","email":"wenzhi_yu@163.com","username":"yuywz"},"change_message_id":"9a9e4654af55dca35315da2e55544791497e2be6","unresolved":false,"context_lines":[{"line_number":729,"context_line":""},{"line_number":730,"context_line":"        self.setup_polling()"},{"line_number":731,"context_line":"        polling_task \u003d list(self.mgr.setup_polling_tasks().values())[0]"},{"line_number":732,"context_line":"        polling_task._send_notification \u003d \\"},{"line_number":733,"context_line":"            mock.Mock(side_effect\u003dfake_send_notification)"},{"line_number":734,"context_line":"        polling_task.poll_and_notify()"},{"line_number":735,"context_line":"        self.assertEqual(self.timestamp, polled_samples[0][\u0027timestamp\u0027])"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_bc161356","line":732,"range":{"start_line":732,"start_character":42,"end_line":732,"end_character":43},"in_reply_to":"1a122d0e_ad14719f","updated":"2016-05-04 01:44:07.000000000","message":"Done","commit_id":"021508b06dcda088a6e3668471cbd8d8225487dd"}],"ceilometer/tests/unit/compute/pollsters/test_cpu.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        pollster \u003d cpu.CPUPollster()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":50,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        def _verify_cpu_metering(expected_time):"},{"line_number":53,"context_line":"            cache \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_424aa784","line":50,"updated":"2016-04-12 15:37:52.000000000","message":"Generally it\u0027s best to avoid continuation characters. I\u0027d write it as:\n\n    pollster.get_current_timestamp \u003d mock.Mock(\n            return_value\u003dself.timestamp)","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        pollster \u003d cpu.CPUUtilPollster()"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":104,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        def _verify_cpu_util_metering(expected_util):"},{"line_number":107,"context_line":"            cache \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_c25eb744","line":104,"updated":"2016-04-12 15:37:52.000000000","message":"see above about continuation","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}],"ceilometer/tests/unit/compute/pollsters/test_diskio.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        pollster \u003d factory()"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":70,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        mgr \u003d manager.AgentManager()"},{"line_number":73,"context_line":"        cache \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_45ff5101","line":70,"updated":"2016-04-12 15:37:52.000000000","message":"same here.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}],"ceilometer/tests/unit/compute/pollsters/test_instance.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def test_get_samples_instance(self):"},{"line_number":32,"context_line":"        mgr \u003d manager.AgentManager()"},{"line_number":33,"context_line":"        pollsters_instance.InstancePollster.get_current_timestamp \u003d \\"},{"line_number":34,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":35,"context_line":"        pollster \u003d pollsters_instance.InstancePollster()"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        samples \u003d list(pollster.get_samples(mgr, {}, [self.instance]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_65cfedb0","line":34,"updated":"2016-04-12 15:37:52.000000000","message":"same concern about continuation. Moving the `mock.Mock(` portion to the parent line will just barely fit.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        pollsters_instance.InstancePollster.get_current_timestamp \u003d \\"},{"line_number":34,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":35,"context_line":"        pollster \u003d pollsters_instance.InstancePollster()"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        samples \u003d list(pollster.get_samples(mgr, {}, [self.instance]))"},{"line_number":38,"context_line":"        self.assertEqual(1, len(samples))"},{"line_number":39,"context_line":"        self.assertEqual(\u0027instance\u0027, samples[0].name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_e552dddc","line":36,"updated":"2016-04-12 15:37:52.000000000","message":"Unrelated whitespace change. Please try to avoid these.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}],"ceilometer/tests/unit/compute/pollsters/test_memory.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        pollster \u003d memory.MemoryUsagePollster()"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":52,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        @mock.patch(\u0027ceilometer.compute.pollsters.memory.LOG\u0027)"},{"line_number":55,"context_line":"        def _verify_memory_metering(expected_count, expected_memory_mb, mylog):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_c572417c","line":52,"updated":"2016-04-12 15:37:52.000000000","message":"continuation","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        pollster \u003d memory.MemoryResidentPollster()"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":102,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        @mock.patch(\u0027ceilometer.compute.pollsters.memory.LOG\u0027)"},{"line_number":105,"context_line":"        def _verify_resident_memory_metering(expected_count,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_450591c0","line":102,"updated":"2016-04-12 15:37:52.000000000","message":"continuation","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}],"ceilometer/tests/unit/compute/pollsters/test_net.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    def _check_get_samples(self, factory, expected):"},{"line_number":114,"context_line":"        mgr \u003d manager.AgentManager()"},{"line_number":115,"context_line":"        pollster \u003d factory()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":118,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_65b9ade9","line":116,"updated":"2016-04-12 15:37:52.000000000","message":"Unrelated whitespace change.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        pollster \u003d factory()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":118,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        samples \u003d list(pollster.get_samples(mgr, {}, [self.instance]))"},{"line_number":121,"context_line":"        self.assertEqual(3, len(samples))  # one for each nic"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_25a695c1","line":118,"updated":"2016-04-12 15:37:52.000000000","message":"continuation","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":216,"context_line":""},{"line_number":217,"context_line":"        mgr \u003d manager.AgentManager()"},{"line_number":218,"context_line":"        pollster \u003d factory()"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":221,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":222,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_e59c9d6b","line":219,"updated":"2016-04-12 15:37:52.000000000","message":"Unrelated whitespace change","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":218,"context_line":"        pollster \u003d factory()"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":221,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        cache \u003d {"},{"line_number":224,"context_line":"            pollster.CACHE_KEY_VNIC: {"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_858209ff","line":221,"updated":"2016-04-12 15:37:52.000000000","message":"continuation","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"fbc49654504b1cbd719aba61bbdb3dcf2973e133","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        pollster \u003d factory()"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"        pollster.get_current_timestamp \u003d \\"},{"line_number":293,"context_line":"            mock.Mock(return_value\u003dself.timestamp)"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"        samples \u003d list(pollster.get_samples(mgr, {}, [self.instance]))"},{"line_number":296,"context_line":"        self.assertEqual(3, len(samples))  # one for each nic"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_458f5123","line":293,"updated":"2016-04-12 15:37:52.000000000","message":"whitespace and continuation.","commit_id":"9562e570b77338a325b75a607dbe92753d2a465b"}]}