Skip to content

Commit

Permalink
Merge pull request #37 from KIT-MRT/test_released_deb_package
Browse files Browse the repository at this point in the history
Fix release tests
  • Loading branch information
orzechow authored Nov 12, 2024
2 parents 717657e + 417ec26 commit 3e4d78f
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 25 deletions.
17 changes: 17 additions & 0 deletions .github/workflows/bump-version-and-create-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -96,3 +96,20 @@ jobs:
tag: ${{ needs.compute-version.outputs.new_tag }}
body: ${{ github.event.pull_request.body }}


build-and-run-release-tests:
needs: [compute-version, create-release]
runs-on: ubuntu-latest
steps:
- name: Build release test Docker image
uses: docker/build-push-action@v6
with:
build-args: |
RELEASE_DOWNLOAD_URL=https://github.com/KIT-MRT/util_caching/releases/download/${{ needs.compute-version.outputs.new_tag }}
push: false
tags: release_tester
target: release_test

- name: Run unit tests with/against released version
run: |
docker run --rm release_tester
24 changes: 0 additions & 24 deletions .github/workflows/run-release-tests.yaml

This file was deleted.

4 changes: 3 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,12 @@ RUN cmake .. && \

FROM base AS release_test

ARG RELEASE_DOWNLOAD_URL=https://github.com/KIT-MRT/util_caching/releases/latest/download/

# This downloads the latest util_caching debian release and adds it to the docker image
# This "bloats" the image. But otherwise, we'd have to installing wget and ca-certificates
# temporarily to download and install the package in one docker layer…
ADD https://github.com/KIT-MRT/util_caching/releases/latest/download/libutil-caching-dev.deb /tmp/debfiles/
ADD ${RELEASE_DOWNLOAD_URL}/libutil-caching-dev.deb /tmp/debfiles/

# Install util_caching from release debian package
RUN dpkg -i /tmp/debfiles/*.deb && \
Expand Down

0 comments on commit 3e4d78f

Please sign in to comment.