[Forwardport main] Minor changes for cypress workflow

Signed-off-by: Ryan Liang <[email protected]>
opensearch-project · Dec 15, 2023 · a277bc4 · a277bc4
1 parent c775d85
commit a277bc4
Showing 1 changed file with 10 additions and 17 deletions.
diff --git a/.github/workflows/integration-tests-workflow.yml b/.github/workflows/integration-tests-workflow.yml
@@ -1,8 +1,7 @@
 name: Dashboards observability plugin E2E test
-on:
-  schedule:
-    - cron: '0 0 * * *'
-  workflow_dispatch:
+
+on: [pull_request, push]
+
 env:
   PLUGIN_NAME: dashboards-observability
   OPENSEARCH_DASHBOARDS_VERSION: 'main'
@@ -11,6 +10,7 @@ env:
 
 jobs:
   tests:
+    name: Run test group of ${{ matrix.testgroups }}
     env:
       # Prevents extra Cypress installation progress messages
       CI: 1
@@ -25,7 +25,6 @@ jobs:
         java: [11]
         testgroups: [
           app_analytics_test,
-          notebooks_test,
           datasources_test,
           event_analytics_test,
           integrations_test,
@@ -130,18 +129,12 @@ jobs:
         if: ${{ runner.os == 'Linux' }}
         run: |
           cd ./OpenSearch-Dashboards
-          echo "Start checking OpenSearch Dashboards."
-          for i in {1..60}; do
-            if grep -q "bundles compiled successfully after" "dashboard.log"; then
-              echo "OpenSearch Dashboards compiled successfully."
-              break
-            fi
-            if [ $i -eq 60 ]; then
-              echo "Timeout for 600 seconds reached. OpenSearch Dashboards did not finish compiling."
-              exit 1
-            fi
-            sleep 10
-          done
+          if timeout 600 grep -q "bundles compiled successfully after" <(tail -n0 -f dashboard.log); then
+            echo "OpenSearch Dashboards compiled successfully."
+          else
+            echo "Timeout for 600 seconds reached. OpenSearch Dashboards did not finish compiling."
+            exit 1
+          fi
 
       - name: Install Cypress
         run: |