Skip to content

Commit

Permalink
fix(docker): Repair the docker setup
Browse files Browse the repository at this point in the history
The docker setup is broken since the config migration and other changes.
This also adds a health check of the container at the default port 8082.
This also removes the old config migration notice. The notice is not necessary anymore for the next release.
  • Loading branch information
MichaelsJP committed Nov 24, 2024
1 parent 7f0d534 commit e0cd2c6
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/docker-build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ jobs:
runs-on: ubuntu-latest
needs:
- prepare_environment
outputs:
docker_image_id: ${{ steps.docker_build.outputs.imageid }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -67,6 +69,7 @@ jobs:
"/home/runner/.m2": "/root/.m2"
}
- name: Build image for platforms ${{ needs.prepare_environment.outputs.build_platforms }}
id: docker_build
uses: docker/build-push-action@v6
with:
context: .
Expand Down Expand Up @@ -154,7 +157,7 @@ jobs:
with:
path: |
ors-docker/graphs
key: ${{ runner.os }}-ors-test-scenarios-builder-images-${{ matrix.platform }}-${{ steps.docker_build.outputs.imageid }}
key: ${{ runner.os }}-ors-test-scenarios-builder-images-${{ matrix.platform }}-${{ needs.build_docker_images.outputs.docker_image_id }}
- name: Start container from previously build image and wait for successful checks
run: |
mkdir -p $(pwd)/ors-docker/graphs $(pwd)/ors-docker/config $(pwd)/ors-docker/elevation_cache
Expand Down

0 comments on commit e0cd2c6

Please sign in to comment.