)]}'
{"tempest/api/telemetry/admin/test_telemetry_samples_api.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"14d26aaca99e290fe04df3cce2d625d3d9f98943","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    @test.idempotent_id(\u0027dd900d0e-e63e-49f9-a99f-3f1c71ef00e6\u0027)"},{"line_number":28,"context_line":"    def test_create_get_and_list_samples(self):"},{"line_number":29,"context_line":"        # Get meter and resource details"},{"line_number":30,"context_line":"        meters \u003d self.client.list_meters()"},{"line_number":31,"context_line":"        volume \u003d 2"},{"line_number":32,"context_line":"        sample_list \u003d self.client.add_sample("},{"line_number":33,"context_line":"            meter_name\u003dmeters[0][\u0027name\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a4dcd0f_486585ac","line":30,"updated":"2015-08-13 12:58:06.000000000","message":"add a limit\u003d1 here.\nOr create a sample from scratch. You don\"t have to copy/mimic an existing sample. You can come up with your own meter_name and resource_id.","commit_id":"ced1c7c9e6eb33185e9672b9d5e4ec7133086053"}],"tempest/api/telemetry/base.py":[{"author":{"_account_id":8001,"name":"John Warren","email":"jswarren@us.ibm.com","username":"jswarren"},"change_message_id":"3eac40376bd58b50c4d908dd2e260ca2e605ffa1","unresolved":false,"context_lines":[{"line_number":127,"context_line":"            \u0027database within %d seconds\u0027 % (metric, query,"},{"line_number":128,"context_line":"                                            CONF.compute.build_timeout))"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def wait_for_samples(self, message_id, meter_name):"},{"line_number":131,"context_line":"        query \u003d (\u0027message_id\u0027, \u0027eq\u0027, message_id)"},{"line_number":132,"context_line":"        timeout \u003d CONF.compute.build_timeout"},{"line_number":133,"context_line":"        start \u003d timeutils.utcnow()"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_7b70addc","line":130,"updated":"2015-08-19 20:50:02.000000000","message":"Could you add documentation to explain the purpose of this method and what the parameters are used for?  I noticed that there\u0027s already a \"await_samples\" method and \"await\" and \"wait for\" are synonyms, so it would be good to explain the purpose of the method so it is clear how it differs from the other method.","commit_id":"fafb04e3d3e48f0ecfa9ecd6f894e49542e7bf7d"}],"tempest/api/telemetry/test_telemetry_samples_api.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c66973b8fdb6a57cfcfeaa4c5827bc00055fca4b","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # List Samples and Verify the created sample in the list"},{"line_number":57,"context_line":"        samples \u003d self.client.list_samples(meter_name\u003dmeter_name)"},{"line_number":58,"context_line":"        self.assertIn(get_sample[\u0027id\u0027],"},{"line_number":59,"context_line":"                      map(lambda x: x[\u0027message_id\u0027], samples))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_e1c46341","line":59,"range":{"start_line":59,"start_character":20,"end_line":59,"end_character":25},"updated":"2015-08-18 11:20:25.000000000","message":"Use a list comprehension here:\n  [ x[\u0027message_id\u0027] for x in samples ]\n\nshould work","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"},{"author":{"_account_id":2222,"name":"meenakshi-m","username":"meenakshi-m"},"change_message_id":"2b359e6c355e6147632dbb60a442f54c605d1e8f","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # List Samples and Verify the created sample in the list"},{"line_number":57,"context_line":"        samples \u003d self.client.list_samples(meter_name\u003dmeter_name)"},{"line_number":58,"context_line":"        self.assertIn(get_sample[\u0027id\u0027],"},{"line_number":59,"context_line":"                      map(lambda x: x[\u0027message_id\u0027], samples))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_f21cfec2","line":59,"in_reply_to":"fa1b9901_e1c46341","updated":"2015-08-19 10:51:04.000000000","message":"Done","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"}],"tempest/services/telemetry/json/telemetry_client.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c66973b8fdb6a57cfcfeaa4c5827bc00055fca4b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        for key in kwargs:"},{"line_number":39,"context_line":"            sample[key] \u003d kwargs[key]"},{"line_number":40,"context_line":"        sample_list \u003d []"},{"line_number":41,"context_line":"        sample_list.append(sample)"},{"line_number":42,"context_line":"        return sample_list"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def create_sample(self, meter_name, sample_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_2196db4c","line":41,"updated":"2015-08-18 11:20:25.000000000","message":"Just initialize like this:\n   sample_list \u003d [sample]","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"},{"author":{"_account_id":2222,"name":"meenakshi-m","username":"meenakshi-m"},"change_message_id":"2b359e6c355e6147632dbb60a442f54c605d1e8f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        for key in kwargs:"},{"line_number":39,"context_line":"            sample[key] \u003d kwargs[key]"},{"line_number":40,"context_line":"        sample_list \u003d []"},{"line_number":41,"context_line":"        sample_list.append(sample)"},{"line_number":42,"context_line":"        return sample_list"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def create_sample(self, meter_name, sample_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_f2739e13","line":41,"in_reply_to":"fa1b9901_2196db4c","updated":"2015-08-19 10:51:04.000000000","message":"Done","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c66973b8fdb6a57cfcfeaa4c5827bc00055fca4b","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        uri \u003d \"%s/meters/%s\" % (self.uri_prefix, meter_name)"},{"line_number":46,"context_line":"        body \u003d self.serialize(sample_list)"},{"line_number":47,"context_line":"        resp, body \u003d self.post(uri, body)"},{"line_number":48,"context_line":"        self.expected_success([200, 201], resp.status)"},{"line_number":49,"context_line":"        body \u003d self.deserialize(body)"},{"line_number":50,"context_line":"        return service_client.ResponseBodyList(resp, body)"},{"line_number":51,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_8187a70e","line":48,"updated":"2015-08-18 11:20:25.000000000","message":"Do you have a link/a reference where it\u0027s written that 201 is an accepted status code ?","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"},{"author":{"_account_id":2222,"name":"meenakshi-m","username":"meenakshi-m"},"change_message_id":"2b359e6c355e6147632dbb60a442f54c605d1e8f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        uri \u003d \"%s/meters/%s\" % (self.uri_prefix, meter_name)"},{"line_number":46,"context_line":"        body \u003d self.serialize(sample_list)"},{"line_number":47,"context_line":"        resp, body \u003d self.post(uri, body)"},{"line_number":48,"context_line":"        self.expected_success([200, 201], resp.status)"},{"line_number":49,"context_line":"        body \u003d self.deserialize(body)"},{"line_number":50,"context_line":"        return service_client.ResponseBodyList(resp, body)"},{"line_number":51,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_526e0a02","line":48,"in_reply_to":"fa1b9901_8187a70e","updated":"2015-08-19 10:51:04.000000000","message":"Hi Jordan,\n\nI have verified through command line, for current Openstack Liberty release the response status is returned as \u0027201\u0027, where as for Juno release the response code returned as \u0027200\u0027.","commit_id":"5f9756659ffd4d9cd219b0c3dde7e1a1f9dc1a5a"}]}