)]}'
{"tempest/api/network/test_load_balancer.py":[{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"204bf5151554ca509a10a07d919b9d152bc1a895","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    def test_create_vip_with_address(self):"},{"line_number":135,"context_line":"        # Create VIP specifying address"},{"line_number":136,"context_line":"        name \u003d data_utils.rand_name(\u0027vip-\u0027)"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_pool("},{"line_number":138,"context_line":"            name\u003ddata_utils.rand_name(\"pool-\"),"},{"line_number":139,"context_line":"            lb_method\u003d\u0027ROUND_ROBIN\u0027,"},{"line_number":140,"context_line":"            protocol\u003d\u0027HTTP\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2FSqA%3D","line":137,"updated":"2014-01-15 12:23:17.000000000","message":"Why do you create new pool instead of usage self.pool?","commit_id":"39be40ee11841397818037bd224ad0d95ca0f591"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"bc2fa365c8a6647621fc5deea3eb98c6deabe68e","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    def test_create_vip_with_address(self):"},{"line_number":135,"context_line":"        # Create VIP specifying address"},{"line_number":136,"context_line":"        name \u003d data_utils.rand_name(\u0027vip-\u0027)"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_pool("},{"line_number":138,"context_line":"            name\u003ddata_utils.rand_name(\"pool-\"),"},{"line_number":139,"context_line":"            lb_method\u003d\u0027ROUND_ROBIN\u0027,"},{"line_number":140,"context_line":"            protocol\u003d\u0027HTTP\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2FOCE%3D","line":137,"in_reply_to":"AAAAUH%2F%2FSqA%3D","updated":"2014-01-16 08:56:44.000000000","message":"That\u0027s because self.pool already has a VIP associated with it (self.vip). So we can\u0027t associate it with another one.","commit_id":"39be40ee11841397818037bd224ad0d95ca0f591"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"204bf5151554ca509a10a07d919b9d152bc1a895","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":156,"context_line":"        self.assertEqual(address, vip[\u0027address\u0027])"},{"line_number":157,"context_line":"        self.client.delete_vip(vip[\u0027id\u0027])"},{"line_number":158,"context_line":"        self.client.delete_pool(pool[\u0027id\u0027])"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":161,"context_line":"    def test_update_vip_session_persistence(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2FSrE%3D","line":158,"updated":"2014-01-15 12:23:17.000000000","message":"It is better to use addCleanup, I think.","commit_id":"39be40ee11841397818037bd224ad0d95ca0f591"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"bc2fa365c8a6647621fc5deea3eb98c6deabe68e","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":156,"context_line":"        self.assertEqual(address, vip[\u0027address\u0027])"},{"line_number":157,"context_line":"        self.client.delete_vip(vip[\u0027id\u0027])"},{"line_number":158,"context_line":"        self.client.delete_pool(pool[\u0027id\u0027])"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":161,"context_line":"    def test_update_vip_session_persistence(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2FOCc%3D","line":158,"in_reply_to":"AAAAUH%2F%2FSrE%3D","updated":"2014-01-16 08:56:44.000000000","message":"Done","commit_id":"39be40ee11841397818037bd224ad0d95ca0f591"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"3aa49854bd5891d3f636173b12d3dc4a10b32430","unresolved":false,"context_lines":[{"line_number":322,"context_line":"        resp, body \u003d self.admin_client.create_pool("},{"line_number":323,"context_line":"            name\u003ddata_utils.rand_name(\u0027pool-\u0027), lb_method\u003d\"ROUND_ROBIN\","},{"line_number":324,"context_line":"            protocol\u003d\"HTTP\", subnet_id\u003dself.subnet[\u0027id\u0027],"},{"line_number":325,"context_line":"            tenant_id\u003d\u00277688e351a2f24be7b7d6b592a5bbe64b\u0027)"},{"line_number":326,"context_line":"        pool \u003d body[\u0027pool\u0027]"},{"line_number":327,"context_line":"        resp, body \u003d self.admin_client.create_vip(name\u003dname,"},{"line_number":328,"context_line":"                                                  protocol\u003d\"HTTP\","}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2Bq9s%3D","line":325,"updated":"2014-01-23 11:17:12.000000000","message":"Take a look https://review.openstack.org/#/c/65943/13/tempest/api/network/test_load_balancer.py class LoadBalancerAdminJSON work with tenant_id and do the same here.","commit_id":"1f2352c49af44668fd7ce03d17699ac37949b87e"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"6af904a678e6bcaa68ba7af96d6b076602e65b12","unresolved":false,"context_lines":[{"line_number":322,"context_line":"        resp, body \u003d self.admin_client.create_pool("},{"line_number":323,"context_line":"            name\u003ddata_utils.rand_name(\u0027pool-\u0027), lb_method\u003d\"ROUND_ROBIN\","},{"line_number":324,"context_line":"            protocol\u003d\"HTTP\", subnet_id\u003dself.subnet[\u0027id\u0027],"},{"line_number":325,"context_line":"            tenant_id\u003d\u00277688e351a2f24be7b7d6b592a5bbe64b\u0027)"},{"line_number":326,"context_line":"        pool \u003d body[\u0027pool\u0027]"},{"line_number":327,"context_line":"        resp, body \u003d self.admin_client.create_vip(name\u003dname,"},{"line_number":328,"context_line":"                                                  protocol\u003d\"HTTP\","}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2Bqvw%3D","line":325,"in_reply_to":"AAAAUH%2F%2Bq9s%3D","updated":"2014-01-23 11:34:54.000000000","message":"Thank you, I will fix it. I was thinking of doing it this way from the very beginning:)","commit_id":"1f2352c49af44668fd7ce03d17699ac37949b87e"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b2f41c48bb71ba4a2352f77f5a29a6f98e3760f5","unresolved":false,"context_lines":[{"line_number":138,"context_line":"            name\u003ddata_utils.rand_name(\"pool-\"),"},{"line_number":139,"context_line":"            lb_method\u003d\u0027ROUND_ROBIN\u0027,"},{"line_number":140,"context_line":"            protocol\u003d\u0027HTTP\u0027,"},{"line_number":141,"context_line":"            subnet_id\u003dself.subnet[\u0027id\u0027])"},{"line_number":142,"context_line":"        pool \u003d body[\u0027pool\u0027]"},{"line_number":143,"context_line":"        self.addCleanup(self.client.delete_pool, pool[\u0027id\u0027])"},{"line_number":144,"context_line":"        address \u003d self.subnet[\u0027allocation_pools\u0027][0][\u0027end\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAUH%2F%2BhWA%3D","line":141,"updated":"2014-01-24 20:48:15.000000000","message":"It is better if you add self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027]) after pool creation. If the api fails to create the pool, we want to report just that, instead of failing in the following line when accessing an empty dictionary: pool \u003d body[\u0027pool\u0027] or when deleting the pool during clean-up","commit_id":"57b5e7dea12d16ab039311dec4d6b37364482bb5"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b2f41c48bb71ba4a2352f77f5a29a6f98e3760f5","unresolved":false,"context_lines":[{"line_number":148,"context_line":"                                            subnet_id\u003dself.subnet[\u0027id\u0027],"},{"line_number":149,"context_line":"                                            pool_id\u003dpool[\u0027id\u0027],"},{"line_number":150,"context_line":"                                            address\u003daddress)"},{"line_number":151,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":152,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":153,"context_line":"        vip_id \u003d vip[\u0027id\u0027]"},{"line_number":154,"context_line":"        resp, body \u003d self.client.show_vip(vip_id)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAUH%2F%2BhUI%3D","line":151,"updated":"2014-01-24 20:48:15.000000000","message":"Please move call addCleanup in line 157 to this point. This way, we will cleanup the vip regardless of what happens to asserts below","commit_id":"57b5e7dea12d16ab039311dec4d6b37364482bb5"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b2f41c48bb71ba4a2352f77f5a29a6f98e3760f5","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            name\u003ddata_utils.rand_name(\"pool-\"),"},{"line_number":165,"context_line":"            lb_method\u003d\u0027ROUND_ROBIN\u0027,"},{"line_number":166,"context_line":"            protocol\u003d\u0027HTTP\u0027,"},{"line_number":167,"context_line":"            subnet_id\u003dself.subnet[\u0027id\u0027])"},{"line_number":168,"context_line":"        pool \u003d body[\u0027pool\u0027]"},{"line_number":169,"context_line":"        self.addCleanup(self.client.delete_pool, pool[\u0027id\u0027])"},{"line_number":170,"context_line":"        resp, body \u003d self.client.create_vip(name\u003dname,"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAUH%2F%2BhPk%3D","line":167,"updated":"2014-01-24 20:48:15.000000000","message":"Please see comment to line 141. Same observation here","commit_id":"57b5e7dea12d16ab039311dec4d6b37364482bb5"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b2f41c48bb71ba4a2352f77f5a29a6f98e3760f5","unresolved":false,"context_lines":[{"line_number":172,"context_line":"                                            protocol_port\u003d80,"},{"line_number":173,"context_line":"                                            subnet_id\u003dself.subnet[\u0027id\u0027],"},{"line_number":174,"context_line":"                                            pool_id\u003dpool[\u0027id\u0027])"},{"line_number":175,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":176,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":177,"context_line":"        persistence_type \u003d \"HTTP_COOKIE\""},{"line_number":178,"context_line":"        update_data \u003d {\"session_persistence\": {"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAUH%2F%2BhJk%3D","line":175,"updated":"2014-01-24 20:48:15.000000000","message":"Please see comment to line 151. Same observation here","commit_id":"57b5e7dea12d16ab039311dec4d6b37364482bb5"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b2f41c48bb71ba4a2352f77f5a29a6f98e3760f5","unresolved":false,"context_lines":[{"line_number":328,"context_line":"        resp, body \u003d self.admin_client.create_pool("},{"line_number":329,"context_line":"            name\u003ddata_utils.rand_name(\u0027pool-\u0027), lb_method\u003d\"ROUND_ROBIN\","},{"line_number":330,"context_line":"            protocol\u003d\"HTTP\", subnet_id\u003dself.subnet[\u0027id\u0027],"},{"line_number":331,"context_line":"            tenant_id\u003dtenant_id)"},{"line_number":332,"context_line":"        pool \u003d body[\u0027pool\u0027]"},{"line_number":333,"context_line":"        self.addCleanup(self.admin_client.delete_pool, pool[\u0027id\u0027])"},{"line_number":334,"context_line":"        resp, body \u003d self.admin_client.create_vip(name\u003dname,"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAUH%2F%2BhH8%3D","line":331,"updated":"2014-01-24 20:48:15.000000000","message":"Same observation as in line 141","commit_id":"57b5e7dea12d16ab039311dec4d6b37364482bb5"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8075baf284e851c4e68488950a65bedb614f08c3","unresolved":false,"context_lines":[{"line_number":149,"context_line":"                                            address\u003daddress)"},{"line_number":150,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":151,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":152,"context_line":"        self.addCleanup(self.client.delete_vip, vip[\u0027id\u0027])"},{"line_number":153,"context_line":"        vip_id \u003d vip[\u0027id\u0027]"},{"line_number":154,"context_line":"        resp, body \u003d self.client.show_vip(vip_id)"},{"line_number":155,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAUH%2F91KM%3D","line":152,"updated":"2014-02-03 15:25:41.000000000","message":"I think this should come before the assert, otherwise there will be a resource left around if the assert fails\n\nThe same comment applies to other tests in this module as well","commit_id":"5335437d4a388bfe3bef2048928c8c58331b6192"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8075baf284e851c4e68488950a65bedb614f08c3","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        resp, body \u003d self.client.show_vip(vip_id)"},{"line_number":155,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":156,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":157,"context_line":"        self.assertEqual(address, vip[\u0027address\u0027])"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":160,"context_line":"    def test_update_vip_session_persistence(self):"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAUH%2F91Kw%3D","line":157,"updated":"2014-02-03 15:25:41.000000000","message":"I am happy to see new tests, but I don\u0027t see a compelling reason for adding a new test here. Can\u0027t we just add address as well to test_create_update_delete_vip_pool?\n\nOr are we going to cover some significantly different use case with this test?","commit_id":"5335437d4a388bfe3bef2048928c8c58331b6192"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8075baf284e851c4e68488950a65bedb614f08c3","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        self.assertEqual(address, vip[\u0027address\u0027])"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":160,"context_line":"    def test_update_vip_session_persistence(self):"},{"line_number":161,"context_line":"        name \u003d data_utils.rand_name(\u0027vip-\u0027)"},{"line_number":162,"context_line":"        resp, body \u003d self.client.create_pool("},{"line_number":163,"context_line":"            name\u003ddata_utils.rand_name(\"pool-\"),"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAUH%2F91HE%3D","line":160,"updated":"2014-02-03 15:25:41.000000000","message":"what would be the reason for specifically picking the session_persistence attribute?\n\nIs this covering some use case not covered by test_create_update_delete_pool_vip?","commit_id":"5335437d4a388bfe3bef2048928c8c58331b6192"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8075baf284e851c4e68488950a65bedb614f08c3","unresolved":false,"context_lines":[{"line_number":343,"context_line":"            description\u003dtest_description)"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":346,"context_line":"    def test_create_vip_from_admin_for_another_tenant(self):"},{"line_number":347,"context_line":"        tenant_id \u003d self.tenant[\u0027id\u0027]"},{"line_number":348,"context_line":"        name \u003d data_utils.rand_name(\u0027vip-\u0027)"},{"line_number":349,"context_line":"        resp, body \u003d self.admin_client.create_pool("}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAUH%2F91Gw%3D","line":346,"updated":"2014-02-03 15:25:41.000000000","message":"nit: as_admin","commit_id":"5335437d4a388bfe3bef2048928c8c58331b6192"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8075baf284e851c4e68488950a65bedb614f08c3","unresolved":false,"context_lines":[{"line_number":363,"context_line":"        vip \u003d body[\u0027vip\u0027]"},{"line_number":364,"context_line":"        self.addCleanup(self.admin_client.delete_vip, vip[\u0027id\u0027])"},{"line_number":365,"context_line":"        self.assertIsNotNone(vip[\u0027id\u0027])"},{"line_number":366,"context_line":"        self.assertEqual(tenant_id, vip[\u0027tenant_id\u0027])"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":369,"context_line":"    def test_create_health_monitor_from_admin_for_another_tenant(self):"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAUH%2F91GU%3D","line":366,"updated":"2014-02-03 15:25:41.000000000","message":"I think this test should do a show using the \u0027standard\u0027 client (ie: as tenant_id), and then verify the load balancer is correctly retrieved","commit_id":"5335437d4a388bfe3bef2048928c8c58331b6192"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"d7c48fc76f4ebfa11c1c7dd3e7acedcc76fb640f","unresolved":false,"context_lines":[{"line_number":291,"context_line":"        if not test.is_extension_enabled(\u0027lbaas\u0027, \u0027network\u0027):"},{"line_number":292,"context_line":"            msg \u003d \"lbaas extension not enabled.\""},{"line_number":293,"context_line":"            raise cls.skipException(msg)"},{"line_number":294,"context_line":"        cls.identity_admin_client \u003d cls.admin_manager.identity_client"},{"line_number":295,"context_line":"        test_tenant \u003d data_utils.rand_name(\u0027test_tenant\u0027)"},{"line_number":296,"context_line":"        test_description \u003d data_utils.rand_name(\u0027desc_\u0027)"},{"line_number":297,"context_line":"        _, cls.tenant \u003d cls.identity_admin_client.create_tenant("}],"source_content_type":"text/x-python","patch_set":12,"id":"AAAAUX%2F%2F3TU%3D","line":294,"updated":"2014-02-10 11:09:58.000000000","message":"This need to be rebased. Now os_adm used instead of  admin_manager.","commit_id":"d713c9f626bac6bc85eb22fb8b14b15d769707bc"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"ac66ac76e9c4077429d9e20884d57c8177fb7fce","unresolved":false,"context_lines":[{"line_number":291,"context_line":"        if not test.is_extension_enabled(\u0027lbaas\u0027, \u0027network\u0027):"},{"line_number":292,"context_line":"            msg \u003d \"lbaas extension not enabled.\""},{"line_number":293,"context_line":"            raise cls.skipException(msg)"},{"line_number":294,"context_line":"        cls.identity_admin_client \u003d cls.admin_manager.identity_client"},{"line_number":295,"context_line":"        test_tenant \u003d data_utils.rand_name(\u0027test_tenant\u0027)"},{"line_number":296,"context_line":"        test_description \u003d data_utils.rand_name(\u0027desc_\u0027)"},{"line_number":297,"context_line":"        _, cls.tenant \u003d cls.identity_admin_client.create_tenant("}],"source_content_type":"text/x-python","patch_set":12,"id":"AAAAUX%2F%2F2Ds%3D","line":294,"in_reply_to":"AAAAUX%2F%2F3TU%3D","updated":"2014-02-10 13:21:05.000000000","message":"Done","commit_id":"d713c9f626bac6bc85eb22fb8b14b15d769707bc"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"ebfffabc3e94e486e56d46820ce22d38a6f38330","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        self.assertEqual(hm_type, health_monitor[\u0027type\u0027])"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":239,"context_line":"    def test_update_health_monitor_http_type(self):"},{"line_number":240,"context_line":"        resp, body \u003d self.client.create_health_monitor(delay\u003d4,"},{"line_number":241,"context_line":"                                                       max_retries\u003d3,"},{"line_number":242,"context_line":"                                                       type\u003d\"HTTP\","}],"source_content_type":"text/x-python","patch_set":13,"id":"AAAAUX%2F%2FUp4%3D","line":239,"updated":"2014-02-13 23:52:47.000000000","message":"I think it would be clearer if you call this method: \n\ntest_update_health_monitor_http_method","commit_id":"4ba71b8ff825cc2463191c48b28815d0a721136c"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"2b19278b51a5cf68960002dfca6c1bbfdce9d704","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        self.assertEqual(hm_type, health_monitor[\u0027type\u0027])"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":239,"context_line":"    def test_update_health_monitor_http_type(self):"},{"line_number":240,"context_line":"        resp, body \u003d self.client.create_health_monitor(delay\u003d4,"},{"line_number":241,"context_line":"                                                       max_retries\u003d3,"},{"line_number":242,"context_line":"                                                       type\u003d\"HTTP\","}],"source_content_type":"text/x-python","patch_set":13,"id":"AAAAUX%2F%2FDG4%3D","line":239,"in_reply_to":"AAAAUX%2F%2FUp4%3D","updated":"2014-02-17 13:01:34.000000000","message":"Done","commit_id":"4ba71b8ff825cc2463191c48b28815d0a721136c"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"5e874f4332a2536370c8dd075a40fa02ebe36512","unresolved":false,"context_lines":[{"line_number":302,"context_line":"        if not test.is_extension_enabled(\u0027lbaas\u0027, \u0027network\u0027):"},{"line_number":303,"context_line":"            msg \u003d \"lbaas extension not enabled.\""},{"line_number":304,"context_line":"            raise cls.skipException(msg)"},{"line_number":305,"context_line":"        cls.identity_admin_client \u003d cls.os_adm.identity_client"},{"line_number":306,"context_line":"        test_tenant \u003d data_utils.rand_name(\u0027test_tenant\u0027)"},{"line_number":307,"context_line":"        test_description \u003d data_utils.rand_name(\u0027desc_\u0027)"},{"line_number":308,"context_line":"        _, cls.tenant \u003d cls.identity_admin_client.create_tenant("}],"source_content_type":"text/x-python","patch_set":15,"id":"AAAAWH%2F%2F8QI%3D","line":305,"updated":"2014-03-12 19:36:46.000000000","message":"Is there a reason this class can\u0027t set \n\nforce_tenant_isolation \u003d True\n\nand use\n\nmanager \u003d os.get_client_manager()","commit_id":"43c0d283f1f8c58720773c6c39f6161b5967ac6e"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"ace598ce036c7b0fa66abc4ad78fa4d7740d1985","unresolved":false,"context_lines":[{"line_number":302,"context_line":"        if not test.is_extension_enabled(\u0027lbaas\u0027, \u0027network\u0027):"},{"line_number":303,"context_line":"            msg \u003d \"lbaas extension not enabled.\""},{"line_number":304,"context_line":"            raise cls.skipException(msg)"},{"line_number":305,"context_line":"        cls.identity_admin_client \u003d cls.os_adm.identity_client"},{"line_number":306,"context_line":"        test_tenant \u003d data_utils.rand_name(\u0027test_tenant\u0027)"},{"line_number":307,"context_line":"        test_description \u003d data_utils.rand_name(\u0027desc_\u0027)"},{"line_number":308,"context_line":"        _, cls.tenant \u003d cls.identity_admin_client.create_tenant("}],"source_content_type":"text/x-python","patch_set":15,"id":"AAAAWH%2F%2F1Qg%3D","line":305,"in_reply_to":"AAAAWH%2F%2F8QI%3D","updated":"2014-03-13 14:29:23.000000000","message":"I just didn\u0027t think of this possibility. Thank you for pointing this out, I\u0027ve implemented it in the next patchset.","commit_id":"43c0d283f1f8c58720773c6c39f6161b5967ac6e"}]}