)]}'
{"rally-jobs/rally-monasca.yaml":[{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":52,"context_line":"         - \"monasca-user\""},{"line_number":53,"context_line":"        monasca_metrics:"},{"line_number":54,"context_line":"          \"dimensions\":"},{"line_number":55,"context_line":"              \"region\": \"RegionOne\""},{"line_number":56,"context_line":"              \"service\": \"identity\""},{"line_number":57,"context_line":"              \"hostname\": \"fake_host\""},{"line_number":58,"context_line":"          \"metrics_per_tenant\": 10"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3a461143_9bcd7433","line":55,"range":{"start_line":55,"start_character":14,"end_line":55,"end_character":15},"updated":"2017-01-31 09:27:33.000000000","message":"Quotas look strange here. They are not required in YAML keys.","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"}],"rally/plugins/openstack/context/monasca/metrics.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":87,"context_line":"            new_metric[\"name\"] \u003d self.config[\"name\"]"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"        if \"dimensions\" in self.config:"},{"line_number":90,"context_line":"            new_metric[\"dimension\"] \u003d self.config[\"dimensions\"]"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        for user, tenant_id in rutils.iterate_per_tenants("},{"line_number":93,"context_line":"                self.context[\"users\"]):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_f6fc7540","line":90,"updated":"2016-11-23 18:08:13.000000000","message":"isn\u0027t here the typo? https://github.com/openstack/python-monascaclient/blob/master/monascaclient/v2_0/shell.py#L78","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"abd6532fde2da52fed1b701c73012789c665156f","unresolved":false,"context_lines":[{"line_number":87,"context_line":"            new_metric[\"name\"] \u003d self.config[\"name\"]"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"        if \"dimensions\" in self.config:"},{"line_number":90,"context_line":"            new_metric[\"dimension\"] \u003d self.config[\"dimensions\"]"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        for user, tenant_id in rutils.iterate_per_tenants("},{"line_number":93,"context_line":"                self.context[\"users\"]):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_47b8acc8","line":90,"in_reply_to":"5a74a57a_f6fc7540","updated":"2016-11-26 00:53:02.000000000","message":"fixed in next patch","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"}],"rally/plugins/openstack/scenarios/monasca/utils.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":35,"context_line":"benchmark_group \u003d cfg.OptGroup(name\u003d\"benchmark\", title\u003d\"benchmark options\")"},{"line_number":36,"context_line":"CONF.register_opts(MONASCA_BENCHMARK_OPTS, group\u003dbenchmark_group)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"cur_time \u003d int(time.time())"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"class MonascaScenario(scenario.OpenStackScenario):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_9649818e","line":38,"updated":"2016-11-23 18:08:13.000000000","message":"noooo, use directly time.time() when its needed","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"abd6532fde2da52fed1b701c73012789c665156f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"benchmark_group \u003d cfg.OptGroup(name\u003d\"benchmark\", title\u003d\"benchmark options\")"},{"line_number":36,"context_line":"CONF.register_opts(MONASCA_BENCHMARK_OPTS, group\u003dbenchmark_group)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"cur_time \u003d int(time.time())"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"class MonascaScenario(scenario.OpenStackScenario):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_c77e7c18","line":38,"in_reply_to":"5a74a57a_9649818e","updated":"2016-11-26 00:53:02.000000000","message":"fixed in next  patch","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        \"\"\"list metric statistics."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        :param name: metric name to be filtered"},{"line_number":77,"context_line":"        :param statistics: valid statistics are avg, min, max, sum and count"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        start_time \u003d dt.datetime.fromtimestamp("},{"line_number":80,"context_line":"            cur_time - 1.0).isoformat() + \"Z\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_f11d5f67","line":77,"updated":"2016-11-23 18:08:13.000000000","message":"kwargs, return value aren\u0027t documeted","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"abd6532fde2da52fed1b701c73012789c665156f","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        \"\"\"list metric statistics."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        :param name: metric name to be filtered"},{"line_number":77,"context_line":"        :param statistics: valid statistics are avg, min, max, sum and count"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        start_time \u003d dt.datetime.fromtimestamp("},{"line_number":80,"context_line":"            cur_time - 1.0).isoformat() + \"Z\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_e77b7826","line":77,"in_reply_to":"5a74a57a_f11d5f67","updated":"2016-11-26 00:53:02.000000000","message":"fixed in next patch","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :param name: metric name to be filtered"},{"line_number":77,"context_line":"        :param statistics: valid statistics are avg, min, max, sum and count"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        start_time \u003d dt.datetime.fromtimestamp("},{"line_number":80,"context_line":"            cur_time - 1.0).isoformat() + \"Z\""},{"line_number":81,"context_line":"        end_time \u003d dt.datetime.fromtimestamp("},{"line_number":82,"context_line":"            cur_time + 1000 * 3).isoformat() + \"Z\""},{"line_number":83,"context_line":"        kwargs.update({\"name\": name,"},{"line_number":84,"context_line":"                       \"statistics\": statistics,"},{"line_number":85,"context_line":"                       \"start_time\": start_time,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_f1f1dfce","line":82,"range":{"start_line":79,"start_character":2,"end_line":82,"end_character":50},"updated":"2016-11-23 18:08:13.000000000","message":"why are they not configured through arguments? What if I\u0027m not specify them? - Will we get all statistics?","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"abd6532fde2da52fed1b701c73012789c665156f","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :param name: metric name to be filtered"},{"line_number":77,"context_line":"        :param statistics: valid statistics are avg, min, max, sum and count"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        start_time \u003d dt.datetime.fromtimestamp("},{"line_number":80,"context_line":"            cur_time - 1.0).isoformat() + \"Z\""},{"line_number":81,"context_line":"        end_time \u003d dt.datetime.fromtimestamp("},{"line_number":82,"context_line":"            cur_time + 1000 * 3).isoformat() + \"Z\""},{"line_number":83,"context_line":"        kwargs.update({\"name\": name,"},{"line_number":84,"context_line":"                       \"statistics\": statistics,"},{"line_number":85,"context_line":"                       \"start_time\": start_time,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_e7a29898","line":82,"range":{"start_line":79,"start_character":2,"end_line":82,"end_character":50},"in_reply_to":"5a74a57a_f1f1dfce","updated":"2016-11-26 00:53:02.000000000","message":"Start time is required and end time is optional. If start time is not specified, monasca api returns error.\n\nStart time is not configured through arguments and this is because there are no metrics auto created in devstack. In devstack, metrics are created in context right before get metric statistics call.","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                       \"statistics\": statistics,"},{"line_number":85,"context_line":"                       \"start_time\": start_time,"},{"line_number":86,"context_line":"                       \"end_time\": end_time,"},{"line_number":87,"context_line":"                       \"merge_metrics\": True"},{"line_number":88,"context_line":"                       })"},{"line_number":89,"context_line":"        if \"\u0027dimensions\u0027\" in kwargs:"},{"line_number":90,"context_line":"            kwargs.update({\"dimensions\": {"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_f190ff97","line":87,"updated":"2016-11-23 18:08:13.000000000","message":"What is this hard code means?","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"abd6532fde2da52fed1b701c73012789c665156f","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                       \"statistics\": statistics,"},{"line_number":85,"context_line":"                       \"start_time\": start_time,"},{"line_number":86,"context_line":"                       \"end_time\": end_time,"},{"line_number":87,"context_line":"                       \"merge_metrics\": True"},{"line_number":88,"context_line":"                       })"},{"line_number":89,"context_line":"        if \"\u0027dimensions\u0027\" in kwargs:"},{"line_number":90,"context_line":"            kwargs.update({\"dimensions\": {"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_27a9f07b","line":87,"in_reply_to":"5a74a57a_f190ff97","updated":"2016-11-26 00:53:02.000000000","message":"This flag is to allow multiple metrics to be combined into a single list of statistics","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":35,"context_line":"benchmark_group \u003d cfg.OptGroup(name\u003d\"benchmark\", title\u003d\"benchmark options\")"},{"line_number":36,"context_line":"CONF.register_opts(MONASCA_BENCHMARK_OPTS, group\u003dbenchmark_group)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"cur_time \u003d time.time()"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"class MonascaScenario(scenario.OpenStackScenario):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a461143_7eec362a","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":22},"updated":"2017-01-31 09:27:33.000000000","message":"Why do we need it right here in the top level of a module?\n\nIt actually saves time when this module was loaded not a current time.","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":79,"context_line":"                       dimensions, end_time"},{"line_number":80,"context_line":"        \"\"\""},{"line_number":81,"context_line":"        start_time \u003d dt.datetime.fromtimestamp("},{"line_number":82,"context_line":"            cur_time - 1.0).isoformat() + \"Z\""},{"line_number":83,"context_line":"        end_time \u003d dt.datetime.fromtimestamp("},{"line_number":84,"context_line":"            cur_time + 1000 * 3).isoformat() + \"Z\""},{"line_number":85,"context_line":"        kwargs.update({\"name\": name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a461143_de5a8af1","line":82,"range":{"start_line":82,"start_character":12,"end_line":82,"end_character":20},"updated":"2017-01-31 09:27:33.000000000","message":"curr_time should be initialized here instead of module level.","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                       \"end_time\": end_time,"},{"line_number":89,"context_line":"                       \"merge_metrics\": True"},{"line_number":90,"context_line":"                       })"},{"line_number":91,"context_line":"        if \"\u0027dimensions\u0027\" in kwargs:"},{"line_number":92,"context_line":"            kwargs.update({\"dimensions\": {"},{"line_number":93,"context_line":"                           \"region\": self.region,"},{"line_number":94,"context_line":"                           \"service\": self.service,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a461143_3b2f20a2","line":91,"range":{"start_line":91,"start_character":11,"end_line":91,"end_character":25},"updated":"2017-01-31 09:27:33.000000000","message":"Why is it double quoted?","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"}],"samples/tasks/scenarios/monasca/list-statistics.json":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a448feecee9ed1729c0970986b90163a93095caf","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          \"service\": \"monitoring\","},{"line_number":33,"context_line":"          \"hostname\": \"fake_host\""},{"line_number":34,"context_line":"        }"},{"line_number":35,"context_line":"      }"},{"line_number":36,"context_line":"    }"},{"line_number":37,"context_line":"  ]"},{"line_number":38,"context_line":"}"}],"source_content_type":"application/json","patch_set":2,"id":"5a74a57a_91b98bec","line":35,"updated":"2016-11-23 18:08:13.000000000","message":"please add sla section","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":16558,"name":"Yan Ning","email":"yan.ning@charter.com","username":"ningyan"},"change_message_id":"17e3acc7ddd9c8b6e1a0d1345f9ebbf0b8a0ac71","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          \"service\": \"monitoring\","},{"line_number":33,"context_line":"          \"hostname\": \"fake_host\""},{"line_number":34,"context_line":"        }"},{"line_number":35,"context_line":"      }"},{"line_number":36,"context_line":"    }"},{"line_number":37,"context_line":"  ]"},{"line_number":38,"context_line":"}"}],"source_content_type":"application/json","patch_set":2,"id":"5a74a57a_87ae447f","line":35,"in_reply_to":"5a74a57a_91b98bec","updated":"2016-11-26 00:55:34.000000000","message":"Isn\u0027t sla section only in rally-monasca.yaml ?","commit_id":"2715a2b7ad947be81764e5304cba0305ce9b79cf"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"  \"MonascaMetrics.list_statistics\": ["},{"line_number":3,"context_line":"    {"},{"line_number":4,"context_line":"      \"runner\": {"},{"line_number":5,"context_line":"        \"type\": \"constant\","}],"source_content_type":"application/json","patch_set":3,"id":"3a461143_1e3ed227","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":2},"updated":"2017-01-31 09:27:33.000000000","message":"Reindent this file with 4 spaces.","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"}],"samples/tasks/scenarios/monasca/list-statistics.yaml":[{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"1be851d2edf7b976e94c7afbf0ed7a7b58f89014","unresolved":false,"context_lines":[{"line_number":12,"context_line":"        roles:"},{"line_number":13,"context_line":"         - \"monasca-user\""},{"line_number":14,"context_line":"        monasca_metrics:"},{"line_number":15,"context_line":"          \"name\": \"host_alive_status\""},{"line_number":16,"context_line":"          \"dimensions\":"},{"line_number":17,"context_line":"              \"region\": \"RegionOne\""},{"line_number":18,"context_line":"              \"service\": \"monitoring\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3a461143_7e7c36ec","line":15,"range":{"start_line":15,"start_character":10,"end_line":15,"end_character":11},"updated":"2017-01-31 09:27:33.000000000","message":"ditto","commit_id":"46ae06cf15945a23fb8b1974a45e31f53b112deb"}]}