Skip to content

Commit

Permalink
Fix docker image creation from develop branch
Browse files Browse the repository at this point in the history
This changes the testing-develop.yml workflow to be triggered via
workflow_dispatch, as is also already done for testing-stable.yml, and
changes the image creation logic in testing.yml to always trigger a
build when a workflow_dispatch is used to run the workflow.
  • Loading branch information
matrss committed Mar 22, 2024
1 parent af73526 commit 0d229ad
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 19 deletions.
1 change: 1 addition & 0 deletions .github/workflows/testing-develop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ on:
pull_request:
branches:
- develop
workflow_dispatch:

jobs:
test-develop:
Expand Down
17 changes: 9 additions & 8 deletions .github/workflows/testing-scheduled.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,12 @@ jobs:
workflow: testing-stable.yml
ref: stable

test-develop-scheduled:
uses:
./.github/workflows/testing.yml
with:
branch_name: develop
event_name: ${{ github.event_name }}
secrets:
PAT: ${{ secrets.PAT }}
trigger-testing-develop:
runs-on: ubuntu-latest
permissions:
actions: write
steps:
- uses: benc-uk/[email protected]
with:
workflow: testing-develop.yml
ref: develop
22 changes: 11 additions & 11 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,19 @@ jobs:
source /opt/conda/etc/profile.d/mamba.sh
mamba install -n mss-${{ inputs.branch_name }}-env pyvirtualdisplay
- name: Always rebuild dependencies for scheduled builds
if: ${{ inputs.event_name == 'schedule' && inputs.branch_name == 'stable' && env.triggerdockerbuild != 'yes' }}
- name: Always rebuild dependencies for scheduled builds (started from testing-scheduled.yml)
if: ${{ inputs.event_name == 'workflow_dispatch' }}
run: echo "triggerdockerbuild=yes" >> $GITHUB_ENV

- name: Invoke dockertesting image creation
if: ${{ env.triggerdockerbuild == 'yes' && matrix.order == 'normal' }}
uses: benc-uk/[email protected]
with:
workflow: Update Image testing-${{ inputs.branch_name }}
repo: Open-MSS/dockertesting
ref: main
token: ${{ secrets.PAT }}

- name: Reinstall dependencies if changed
if: ${{ env.triggerdockerbuild == 'yes' }}
run: |
Expand Down Expand Up @@ -76,12 +85,3 @@ jobs:
git config --global --add safe.directory /__w/MSS/MSS
mamba install -n ${{ env.mamba-env }} coveralls
mamba run --no-capture-output -n ${{ env.mamba-env }} coveralls --service=github
- name: Invoke dockertesting image creation
if: ${{ !cancelled() && inputs.event_name == 'push' && env.triggerdockerbuild == 'yes' && matrix.order == 'normal' }}
uses: benc-uk/[email protected]
with:
workflow: Update Image testing-${{ inputs.branch_name }}
repo: Open-MSS/dockertesting
ref: main
token: ${{ secrets.PAT }}

0 comments on commit 0d229ad

Please sign in to comment.