diff --git a/test/e2e-common.sh b/test/e2e-common.sh index 3721c23f6495..67b937c3bfcd 100644 --- a/test/e2e-common.sh +++ b/test/e2e-common.sh @@ -402,8 +402,9 @@ function install() { kubectl wait --timeout=60s --for=condition=Available deployment -n ${SYSTEM_NAMESPACE} activator fi - toggle_feature loglevel.activator debug config-logging - toggle_feature logging.enable-request-log true config-observability + echo "Enable request logging" + toggle_feature "loglevel.activator" debug config-logging + toggle_feature "logging.enable-request-log" true config-observability } diff --git a/test/e2e-tests.sh b/test/e2e-tests.sh index f8c1e51ed2fe..0797694932bc 100755 --- a/test/e2e-tests.sh +++ b/test/e2e-tests.sh @@ -85,15 +85,15 @@ toggle_feature cluster-local-domain-tls enabled config-network || fail_test go_test_e2e -timeout=2m ./test/e2e/clusterlocaldomaintls ${E2E_TEST_FLAGS} || failed=1 toggle_feature cluster-local-domain-tls disabled config-network || fail_test -toggle_feature system-internal-tls enabled config-network || fail_test -toggle_feature "logging.enable-request-log" true config-observability || fail_test -toggle_feature "logging.request-log-template" "TLS: {{.Request.TLS}}" config-observability || fail_test -# with current implementation, Activator must be restarted when configuring system-internal-tls. See https://github.com/knative/serving/issues/13754 -restart_pod ${SYSTEM_NAMESPACE} "app=activator" -go_test_e2e -timeout=2m ./test/e2e/systeminternaltls ${E2E_TEST_FLAGS} || failed=1 -toggle_feature system-internal-tls disabled config-network || fail_test -toggle_feature enable-request-log false config-observability || fail_test -toggle_feature request-log-template '' config-observability || fail_test +#toggle_feature system-internal-tls enabled config-network || fail_test +#toggle_feature "logging.enable-request-log" true config-observability || fail_test +#toggle_feature "logging.request-log-template" "TLS: {{.Request.TLS}}" config-observability || fail_test +## with current implementation, Activator must be restarted when configuring system-internal-tls. See https://github.com/knative/serving/issues/13754 +#restart_pod ${SYSTEM_NAMESPACE} "app=activator" +#go_test_e2e -timeout=2m ./test/e2e/systeminternaltls ${E2E_TEST_FLAGS} || failed=1 +#toggle_feature system-internal-tls disabled config-network || fail_test +##toggle_feature enable-request-log false config-observability || fail_test +#toggle_feature request-log-template '' config-observability || fail_test # with the current implementation, Activator is always in the request path, and needs to be restarted after configuring system-internal-tls restart_pod ${SYSTEM_NAMESPACE} "app=activator"