From d327d459c6bea7e5e13573e975cf68fc522b127b Mon Sep 17 00:00:00 2001 From: Nick De Villiers Date: Tue, 22 Oct 2024 09:42:51 +0100 Subject: [PATCH] fix: remove 'Wait for MAAS' step from sitespeed workflow --- .github/workflows/sitespeed.yml | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/.github/workflows/sitespeed.yml b/.github/workflows/sitespeed.yml index c7c9d159c4..c4dc0c5b5a 100644 --- a/.github/workflows/sitespeed.yml +++ b/.github/workflows/sitespeed.yml @@ -4,6 +4,8 @@ on: branches: - main - "3.*" + pull_request: + types: [opened, synchronize, reopened] jobs: run-sitespeed: name: Run sitespeed.io @@ -23,13 +25,13 @@ jobs: maasdb-dump-url: https://github.com/canonical/maas-ui-testing/raw/main/db/maasdb-22.04-master-1000.dump - name: Create MAAS admin run: sudo maas createadmin --username=admin --password=test --email=fake@example.org - - name: Wait for MAAS - uses: nev7n/wait_for_response@v1 - with: - url: "http://${{env.MAAS_DOMAIN}}:5240/MAAS/r" - responseCode: 200 - timeout: 200000 - interval: 500 + # - name: Wait for MAAS + # uses: nev7n/wait_for_response@v1 + # with: + # url: "http://${{env.MAAS_DOMAIN}}:5240/MAAS/r" + # 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