)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13555,"name":"Pradeep K Surisetty","username":"pradeepkumars","inactive":true},"change_message_id":"c6b01a55fc05580a03842a4664d9a38f5f42fc89","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Added context ceilometer.py for creating samples"},{"line_number":10,"context_line":"and collecting resources in list"},{"line_number":11,"context_line":"- Added CeilometerSamples.list_samples scenario"},{"line_number":12,"context_line":"- Added CeilometerResource.get_resources_on_tenant scenario"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4cf6148cdd3fa685a2ae17eb16f1e205e1edb2bf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"3acd31a7_a32f8fa0","line":11,"updated":"2015-04-30 02:35:15.000000000","message":"list_samples already there. \nresend patchset without that","commit_id":"c7e8a6abf827d8ca7f56653a4d3e1adb1551b1e0"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"3a16d9b09703805b9098eb0b593924d996326e28","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Added context ceilometer.py for creating samples"},{"line_number":10,"context_line":"and collecting resources in list"},{"line_number":11,"context_line":"- Added CeilometerSamples.list_samples scenario"},{"line_number":12,"context_line":"- Added CeilometerResource.get_resources_on_tenant scenario"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4cf6148cdd3fa685a2ae17eb16f1e205e1edb2bf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"3acd31a7_dc02683e","line":11,"in_reply_to":"3acd31a7_a32f8fa0","updated":"2015-04-30 06:09:16.000000000","message":"Done","commit_id":"c7e8a6abf827d8ca7f56653a4d3e1adb1551b1e0"}],"rally/benchmark/context/ceilometer.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"ad155a55699e09c60e44dd5ededc2fe4edaf6642","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"@base.context(name\u003d\"ceilometer\", order\u003d450)"},{"line_number":29,"context_line":"class CeilometerGenerator(base.Context):"},{"line_number":30,"context_line":"    \"\"\"Context class for creating samples and resources for benchmarks.\"\"\""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    CONFIG_SCHEMA \u003d {"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6ced46_c752e212","line":29,"updated":"2015-03-23 14:20:26.000000000","message":"Name `SamplesGenerator\u0027 looks much better, since this actually generates samples","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"5c8d57e79020c5ad023ffff8d7bb859739b2f72d","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        counter_type \u003d self.config[\"counter_type\"]"},{"line_number":67,"context_line":"        counter_unit \u003d self.config[\"counter_unit\"]"},{"line_number":68,"context_line":"        counter_volume \u003d self.config[\"counter_volume\"]"},{"line_number":69,"context_line":"        samples_per_tenant \u003d self.config[\"samples_per_tenant\"]"},{"line_number":70,"context_line":"        for user, tenant_id in rutils.iterate_per_tenants("},{"line_number":71,"context_line":"                self.context[\"users\"]):"},{"line_number":72,"context_line":"            self.context[\"tenants\"][tenant_id][\"samples\"] \u003d []"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_3d413ea6","line":69,"updated":"2015-03-20 22:44:20.000000000","message":"Why we create all this variable? As I see we can delete this lines without pain","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"ad155a55699e09c60e44dd5ededc2fe4edaf6642","unresolved":false,"context_lines":[{"line_number":89,"context_line":"                        sample[0].resource_id)"},{"line_number":90,"context_line":"                except Exception as e:"},{"line_number":91,"context_line":"                    LOG.error(\"Creating a sample for tenant %(tenant_id)s\""},{"line_number":92,"context_line":"                              \"was failed. Exception: %(e)s\" %"},{"line_number":93,"context_line":"                              {\"tenant_id\": tenant_id, \"e\": e})"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    @rutils.log_task_wrapper(LOG.info, _(\"Exit context: `Ceilometer`\"))"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6ced46_2aca9b4f","line":92,"updated":"2015-03-23 14:20:26.000000000","message":"s/%(e)s/%(error)s/  # `e\u0027 is not a human-readable name","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"5c8d57e79020c5ad023ffff8d7bb859739b2f72d","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    @rutils.log_task_wrapper(LOG.info, _(\"Exit context: `Ceilometer`\"))"},{"line_number":96,"context_line":"    def cleanup(self):"},{"line_number":97,"context_line":"        resource_manager.cleanup(names\u003d[\"ceilometer.samples\"],"},{"line_number":98,"context_line":"                                 users\u003dself.context.get(\"users\", []))"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_98beb8d2","line":97,"updated":"2015-03-20 22:44:20.000000000","message":"As I see you don\u0027t add celiometer.samples to context/cleanup/resource.py and why we not add here celiometer.resources?","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            scenario \u003d ceilometer_utils.CeilometerScenario("},{"line_number":75,"context_line":"                clients\u003dclients)"},{"line_number":76,"context_line":"            for i in range(samples_per_tenant):"},{"line_number":77,"context_line":"                rnd_resource \u003d scenario._generate_random_name("},{"line_number":78,"context_line":"                    prefix\u003d\"rally_ctx_resource_\")"},{"line_number":79,"context_line":"                try:"},{"line_number":80,"context_line":"                    sample \u003d scenario._create_sample(counter_name,"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_d0659645","line":77,"updated":"2015-04-30 15:36:35.000000000","message":"Suggestion: make `rnd_resource\u0027 optional argument,\nand move _generate_random_name(...) to scenario._create_sample()\n\nExample:\nhttps://github.com/openstack/rally/blob/master/rally/benchmark/scenarios/nova/utils.py#L105","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            scenario \u003d ceilometer_utils.CeilometerScenario("},{"line_number":75,"context_line":"                clients\u003dclients)"},{"line_number":76,"context_line":"            for i in range(samples_per_tenant):"},{"line_number":77,"context_line":"                rnd_resource \u003d scenario._generate_random_name("},{"line_number":78,"context_line":"                    prefix\u003d\"rally_ctx_resource_\")"},{"line_number":79,"context_line":"                try:"},{"line_number":80,"context_line":"                    sample \u003d scenario._create_sample(counter_name,"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_eb75e3f9","line":77,"in_reply_to":"3acd31a7_d0659645","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                        sample[0].resource_id)"},{"line_number":89,"context_line":"                except Exception as e:"},{"line_number":90,"context_line":"                    LOG.error(\"Creating a sample for tenant %(tenant_id)s\""},{"line_number":91,"context_line":"                              \"was failed. Exception: %(error)s\" %"},{"line_number":92,"context_line":"                              {\"tenant_id\": tenant_id, \"error\": e})"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @rutils.log_task_wrapper(LOG.info, _(\"Exit context: `Ceilometer`\"))"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_d0f35672","line":91,"updated":"2015-04-30 15:36:35.000000000","message":"Let\u0027s simplify: s/failed. Exception:/failed:/","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                        sample[0].resource_id)"},{"line_number":89,"context_line":"                except Exception as e:"},{"line_number":90,"context_line":"                    LOG.error(\"Creating a sample for tenant %(tenant_id)s\""},{"line_number":91,"context_line":"                              \"was failed. Exception: %(error)s\" %"},{"line_number":92,"context_line":"                              {\"tenant_id\": tenant_id, \"error\": e})"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @rutils.log_task_wrapper(LOG.info, _(\"Exit context: `Ceilometer`\"))"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_10ce0a82","line":91,"in_reply_to":"3acd31a7_d0f35672","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}],"rally/benchmark/scenarios/ceilometer/resources.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @validation.required_services(consts.Service.CEILOMETER)"},{"line_number":36,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":37,"context_line":"    @base.scenario(context\u003d{\"cleanup\": [\"ceilometer\"]})"},{"line_number":38,"context_line":"    def get_tenant_resources(self):"},{"line_number":39,"context_line":"        \"\"\"Get all tenant resources."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_73cfec7f","line":37,"updated":"2015-04-30 15:36:35.000000000","message":"@base.scenario()  # Nothing to cleanup - we have only getters here","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @validation.required_services(consts.Service.CEILOMETER)"},{"line_number":36,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":37,"context_line":"    @base.scenario(context\u003d{\"cleanup\": [\"ceilometer\"]})"},{"line_number":38,"context_line":"    def get_tenant_resources(self):"},{"line_number":39,"context_line":"        \"\"\"Get all tenant resources."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_d0c3f255","line":37,"in_reply_to":"3acd31a7_73cfec7f","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        if resources:"},{"line_number":45,"context_line":"            for res_id in resources:"},{"line_number":46,"context_line":"                self._get_resource(res_id)"},{"line_number":47,"context_line":"        else:"},{"line_number":48,"context_line":"            msg \u003d (\"No resources found for tenant: %s\""},{"line_number":49,"context_line":"                   % self.context[\"tenant\"].get(\"name\"))"},{"line_number":50,"context_line":"            raise exceptions.NotFoundException(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_70cb6254","line":47,"updated":"2015-04-30 15:36:35.000000000","message":"Inverted condition is smaller and easier to read:\n\n  if not resources:\n      msg \u003d (\"No resources found for tenant: %s\"\n          % self.context[\"tenant\"].get(\"name\"))\n      raise exceptions.NotFoundException(message\u003dmsg)\n  for res_id in resources:\n      self._get_resource(res_id)","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        if resources:"},{"line_number":45,"context_line":"            for res_id in resources:"},{"line_number":46,"context_line":"                self._get_resource(res_id)"},{"line_number":47,"context_line":"        else:"},{"line_number":48,"context_line":"            msg \u003d (\"No resources found for tenant: %s\""},{"line_number":49,"context_line":"                   % self.context[\"tenant\"].get(\"name\"))"},{"line_number":50,"context_line":"            raise exceptions.NotFoundException(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_b0a3d6b1","line":47,"in_reply_to":"3acd31a7_70cb6254","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}],"rally/benchmark/scenarios/ceilometer/samples.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    @validation.required_services(consts.Service.CEILOMETER)"},{"line_number":25,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":26,"context_line":"    @base.scenario(context\u003d{\"cleanup\": [\"ceilometer\"]})"},{"line_number":27,"context_line":"    def list_samples(self):"},{"line_number":28,"context_line":"        \"\"\"Fetch all samples."},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_507506c9","line":26,"updated":"2015-04-30 15:36:35.000000000","message":"What resources are expected to be deleted? There is nothing created by this scenario :) Just revert this change","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    @validation.required_services(consts.Service.CEILOMETER)"},{"line_number":25,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":26,"context_line":"    @base.scenario(context\u003d{\"cleanup\": [\"ceilometer\"]})"},{"line_number":27,"context_line":"    def list_samples(self):"},{"line_number":28,"context_line":"        \"\"\"Fetch all samples."},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_90a8da8e","line":26,"in_reply_to":"3acd31a7_507506c9","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}],"rally/benchmark/scenarios/ceilometer/utils.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"ad155a55699e09c60e44dd5ededc2fe4edaf6642","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        \"\"\""},{"line_number":132,"context_line":"        return self.clients(\"ceilometer\").resources.list()"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    @base.atomic_action_timer(\"ceilometer.get_resources_on_tenant\")"},{"line_number":135,"context_line":"    def _get_resources_on_tenant(self):"},{"line_number":136,"context_line":"        \"\"\"Get information about resources on tenant."},{"line_number":137,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6ced46_8d52c9db","line":134,"updated":"2015-03-23 14:20:26.000000000","message":"Metering each self.clients(\"ceilometer\").resources.get(resource_id) separately is more logical for me.\n\nPlease consider about this example:\n\n utils.py\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\n @base.atomic_action_timer(\"ceilometer.get_resource\")\n def _get_resource(self, resource_id):\n   \"\"\"{Docstring}.\"\"\"\n   return self.clients(\"ceilometer\").resources.get(resource_id)\n\n samples.py\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n def get_tenant_resources(self):\n     \"\"\"Get all tenant resources.\n\n     :returns: list with resources data\n     \"\"\"\n     resources \u003d self.context[\"tenant\"].get(\"resources\", [])\n     if resources:\n         return [self._get_resource(res_id) for res_id in resource_list]\n     msg \u003d (\"No resources found for tenant: %s\"\n            % self.context[\"tenant\"].get(\"name\"))\n     raise exceptions.NotFoundException(message\u003dmsg)","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"5c8d57e79020c5ad023ffff8d7bb859739b2f72d","unresolved":false,"context_lines":[{"line_number":139,"context_line":"        \"\"\""},{"line_number":140,"context_line":"        resource_list \u003d self.context[\"tenant\"].get(\"resources\", [])"},{"line_number":141,"context_line":"        if not resource_list:"},{"line_number":142,"context_line":"            msg \u003d (\"No resources found for tenant: %s\""},{"line_number":143,"context_line":"                   % self.context[\"tenant\"].get(\"name\"))"},{"line_number":144,"context_line":"            raise exceptions.NotFoundException(message\u003dmsg)"},{"line_number":145,"context_line":"        return [self.clients(\"ceilometer\").resources.get(resource_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_dd0ca244","line":142,"updated":"2015-03-20 22:44:20.000000000","message":"Maybe its better to internationalize this message","commit_id":"f57f79834fc867cef5ae7c02f23ddc50f20bab58"}],"tests/unit/benchmark/context/test_ceilometer.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        self.assertEqual(new_context, ceilometer_ctx.context)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    @mock.patch(\"%s.ceilometer.osclients\" % CTX)"},{"line_number":110,"context_line":"    def test_cleanup(self, mock_cleanup):"},{"line_number":111,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_d387e05f","line":110,"updated":"2015-04-30 15:36:35.000000000","message":"mock_osclients","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        self.assertEqual(new_context, ceilometer_ctx.context)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    @mock.patch(\"%s.ceilometer.osclients\" % CTX)"},{"line_number":110,"context_line":"    def test_cleanup(self, mock_cleanup):"},{"line_number":111,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_1095ea3d","line":110,"in_reply_to":"3acd31a7_d387e05f","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    @mock.patch(\"%s.ceilometer.osclients\" % CTX)"},{"line_number":110,"context_line":"    def test_cleanup(self, mock_cleanup):"},{"line_number":111,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_33ba9424","line":111,"updated":"2015-04-30 15:36:35.000000000","message":"`pass\u0027 makes no sense of this test. Just call cleanup() to be sure that no exceptions raised","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    @mock.patch(\"%s.ceilometer.osclients\" % CTX)"},{"line_number":110,"context_line":"    def test_cleanup(self, mock_cleanup):"},{"line_number":111,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_f09dce66","line":111,"in_reply_to":"3acd31a7_33ba9424","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}],"tests/unit/benchmark/scenarios/ceilometer/test_resources.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        scenario \u003d resources.CeilometerResource()"},{"line_number":31,"context_line":"        resource_list \u003d [\"id1\", \"id2\", \"id3\", \"id4\"]"},{"line_number":32,"context_line":"        context \u003d {\"user\": {\"tenant_id\": \"fake\"},"},{"line_number":33,"context_line":"                   \"users\": [{\"tenant_id\": \"fake\","},{"line_number":34,"context_line":"                              \"users_per_tenant\": 1}],"},{"line_number":35,"context_line":"                   \"tenant\": {\"id\": \"fake\", \"resources\": resource_list}}"},{"line_number":36,"context_line":"        scenario.context \u003d context"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_73cb6c54","line":33,"updated":"2015-04-30 15:36:35.000000000","message":"Key \"users\" is not required","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        scenario \u003d resources.CeilometerResource()"},{"line_number":31,"context_line":"        resource_list \u003d [\"id1\", \"id2\", \"id3\", \"id4\"]"},{"line_number":32,"context_line":"        context \u003d {\"user\": {\"tenant_id\": \"fake\"},"},{"line_number":33,"context_line":"                   \"users\": [{\"tenant_id\": \"fake\","},{"line_number":34,"context_line":"                              \"users_per_tenant\": 1}],"},{"line_number":35,"context_line":"                   \"tenant\": {\"id\": \"fake\", \"resources\": resource_list}}"},{"line_number":36,"context_line":"        scenario.context \u003d context"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_b08ab619","line":33,"in_reply_to":"3acd31a7_73cb6c54","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        scenario \u003d resources.CeilometerResource()"},{"line_number":44,"context_line":"        resource_list \u003d []"},{"line_number":45,"context_line":"        context \u003d {\"user\": {\"tenant_id\": \"fake\"},"},{"line_number":46,"context_line":"                   \"users\": [{\"tenant_id\": \"fake\","},{"line_number":47,"context_line":"                              \"users_per_tenant\": 1}],"},{"line_number":48,"context_line":"                   \"tenant\": {\"id\": \"fake\", \"resources\": resource_list}}"},{"line_number":49,"context_line":"        scenario.context \u003d context"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_f3dd3c15","line":46,"updated":"2015-04-30 15:36:35.000000000","message":"Key \"users\" is not required","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        scenario \u003d resources.CeilometerResource()"},{"line_number":44,"context_line":"        resource_list \u003d []"},{"line_number":45,"context_line":"        context \u003d {\"user\": {\"tenant_id\": \"fake\"},"},{"line_number":46,"context_line":"                   \"users\": [{\"tenant_id\": \"fake\","},{"line_number":47,"context_line":"                              \"users_per_tenant\": 1}],"},{"line_number":48,"context_line":"                   \"tenant\": {\"id\": \"fake\", \"resources\": resource_list}}"},{"line_number":49,"context_line":"        scenario.context \u003d context"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_70809ef7","line":46,"in_reply_to":"3acd31a7_f3dd3c15","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}],"tests/unit/fakes.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":232,"context_line":"        self.counter_volume \u003d kwargs.get(\"counter_volume\", 100)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def to_dict(self):"},{"line_number":235,"context_line":"        return {\"counter_name\": \"fake-counter-name\","},{"line_number":236,"context_line":"                \"counter_type\": \"fake-counter-type\","},{"line_number":237,"context_line":"                \"counter_unit\": \"fake-counter-unit\","},{"line_number":238,"context_line":"                \"counter_volume\": 100,"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_f3c77c10","line":235,"updated":"2015-04-30 15:36:35.000000000","message":"\"counter_name\": self.counter_name,  # In same way all values below","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":232,"context_line":"        self.counter_volume \u003d kwargs.get(\"counter_volume\", 100)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def to_dict(self):"},{"line_number":235,"context_line":"        return {\"counter_name\": \"fake-counter-name\","},{"line_number":236,"context_line":"                \"counter_type\": \"fake-counter-type\","},{"line_number":237,"context_line":"                \"counter_unit\": \"fake-counter-unit\","},{"line_number":238,"context_line":"                \"counter_volume\": 100,"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_307fc616","line":235,"in_reply_to":"3acd31a7_f3c77c10","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"0ff39621bd1c597ad6dbdd27b19d5d836b060c17","unresolved":false,"context_lines":[{"line_number":334,"context_line":"        self.cache[resource.uuid] \u003d resource"},{"line_number":335,"context_line":"        if inlist:"},{"line_number":336,"context_line":"            return [resource]"},{"line_number":337,"context_line":"        else:"},{"line_number":338,"context_line":"            return resource"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def list(self, **kwargs):"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_f379bcfd","line":337,"updated":"2015-04-30 15:36:35.000000000","message":"`else\u0027 can be removed, since we have `return\u0027 at L336","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"},{"author":{"_account_id":14691,"name":"Vitaly Gusev","email":"vgusev@mirantis.com","username":"vgusev"},"change_message_id":"f0f52564e2904091e729f138f63a246550ea070e","unresolved":false,"context_lines":[{"line_number":334,"context_line":"        self.cache[resource.uuid] \u003d resource"},{"line_number":335,"context_line":"        if inlist:"},{"line_number":336,"context_line":"            return [resource]"},{"line_number":337,"context_line":"        else:"},{"line_number":338,"context_line":"            return resource"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def list(self, **kwargs):"}],"source_content_type":"text/x-python","patch_set":14,"id":"3acd31a7_107cca1d","line":337,"in_reply_to":"3acd31a7_f379bcfd","updated":"2015-05-05 12:36:02.000000000","message":"Done","commit_id":"e5d9da495eaab9728d92155d3406b07fac87cf37"}]}