Revert "[Cherry Pick] Enable canary report generation (awslabs#634)"

This reverts commit 632ec70.
jsitu777 · Apr 12, 2023 · 02c2de6 · 02c2de6
1 parent 43c4ef4
commit 02c2de6
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 105 deletions.
diff --git a/tests/canary/canary.buildspec.yaml b/tests/canary/canary.buildspec.yaml
@@ -6,6 +6,7 @@ phases:
       # Get cached test image
       - aws ecr get-login-password --region $CLUSTER_REGION | docker login --username AWS --password-stdin $ECR_CACHE_URI || true
       - docker pull ${ECR_CACHE_URI}:latest --quiet || true
+
       # Build test image
       - >
         docker build -f ./tests/canary/Dockerfile.canary . -t ${ECR_CACHE_URI}:latest --quiet
@@ -14,15 +15,8 @@ phases:
     commands:
       # Run tests
       - docker run --name kf-distro-canary $(env | cut -f1 -d= | sed 's/^/-e /') --mount type=bind,source="$(pwd)/",target="/kubeflow-manifests/" ${ECR_CACHE_URI}:latest
-  post_build:
-    commands:
-      - docker cp kf-distro-canary:/kubeflow-manifests/tests/canary/integration_tests.xml /tmp/results.xml || true
-        # Push test image to cache ECR repo
+
+      # Push test image to cache ECR repo
       - docker push ${ECR_CACHE_URI}:latest || true
-
-reports:
-  IntegrationTestReport:
-    files:
-      - "results.xml"
-    base-directory: "/tmp"    
+
 
diff --git a/tests/canary/scripts/push_stats_to_cloudwatch.py b/tests/canary/scripts/push_stats_to_cloudwatch.py
diff --git a/tests/canary/scripts/run_test.sh b/tests/canary/scripts/run_test.sh
@@ -9,11 +9,10 @@
 # Script configuration
 set -euo pipefail
 
-function push_to_cloudwatch {
-  echo "Pushing Codebuild stats to Cloudwatch."
-  python ../canary/scripts/push_stats_to_cloudwatch.py
+function onError {
+  echo "Run test FAILED. Exiting."
 }
-trap push_to_cloudwatch EXIT
+trap onError ERR
 
 export CANARY_TEST_DIR=${REPO_PATH}/tests/canary
 export E2E_TEST_DIR=${REPO_PATH}/tests/e2e
@@ -32,5 +31,6 @@ mkdir -p $E2E_TEST_DIR/.metadata/
 cp metadata-canary $E2E_TEST_DIR/.metadata/
 
 cd $E2E_TEST_DIR
-pytest tests/test_sanity_portforward.py -s -q --metadata .metadata/metadata-canary --region $CLUSTER_REGION --installation_option $INSTALLATION_OPTION --junitxml ../canary/integration_tests.xml
+pytest tests/test_sanity_portforward.py -s -q --metadata .metadata/metadata-canary --region $CLUSTER_REGION --installation_option $INSTALLATION_OPTION
+