Skip to content

RMST-234: test cronjob container in CI#1188

Merged
leplatrem merged 4 commits intomainfrom
rmst-234
Mar 2, 2026
Merged

RMST-234: test cronjob container in CI#1188
leplatrem merged 4 commits intomainfrom
rmst-234

Conversation

@leplatrem
Copy link
Contributor

Comment on lines 64 to +142
@@ -103,7 +104,45 @@ jobs:
- name: Run tests
run: make test

cronjobs-test:
needs: run_test
runs-on: ubuntu-latest
env:
DOCKER_CACHE: /tmp/docker-cache
steps:
- uses: actions/checkout@v6
- name: Compute cache key
# Create hash of hashes of checked in files not in Dockerignore
run: echo "CACHE_KEY=$(git ls-tree --full-tree -r HEAD | grep -v -f .dockerignore | awk '{print $3}' | git hash-object --stdin)" >> "$GITHUB_ENV"

- uses: actions/cache@v5
with:
path: ${{ env.DOCKER_CACHE}}
key: docker-build-${{ hashFiles('cronjobs/Dockerfile') }}-${{ env.CACHE_KEY }}
restore-keys: |
docker-build-${{ hashFiles('cronjobs/Dockerfile') }}-${{ env.CACHE_KEY }}
docker-build-${{ hashFiles('cronjobs/Dockerfile') }}-

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
with:
install: true

- name: Build cronjobs container
uses: docker/build-push-action@v6
with:
cache-from: type=local,src=${{ env.DOCKER_CACHE}}/cronjobs
cache-to: type=local,dest=${{ env.DOCKER_CACHE}}/cronjobs,mode=max
file: cronjobs/Dockerfile
context: .
tags: remote-settings/cronjobs # Like with docker compose build cronjobs

- name: Run cronjobs container
# `help` command will import all modules, and make sure all dependencies are correctly installed.
run: docker run --rm remote-settings/cronjobs help

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
@leplatrem leplatrem merged commit a69e2f9 into main Mar 2, 2026
12 checks passed
@leplatrem leplatrem deleted the rmst-234 branch March 2, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants