chore: do not needlessly upload an artifact when there is no build (#916)
- ${{ needs.build.outputs.built }} is true if there is a build from source, false otherwise
- prepare-end-to-end local action does not attempt to download an artifact if inputs.built is false
Reviewed-on: https://code.forgejo.org/forgejo/end-to-end/pulls/916
Reviewed-by: Gusted <gusted@noreply.code.forgejo.org>
Co-authored-by: Earl Warren <contact@earl-warren.org>
Co-committed-by: Earl Warren <contact@earl-warren.org>
This commit is contained in:
parent
761ff8aa08
commit
6455896fd3
2 changed files with 23 additions and 0 deletions
|
|
@ -1,3 +1,7 @@
|
||||||
|
inputs:
|
||||||
|
built:
|
||||||
|
description: 'True if a source build is available, false otherwise'
|
||||||
|
|
||||||
runs:
|
runs:
|
||||||
using: "composite"
|
using: "composite"
|
||||||
steps:
|
steps:
|
||||||
|
|
@ -16,6 +20,7 @@ runs:
|
||||||
install-only: true
|
install-only: true
|
||||||
- run: forgejo-binary.sh ensure_user forgejo
|
- run: forgejo-binary.sh ensure_user forgejo
|
||||||
- uses: actions/download-artifact@v3
|
- uses: actions/download-artifact@v3
|
||||||
|
if: ${{ inputs.built == true }}
|
||||||
with:
|
with:
|
||||||
name: forgejo-dev
|
name: forgejo-dev
|
||||||
path: /srv/forgejo-binaries
|
path: /srv/forgejo-binaries
|
||||||
|
|
|
||||||
|
|
@ -26,18 +26,22 @@ jobs:
|
||||||
runs-on: docker
|
runs-on: docker
|
||||||
container:
|
container:
|
||||||
image: 'code.forgejo.org/oci/node:20-bookworm'
|
image: 'code.forgejo.org/oci/node:20-bookworm'
|
||||||
|
outputs:
|
||||||
|
built: "${{ steps.build.outputs.value }}"
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: https://data.forgejo.org/actions/setup-go@v5
|
- uses: https://data.forgejo.org/actions/setup-go@v5
|
||||||
with:
|
with:
|
||||||
go-version: "1.22"
|
go-version: "1.22"
|
||||||
- name: lib/build.sh
|
- name: lib/build.sh
|
||||||
|
id: build
|
||||||
run: |
|
run: |
|
||||||
mkdir $d /tmp/forgejo-upload
|
mkdir $d /tmp/forgejo-upload
|
||||||
touch /tmp/forgejo-upload/PLACEHOLDER
|
touch /tmp/forgejo-upload/PLACEHOLDER
|
||||||
|
|
||||||
if ! test -f forgejo/build-from-sources; then
|
if ! test -f forgejo/build-from-sources; then
|
||||||
echo forgejo/build-from-sources is not present, do not build any version from source
|
echo forgejo/build-from-sources is not present, do not build any version from source
|
||||||
|
echo "built=false" >> $FORGEJO_OUTPUT
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|
@ -57,7 +61,11 @@ jobs:
|
||||||
$forgejo --version
|
$forgejo --version
|
||||||
mv $forgejo /tmp/forgejo-upload/forgejo-$version
|
mv $forgejo /tmp/forgejo-upload/forgejo-$version
|
||||||
done
|
done
|
||||||
|
|
||||||
|
echo "built=true" >> $FORGEJO_OUTPUT
|
||||||
|
|
||||||
- uses: actions/upload-artifact@v3
|
- uses: actions/upload-artifact@v3
|
||||||
|
if: ${{ steps.build.outputs.built == true }}
|
||||||
with:
|
with:
|
||||||
name: forgejo-dev
|
name: forgejo-dev
|
||||||
path: /tmp/forgejo-upload
|
path: /tmp/forgejo-upload
|
||||||
|
|
@ -68,6 +76,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: ./.forgejo/prepare-end-to-end
|
- uses: ./.forgejo/prepare-end-to-end
|
||||||
|
with:
|
||||||
|
built: ${{ needs.build.outputs.built }}
|
||||||
- run: su forgejo -c "./end-to-end.sh test_packages"
|
- run: su forgejo -c "./end-to-end.sh test_packages"
|
||||||
- name: full logs
|
- name: full logs
|
||||||
if: always()
|
if: always()
|
||||||
|
|
@ -79,6 +89,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: ./.forgejo/prepare-end-to-end
|
- uses: ./.forgejo/prepare-end-to-end
|
||||||
|
with:
|
||||||
|
built: ${{ needs.build.outputs.built }}
|
||||||
- run: ./end-to-end.sh prepare_dockerd
|
- run: ./end-to-end.sh prepare_dockerd
|
||||||
- run: su forgejo -c "./end-to-end.sh test_actions"
|
- run: su forgejo -c "./end-to-end.sh test_actions"
|
||||||
- name: full logs
|
- name: full logs
|
||||||
|
|
@ -91,6 +103,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: ./.forgejo/prepare-end-to-end
|
- uses: ./.forgejo/prepare-end-to-end
|
||||||
|
with:
|
||||||
|
built: ${{ needs.build.outputs.built }}
|
||||||
- name: install zstd
|
- name: install zstd
|
||||||
run: |
|
run: |
|
||||||
export DEBIAN_FRONTEND=noninteractive
|
export DEBIAN_FRONTEND=noninteractive
|
||||||
|
|
@ -126,6 +140,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: ./.forgejo/prepare-end-to-end
|
- uses: ./.forgejo/prepare-end-to-end
|
||||||
|
with:
|
||||||
|
built: ${{ needs.build.outputs.built }}
|
||||||
- name: set full-version
|
- name: set full-version
|
||||||
id: full-version
|
id: full-version
|
||||||
shell: bash
|
shell: bash
|
||||||
|
|
@ -160,6 +176,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: https://data.forgejo.org/actions/checkout@v4
|
- uses: https://data.forgejo.org/actions/checkout@v4
|
||||||
- uses: ./.forgejo/prepare-end-to-end
|
- uses: ./.forgejo/prepare-end-to-end
|
||||||
|
with:
|
||||||
|
built: ${{ needs.build.outputs.built }}
|
||||||
- run: su forgejo -c "./end-to-end.sh test_upgrades"
|
- run: su forgejo -c "./end-to-end.sh test_upgrades"
|
||||||
- run: su forgejo -c "./end-to-end.sh test_storage"
|
- run: su forgejo -c "./end-to-end.sh test_storage"
|
||||||
- name: full logs
|
- name: full logs
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue