)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e1202373808c4497a8be1ffa77168ee81c836d78","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix to use proper random values"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In several tests, they need int values but use rand_name()."},{"line_number":10,"context_line":"This causes nothing wrong. But when we will change the rand_name() to"},{"line_number":11,"context_line":"return string value, the errors will occur."},{"line_number":12,"context_line":"So this commit fixes it and also fixes minor comment typo."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAS3%2F%2F6xY%3D","line":10,"updated":"2013-09-17 08:06:03.000000000","message":"Now we have found a real bug in test_security_group_rules.py through some reviews, so this message also should be changed.","commit_id":"4adf21cb8d674aff82163c4a426feeda181740f2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8af6fc301f974b9a4cd183429b0c0d2dff6970ec","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix to use proper random values"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In several tests, they need int values but use rand_name()."},{"line_number":10,"context_line":"This causes nothing wrong. But when we will change the rand_name() to"},{"line_number":11,"context_line":"return string value, the errors will occur."},{"line_number":12,"context_line":"So this commit fixes it and also fixes minor comment typo."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAS3%2F%2F6wE%3D","line":10,"in_reply_to":"AAAAS3%2F%2F6xY%3D","updated":"2013-09-17 08:09:12.000000000","message":"OK. You\u0027re right. I\u0027ll update the commit message.","commit_id":"4adf21cb8d674aff82163c4a426feeda181740f2"}],"tempest/api/compute/floating_ips/test_floating_ips_actions.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"6a5f3bca8a0d3a44e9f713c0cc5e6b5fe21ad2c4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import uuid"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.api.compute import base"},{"line_number":21,"context_line":"from tempest.common.utils.data_utils import rand_int_id"},{"line_number":22,"context_line":"from tempest import exceptions"},{"line_number":23,"context_line":"from tempest.test import attr"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FD1Q%3D","line":21,"updated":"2013-10-05 11:53:49.000000000","message":"instead of importing the function, can we just import the module and call it that way. So...\n\nimport tempest.common.utils.data_utils as data_utils\n\nthen data_utils.rand_int_id\n\nHACKING officially discourages symbol import directly because it makes it harder to track down that a function was imported.","commit_id":"c9d860c1491c580dc532d10f74385d95c75dc0b2"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"d708aae4e346c7fcc52edae1c7ce60799ab645bb","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import uuid"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.api.compute import base"},{"line_number":21,"context_line":"from tempest.common.utils.data_utils import rand_int_id"},{"line_number":22,"context_line":"from tempest import exceptions"},{"line_number":23,"context_line":"from tempest.test import attr"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FDq4%3D","line":21,"in_reply_to":"AAAAS3%2F%2FD1Q%3D","updated":"2013-10-05 15:35:15.000000000","message":"Hi Sean, I saw there are many other same usage, should we change them once for all?","commit_id":"c9d860c1491c580dc532d10f74385d95c75dc0b2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"c2eab26ff96cdbcc97af60c53d9055ddb63f531d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import uuid"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.api.compute import base"},{"line_number":21,"context_line":"from tempest.common.utils.data_utils import rand_int_id"},{"line_number":22,"context_line":"from tempest import exceptions"},{"line_number":23,"context_line":"from tempest.test import attr"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAATH%2F%2F%2Fuo%3D","line":21,"in_reply_to":"AAAAS3%2F%2FD1Q%3D","updated":"2013-10-05 22:57:56.000000000","message":"Hi Sean, Lingxian, OK. I will fix to import the module for only 6 fixed files in this patch.\nAnd in many other same usage files, I will fix them in another patch.","commit_id":"c9d860c1491c580dc532d10f74385d95c75dc0b2"}],"tempest/api/compute/floating_ips/test_list_floating_ips.py":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"6693d7c08eddb0a029c79e0e7fdacaea8291ea5d","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        while True:"},{"line_number":93,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":94,"context_line":"            service_available \u003d config.TempestConfig().service_available"},{"line_number":95,"context_line":"            if service_available.neutron:"},{"line_number":96,"context_line":"                non_exist_id \u003d str(uuid.uuid4())"},{"line_number":97,"context_line":"            if non_exist_id not in floating_ip_id:"},{"line_number":98,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAS3%2F%2FGOU%3D","line":95,"updated":"2013-10-04 13:30:23.000000000","message":"could you replace this with \"\"if self.config.service_available.neutron\"\" ?","commit_id":"56568957723cbeb0918643b02232c101bff4e9dd"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e87cf152539939bd10e2c32801ffaf5488ac1dd2","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        while True:"},{"line_number":93,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":94,"context_line":"            service_available \u003d config.TempestConfig().service_available"},{"line_number":95,"context_line":"            if service_available.neutron:"},{"line_number":96,"context_line":"                non_exist_id \u003d str(uuid.uuid4())"},{"line_number":97,"context_line":"            if non_exist_id not in floating_ip_id:"},{"line_number":98,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAS3%2F%2FGFU%3D","line":95,"in_reply_to":"AAAAS3%2F%2FGOU%3D","updated":"2013-10-04 14:13:53.000000000","message":"OK! Thank you for your advice!","commit_id":"56568957723cbeb0918643b02232c101bff4e9dd"}],"tempest/api/compute/security_groups/test_security_group_rules.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e9458d5e24513ade0208949ff47a4dd64643161d","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        # Adding rules to the created Security Group"},{"line_number":142,"context_line":"        parent_group_id \u003d securitygroup[\u0027id\u0027]"},{"line_number":143,"context_line":"        ip_protocol \u003d \u0027tcp\u0027"},{"line_number":144,"context_line":"        from_port \u003d rand_int_id(start\u003d999)"},{"line_number":145,"context_line":"        to_port \u003d 22"},{"line_number":146,"context_line":"        self.addCleanup(self.client.delete_security_group, securitygroup[\u0027id\u0027])"},{"line_number":147,"context_line":"        self.assertRaises(exceptions.BadRequest,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F7A4%3D","line":144,"updated":"2013-09-17 06:51:31.000000000","message":"We need to specify \"end\u003d65535\" for making the cause clear.","commit_id":"f7d5c933edddc6c67bd5f38b5e653fc6cb0e58f0"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"21c3e6ae81e9371cbe384455fe6989f9e421b613","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        # Adding rules to the created Security Group"},{"line_number":142,"context_line":"        parent_group_id \u003d securitygroup[\u0027id\u0027]"},{"line_number":143,"context_line":"        ip_protocol \u003d \u0027tcp\u0027"},{"line_number":144,"context_line":"        from_port \u003d rand_int_id(start\u003d999)"},{"line_number":145,"context_line":"        to_port \u003d 22"},{"line_number":146,"context_line":"        self.addCleanup(self.client.delete_security_group, securitygroup[\u0027id\u0027])"},{"line_number":147,"context_line":"        self.assertRaises(exceptions.BadRequest,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F60M%3D","line":144,"in_reply_to":"AAAAS3%2F%2F7A4%3D","updated":"2013-09-17 07:57:51.000000000","message":"Done","commit_id":"f7d5c933edddc6c67bd5f38b5e653fc6cb0e58f0"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"21c3e6ae81e9371cbe384455fe6989f9e421b613","unresolved":false,"context_lines":[{"line_number":151,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":152,"context_line":"    def test_security_group_rules_create_with_invalid_to_port(self):"},{"line_number":153,"context_line":"        # Negative test: Creation of Security Group rule should FAIL"},{"line_number":154,"context_line":"        # with invalid from_port"},{"line_number":155,"context_line":"        # Creating a Security Group to add rule to it"},{"line_number":156,"context_line":"        s_name \u003d rand_name(\u0027securitygroup-\u0027)"},{"line_number":157,"context_line":"        s_description \u003d rand_name(\u0027description-\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F6zY%3D","line":154,"updated":"2013-09-17 07:57:51.000000000","message":"s/from_port/to_port/","commit_id":"f7d5c933edddc6c67bd5f38b5e653fc6cb0e58f0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e9458d5e24513ade0208949ff47a4dd64643161d","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        parent_group_id \u003d securitygroup[\u0027id\u0027]"},{"line_number":162,"context_line":"        ip_protocol \u003d \u0027tcp\u0027"},{"line_number":163,"context_line":"        from_port \u003d 22"},{"line_number":164,"context_line":"        to_port \u003d rand_int_id(start\u003d999)"},{"line_number":165,"context_line":"        self.addCleanup(self.client.delete_security_group, securitygroup[\u0027id\u0027])"},{"line_number":166,"context_line":"        self.assertRaises(exceptions.BadRequest,"},{"line_number":167,"context_line":"                          self.client.create_security_group_rule,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F7A0%3D","line":164,"updated":"2013-09-17 06:51:31.000000000","message":"ditto","commit_id":"f7d5c933edddc6c67bd5f38b5e653fc6cb0e58f0"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"21c3e6ae81e9371cbe384455fe6989f9e421b613","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        parent_group_id \u003d securitygroup[\u0027id\u0027]"},{"line_number":162,"context_line":"        ip_protocol \u003d \u0027tcp\u0027"},{"line_number":163,"context_line":"        from_port \u003d 22"},{"line_number":164,"context_line":"        to_port \u003d rand_int_id(start\u003d999)"},{"line_number":165,"context_line":"        self.addCleanup(self.client.delete_security_group, securitygroup[\u0027id\u0027])"},{"line_number":166,"context_line":"        self.assertRaises(exceptions.BadRequest,"},{"line_number":167,"context_line":"                          self.client.create_security_group_rule,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F6zE%3D","line":164,"in_reply_to":"AAAAS3%2F%2F7A0%3D","updated":"2013-09-17 07:57:51.000000000","message":"I noticed this value should be start\u003d65536.\nBecause 999 to 65535 is valid value.","commit_id":"f7d5c933edddc6c67bd5f38b5e653fc6cb0e58f0"}],"tempest/api/compute/volumes/test_volumes_negative.py":[{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"8c88723dec1c2435aedd0e545add78b4f69df079","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        for i in range(len(body)):"},{"line_number":43,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":44,"context_line":"        while True:"},{"line_number":45,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":46,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":47,"context_line":"                break"},{"line_number":48,"context_line":"        # Trying to GET a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKQU%3D","line":45,"updated":"2013-10-03 07:48:11.000000000","message":"why not just use uuid.uuid4() directly?","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0e2021ac2bf29d74fc33eb9e2cadd53191ed2e15","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        for i in range(len(body)):"},{"line_number":43,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":44,"context_line":"        while True:"},{"line_number":45,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":46,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":47,"context_line":"                break"},{"line_number":48,"context_line":"        # Trying to GET a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKC8%3D","line":45,"in_reply_to":"AAAAS3%2F%2FKPo%3D","updated":"2013-10-03 10:16:47.000000000","message":"Done","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"ab668754ea08e7ca6aa03dd1e7906190ea1b1d24","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        for i in range(len(body)):"},{"line_number":43,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":44,"context_line":"        while True:"},{"line_number":45,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":46,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":47,"context_line":"                break"},{"line_number":48,"context_line":"        # Trying to GET a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKPo%3D","line":45,"in_reply_to":"AAAAS3%2F%2FKQU%3D","updated":"2013-10-03 07:57:40.000000000","message":"As discussed in patch set5 this should be a uuid, not an number. A number will cause a failure anyway","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0e2021ac2bf29d74fc33eb9e2cadd53191ed2e15","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        for i in range(len(body)):"},{"line_number":43,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":44,"context_line":"        while True:"},{"line_number":45,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":46,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":47,"context_line":"                break"},{"line_number":48,"context_line":"        # Trying to GET a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKDA%3D","line":45,"in_reply_to":"AAAAS3%2F%2FKQU%3D","updated":"2013-10-03 10:16:47.000000000","message":"Done","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"8c88723dec1c2435aedd0e545add78b4f69df079","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        for i in range(len(body)):"},{"line_number":59,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":60,"context_line":"        while True:"},{"line_number":61,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":62,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":63,"context_line":"                break"},{"line_number":64,"context_line":"        # Trying to DELETE a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKQQ%3D","line":61,"updated":"2013-10-03 07:48:11.000000000","message":"same comments","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0e2021ac2bf29d74fc33eb9e2cadd53191ed2e15","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        for i in range(len(body)):"},{"line_number":59,"context_line":"            volume_id_list.append(body[i][\u0027id\u0027])"},{"line_number":60,"context_line":"        while True:"},{"line_number":61,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":62,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":63,"context_line":"                break"},{"line_number":64,"context_line":"        # Trying to DELETE a non existant volume"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2FKC4%3D","line":61,"in_reply_to":"AAAAS3%2F%2FKQQ%3D","updated":"2013-10-03 10:16:47.000000000","message":"Done","commit_id":"f9bf23c56929b755c5936867cf2deb3237d5af51"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1c106a4159f958cdb601d11b5020eae4c3bfdced","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                break"},{"line_number":49,"context_line":"        # Trying to GET a non existant volume"},{"line_number":50,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.get_volume,"},{"line_number":51,"context_line":"                          non_exist_id)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":54,"context_line":"    def test_volume_delete_nonexistant_volume_id(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FHNU%3D","line":51,"updated":"2013-10-04 06:04:28.000000000","message":"why don\u0027t just specify str(uuid.uuid4()) as get_volume()\u0027s argument at line 50?\nThe return value of uuid.uuid4() is unique.","commit_id":"da0ab5814acd33cd70a59487f32898f4cae6d476"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"ca149b80b3b8a5c3f84686b1cf8dd00997d87584","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                break"},{"line_number":49,"context_line":"        # Trying to GET a non existant volume"},{"line_number":50,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.get_volume,"},{"line_number":51,"context_line":"                          non_exist_id)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":54,"context_line":"    def test_volume_delete_nonexistant_volume_id(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FGw0%3D","line":51,"in_reply_to":"AAAAS3%2F%2FHNU%3D","updated":"2013-10-04 09:56:38.000000000","message":"OK. You\u0027re right :-)\nI\u0027ll fix these.\nThanks!","commit_id":"da0ab5814acd33cd70a59487f32898f4cae6d476"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1c106a4159f958cdb601d11b5020eae4c3bfdced","unresolved":false,"context_lines":[{"line_number":64,"context_line":"                break"},{"line_number":65,"context_line":"        # Trying to DELETE a non existant volume"},{"line_number":66,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.delete_volume,"},{"line_number":67,"context_line":"                          non_exist_id)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":70,"context_line":"    def test_create_volume_with_invalid_size(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FHNM%3D","line":67,"updated":"2013-10-04 06:04:28.000000000","message":"ditto.","commit_id":"da0ab5814acd33cd70a59487f32898f4cae6d476"}],"tempest/api/volume/test_volumes_negative.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"a328d1d7bd87856dd24d5c56fff4118f392d37d1","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        for i in range(len(volumes)):"},{"line_number":40,"context_line":"            volume_id_list.append(volumes[i][\u0027id\u0027])"},{"line_number":41,"context_line":"        while True:"},{"line_number":42,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":43,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":44,"context_line":"                break"},{"line_number":45,"context_line":"        # Trying to Get a non-existent volume"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAS3%2F%2F15s%3D","line":42,"updated":"2013-09-18 12:15:09.000000000","message":"Thanks to Guilio for pointing out that this is not correct as its a uuid, not an int. Now the original rand_name is not correct either. Either of them might get filtered in the future as a bad request rather than a not found.\n\nShould probably be a random uuid?","commit_id":"86bd2b54aea66cc14c086dcdf0ef7b5861ba4aac"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2a3ce884862ca1b324a937751660d2077bbc9b19","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        for i in range(len(volumes)):"},{"line_number":40,"context_line":"            volume_id_list.append(volumes[i][\u0027id\u0027])"},{"line_number":41,"context_line":"        while True:"},{"line_number":42,"context_line":"            non_exist_id \u003d rand_int_id(start\u003d999)"},{"line_number":43,"context_line":"            if non_exist_id not in volume_id_list:"},{"line_number":44,"context_line":"                break"},{"line_number":45,"context_line":"        # Trying to Get a non-existent volume"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAS3%2F%2Fzl8%3D","line":42,"in_reply_to":"AAAAS3%2F%2F15s%3D","updated":"2013-09-20 04:20:43.000000000","message":"Sure. This should be a randome uuid like below.\nI will fix it.","commit_id":"86bd2b54aea66cc14c086dcdf0ef7b5861ba4aac"}]}