Closed knative-automation closed 3 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: knative-automation Once this PR has been reviewed and has the lgtm label, please assign davidhadas for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.50%. Comparing base (
faf421a
) to head (5f8559a
). Report is 3 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
TestDestroyPodInflight
There is https://github.com/knative/serving/issues/15092 but probably for Contour only. This time it seems to be for Kourier only though. However I dont see any change at the net-kourier side :thinking:
/test kourier-stable
/test ?
@skonto: The following commands are available to trigger required jobs:
/test build-tests
/test contour-latest
/test contour-tls
/test gateway-api-latest
/test istio-latest-no-mesh
/test istio-latest-no-mesh-tls
/test kourier-stable
/test kourier-stable-tls
/test unit-tests
/test upgrade-tests
The following commands are available to trigger optional jobs:
/test gateway-api-latest-and-contour
/test https
/test istio-latest-mesh
/test istio-latest-mesh-short
/test istio-latest-mesh-tls
/test performance-tests
Use /test all
to run the following jobs that were automatically triggered:
build-tests_serving_main
istio-latest-no-mesh-tls_serving_main
istio-latest-no-mesh_serving_main
kourier-stable-tls_serving_main
kourier-stable_serving_main
unit-tests_serving_main
upgrade-tests_serving_main
/test kourier-stable /test kourier-stable-tls
It seems it failed multiple times I will check.
/test kourier-stable
/assign @knative/serving-writers /cc @knative/serving-writers
Produced via:
Details: