Skip to content

Commit b0a34b2

Browse files
authored
Merge pull request #511 from Jooho/cp_9b9b9ff63
[Cherry-pick] enhance e2e test stability
2 parents 2d635f9 + 8279048 commit b0a34b2

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

config/runtimes/kserve-tensorflow-serving.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,8 @@ spec:
2929
- --model_base_path=/mnt/models
3030
- --rest_api_timeout_in_ms=60000
3131
securityContext:
32-
runAsUser: 1000 # User is not defined in the Dockerfile, so we need to set it here to run as non-root
32+
# In OpenShift, the UID is automatically assigned by the platform, so comment this field not to interfere with E2E tests.
33+
# runAsUser: 1000 # User is not defined in the Dockerfile, so we need to set it here to run as non-root
3334
allowPrivilegeEscalation: false
3435
privileged: false
3536
runAsNonRoot: true

test/scripts/openshift-ci/run-e2e-tests.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,9 @@ set -o pipefail
2525
MY_PATH=$(dirname "$0")
2626
PROJECT_ROOT=$MY_PATH/../../../
2727

28-
SETUP_E2E=true
29-
if $SETUP_E2E; then
28+
: "${SETUP_E2E:=true}"
29+
30+
if [ "$SETUP_E2E" = "true" ]; then
3031
echo "Installing on cluster"
3132
pushd $PROJECT_ROOT >/dev/null
3233
./test/scripts/openshift-ci/setup-e2e-tests.sh "$1"

test/scripts/openshift-ci/setup-e2e-tests.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ if [ "$1" != "raw" ]; then
113113
oc create -f -
114114

115115
oc apply -k $PROJECT_ROOT/test/scripts/openshift-ci
116+
oc wait --for=condition=ready pod -l app=odh-model-controller -n kserve --timeout=300s
116117
fi
117118

118119
echo "Add testing models to minio storage ..." # Reference: config/overlays/test/minio/minio-init-job.yaml

0 commit comments

Comments
 (0)