diff --git a/.github/workflows/selfdrive_tests.yaml b/.github/workflows/selfdrive_tests.yaml index a5f2bf3452..e921cd3d00 100644 --- a/.github/workflows/selfdrive_tests.yaml +++ b/.github/workflows/selfdrive_tests.yaml @@ -33,7 +33,6 @@ jobs: name: build release runs-on: ubuntu-20.04 timeout-minutes: 30 - if: github.event_name == 'push' env: STRIPPED_DIR: /tmp/releasepilot steps: @@ -68,7 +67,6 @@ jobs: name: build all runs-on: ubuntu-20.04 timeout-minutes: 30 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -87,7 +85,6 @@ jobs: # name: build macos # runs-on: macos-latest # timeout-minutes: 60 - # if: github.event_name == 'push' # steps: # - uses: actions/checkout@v3 # with: @@ -145,7 +142,7 @@ jobs: name: docker push runs-on: ubuntu-20.04 timeout-minutes: 22 - if: github.ref == 'refs/heads/master' && github.event_name == 'push' && github.repository == 'commaai/openpilot' + if: github.ref == 'refs/heads/master' && github.event_name != 'pull_request' && github.repository == 'commaai/openpilot' needs: static_analysis # hack to ensure slow tests run first since this and static_analysis are fast steps: - uses: actions/checkout@v3 @@ -170,7 +167,6 @@ jobs: name: static analysis runs-on: ubuntu-20.04 timeout-minutes: 20 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -185,7 +181,6 @@ jobs: name: valgrind runs-on: ubuntu-20.04 timeout-minutes: 20 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -204,7 +199,6 @@ jobs: name: unit tests runs-on: ubuntu-20.04 timeout-minutes: 30 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -285,7 +279,6 @@ jobs: name: model tests runs-on: ubuntu-20.04 timeout-minutes: 20 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -314,7 +307,6 @@ jobs: name: longitudinal runs-on: ubuntu-20.04 timeout-minutes: 20 - if: github.event_name == 'push' steps: - uses: actions/checkout@v3 with: @@ -343,7 +335,6 @@ jobs: name: cars runs-on: ubuntu-20.04 timeout-minutes: 20 - if: github.event_name == 'push' strategy: fail-fast: false matrix: diff --git a/.github/workflows/tools_tests.yaml b/.github/workflows/tools_tests.yaml index 25357e0889..549a2f4195 100644 --- a/.github/workflows/tools_tests.yaml +++ b/.github/workflows/tools_tests.yaml @@ -4,6 +4,7 @@ on: push: branches-ignore: - 'testing-closet*' + pull_request: concurrency: group: ${{ github.workflow }}-${{ github.ref != 'refs/heads/master' && github.ref || github.run_id }}-${{ github.event_name }} @@ -82,7 +83,9 @@ jobs: run: | DOCKER_BUILDKIT=1 docker build --pull --build-arg BUILDKIT_INLINE_CACHE=1 --cache-from $DOCKER_REGISTRY/openpilot-docs:latest -t $DOCKER_REGISTRY/openpilot-docs:latest -f docs/docker/Dockerfile . - name: Push docker container - if: github.ref == 'refs/heads/master' && github.repository == 'commaai/openpilot' + if: github.ref == 'refs/heads/master' && github.event_name != 'pull_request' && github.repository == 'commaai/openpilot' run: | $DOCKER_LOGIN docker push $DOCKER_REGISTRY/openpilot-docs:latest + +