remove login retries, match the behaviour from cypress workflow

canonical · Nov 11, 2024 · 70e3e4c · 70e3e4c
1 parent d623cfc
commit 70e3e4c
Showing 1 changed file with 4 additions and 14 deletions.
diff --git a/.github/workflows/sitespeed.yml b/.github/workflows/sitespeed.yml
@@ -32,22 +32,12 @@ jobs:
       #     responseCode: 200
       #     timeout: 200000
       #     interval: 500
-      - name: Login MAAS admin
-        run: |
-          # Retry admin login up to 5 times with 10s intervals to account for delays in MAAS becoming fully operational
-          for attempt in {1..5}; do
-            export API_KEY=$(sudo maas apikey --username=admin)
-            if maas login admin http://localhost:5240/MAAS $API_KEY; then
-              echo "Login successful."
-              break
-            else
-              echo "Login attempt $attempt failed. Retrying in 10 seconds..."
-              sleep 10
-            fi
-          done
       - name: Wait for MAAS boot resources
         shell: bash
-        run: while [ $(maas admin boot-resources is-importing | cat) == "true" ]; do sleep 10; done; echo "syncing finished"
+        run: |
+          export API_KEY=`sudo maas apikey --username=admin`
+          maas login admin http://localhost:5240/MAAS $API_KEY
+          while [ $(maas admin boot-resources is-importing | cat) == "true" ]; do sleep 10; done; echo "syncing finished"
       - name: Run sitespeed.io tests
         run: yarn sitespeed:ci --browsertime.domain=${{env.MAAS_DOMAIN}}
       - name: Upload results