)]}'
{"octavia_tempest_plugin/tests/v2/base.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"9ef924883bae65119e210539620e138012cb6839","unresolved":false,"context_lines":[{"line_number":579,"context_line":"                urllib2.urlopen(\"http://{0}/\".format(self.vip_address)).read())"},{"line_number":580,"context_line":"        self.assertEqual(len(set(resp)), 1)"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"    def _check_cookie_session_persistence(self):"},{"line_number":583,"context_line":"        \"\"\"Check cookie persistence types by injecting cookies in requests.\"\"\""},{"line_number":584,"context_line":""},{"line_number":585,"context_line":"        # Send first request and get cookie from the server\u0027s response"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_41849b0c","line":582,"updated":"2018-01-14 20:19:27.000000000","message":"How is this actually testing the cookie persistence?\nAs written you could get a false positive if \"SOURCE_IP\" persistence was still in effect.","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"},{"author":{"_account_id":18746,"name":"Alex Stafeyev","email":"alexo101087@gmail.com","username":"AlexSTafeyev"},"change_message_id":"5f345451ccfd839074d627f9762b4882e9ad819f","unresolved":false,"context_lines":[{"line_number":579,"context_line":"                urllib2.urlopen(\"http://{0}/\".format(self.vip_address)).read())"},{"line_number":580,"context_line":"        self.assertEqual(len(set(resp)), 1)"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"    def _check_cookie_session_persistence(self):"},{"line_number":583,"context_line":"        \"\"\"Check cookie persistence types by injecting cookies in requests.\"\"\""},{"line_number":584,"context_line":""},{"line_number":585,"context_line":"        # Send first request and get cookie from the server\u0027s response"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_5a7d61e5","line":582,"in_reply_to":"7f96bb07_41849b0c","updated":"2018-01-15 07:22:48.000000000","message":"It tests that there is a persistence with current configuration.","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"}],"octavia_tempest_plugin/tests/v2/scenario/test_persistence_basic.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"9ef924883bae65119e210539620e138012cb6839","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    @decorators.idempotent_id(\u0027250ebc41-645e-43fb-a79a-e3035f338e2a\u0027)"},{"line_number":61,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":62,"context_line":"    def test_session_persistence_basic(self):"},{"line_number":63,"context_line":"        # Create loadbalancer"},{"line_number":64,"context_line":"        params \u003d {}"},{"line_number":65,"context_line":"        if self.vip_network_id:"},{"line_number":66,"context_line":"            params[\u0027vip_network_id\u0027] \u003d self.vip_network_id"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_61ae7799","line":63,"updated":"2018-01-14 20:19:27.000000000","message":"This should be in setUp.","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"},{"author":{"_account_id":18746,"name":"Alex Stafeyev","email":"alexo101087@gmail.com","username":"AlexSTafeyev"},"change_message_id":"5f345451ccfd839074d627f9762b4882e9ad819f","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    @decorators.idempotent_id(\u0027250ebc41-645e-43fb-a79a-e3035f338e2a\u0027)"},{"line_number":61,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":62,"context_line":"    def test_session_persistence_basic(self):"},{"line_number":63,"context_line":"        # Create loadbalancer"},{"line_number":64,"context_line":"        params \u003d {}"},{"line_number":65,"context_line":"        if self.vip_network_id:"},{"line_number":66,"context_line":"            params[\u0027vip_network_id\u0027] \u003d self.vip_network_id"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_5ac241eb","line":63,"in_reply_to":"7f96bb07_61ae7799","updated":"2018-01-15 07:22:48.000000000","message":"This test was written according to the \"test_basic_ops\" test. \n\nThis means both should have LB objects created in setUp?","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"9ef924883bae65119e210539620e138012cb6839","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        pool \u003d self.create_pool(self.lb_id, **persistence_SIP)"},{"line_number":74,"context_line":"        self.pool_id \u003d pool[\u0027id\u0027]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        # Create listener"},{"line_number":77,"context_line":"        params \u003d {\u0027default_pool_id\u0027: self.pool_id}"},{"line_number":78,"context_line":"        listener \u003d self.create_listener(self.lb_id, **params)"},{"line_number":79,"context_line":"        self.listener_id \u003d listener[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_e1992779","line":76,"updated":"2018-01-14 20:19:27.000000000","message":"This should be in setUp.","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"9ef924883bae65119e210539620e138012cb6839","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                params[\u0027subnet_id\u0027] \u003d self.member_subnet_id"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"            self.create_member(self.pool_id, self.lb_id, **params)"},{"line_number":91,"context_line":"        self._check_source_ip_persistence()"},{"line_number":92,"context_line":"        # Update pool to http cookie persistence"},{"line_number":93,"context_line":"        persistence_HTTP \u003d {\u0027session_persistence\u0027: {\u0027type\u0027: \u0027HTTP_COOKIE\u0027}}"},{"line_number":94,"context_line":"        self.update_pool(self.pool_id, self.lb_id, **persistence_HTTP)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_818aa318","line":91,"updated":"2018-01-14 20:19:27.000000000","message":"Would be nice to have a comment here like the others that calls out it is testing the default SourceIP persistence.","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"},{"author":{"_account_id":18746,"name":"Alex Stafeyev","email":"alexo101087@gmail.com","username":"AlexSTafeyev"},"change_message_id":"5f345451ccfd839074d627f9762b4882e9ad819f","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                params[\u0027subnet_id\u0027] \u003d self.member_subnet_id"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"            self.create_member(self.pool_id, self.lb_id, **params)"},{"line_number":91,"context_line":"        self._check_source_ip_persistence()"},{"line_number":92,"context_line":"        # Update pool to http cookie persistence"},{"line_number":93,"context_line":"        persistence_HTTP \u003d {\u0027session_persistence\u0027: {\u0027type\u0027: \u0027HTTP_COOKIE\u0027}}"},{"line_number":94,"context_line":"        self.update_pool(self.pool_id, self.lb_id, **persistence_HTTP)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f96bb07_1a87d9d9","line":91,"in_reply_to":"7f96bb07_818aa318","updated":"2018-01-15 07:22:48.000000000","message":"Agree","commit_id":"94132a71a84430e241a6085f143d13f4b9f72aee"}]}