)]}'
{"octavia_tempest_plugin/tests/api/v2/test_amphora.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"3a01c9cef1204a8d504c450cd5d6e4c73429817d","unresolved":false,"context_lines":[{"line_number":235,"context_line":"    @testtools.skipUnless("},{"line_number":236,"context_line":"        CONF.loadbalancer_feature_enabled.log_offload_enabled,"},{"line_number":237,"context_line":"        \u0027Skipping log offload tests because tempest configuration \u0027"},{"line_number":238,"context_line":"        \u0027[load_balancer] log_offload_enabled is False.\u0027)"},{"line_number":239,"context_line":"    @decorators.idempotent_id(\u00274e3c6fcb-5f83-4da1-8296-56f209eae30d\u0027)"},{"line_number":240,"context_line":"    def test_admin_log(self):"},{"line_number":241,"context_line":"        \"\"\"Tests admin log offloading"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_d624a30f","line":238,"range":{"start_line":238,"start_character":10,"end_line":238,"end_character":23},"updated":"2020-07-13 14:08:19.000000000","message":"nit: configuration option is in \"loadbalancer-feature-enabled\" section.","commit_id":"76451d9c9a5696a1f2495c526c36765542305d17"}],"octavia_tempest_plugin/tests/scenario/v2/test_amphora.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bf6822558d68e2b2e2bc2dd1183dac997e5b8a63","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        # We need to give the log subsystem time to commit the log"},{"line_number":227,"context_line":"        time.sleep(CONF.load_balancer.check_interval)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        # Check for an amphora agent API call"},{"line_number":230,"context_line":"        found \u003d False"},{"line_number":231,"context_line":"        with open(CONF.load_balancer.amphora_admin_log_file) as f:"},{"line_number":232,"context_line":"            for line in f:"},{"line_number":233,"context_line":"                if \u0027Octavia HaProxy Rest Client\u0027 in line:"},{"line_number":234,"context_line":"                    found \u003d True"},{"line_number":235,"context_line":"                    break"},{"line_number":236,"context_line":"        self.assertTrue("},{"line_number":237,"context_line":"            found, \u0027Octavia user agent string was not found in: {0}\u0027.format("},{"line_number":238,"context_line":"                CONF.load_balancer.amphora_admin_log_file))"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"        # Check for amphora agent log entry"},{"line_number":241,"context_line":"        found \u003d False"},{"line_number":242,"context_line":"        with open(CONF.load_balancer.amphora_admin_log_file) as f:"},{"line_number":243,"context_line":"            for line in f:"},{"line_number":244,"context_line":"                if \u0027 amphora-agent: \u0027 in line:"},{"line_number":245,"context_line":"                    found \u003d True"},{"line_number":246,"context_line":"                    break"},{"line_number":247,"context_line":"        self.assertTrue("},{"line_number":248,"context_line":"            found, \u0027Amphora agent string was not found in: {0}\u0027.format("},{"line_number":249,"context_line":"                CONF.load_balancer.amphora_admin_log_file))"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_8a4f6f34","line":246,"range":{"start_line":229,"start_character":1,"end_line":246,"end_character":25},"updated":"2019-11-26 11:03:42.000000000","message":"nit: perhaps you could merge these 2 loops into one","commit_id":"d91a820197a1d53c391ebd0a16102ce516d5c80e"}]}