New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GH actions - build-and-test-all: parameterize workflow to run different docker runner versions #14061
Conversation
Pull Request Test Coverage Report for Build 8657820200Details
💛 - Coveralls |
abb33ba
to
e85b037
Compare
PR updated. Modified to also read from |
fb1ee21
to
eb9d2cf
Compare
steps: | ||
- id: get-runner-image | ||
run: | | ||
echo "image-id=ghcr.io/$(echo '${{ github.repository }}' | cut -d '/' -f 1 | tr '[:upper:]' '[:lower:]')/${{ inputs.runner-docker-image-name || env.DEFAULT_RUNNER_DOCKER_IMAGE }}" >> "$GITHUB_OUTPUT" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't the github.repository
part break CI on forks? In my fork it yields pull ghcr.io/rgacogne/base-pdns-ci-image/debian-12-pdns-base:master
which fails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right! @Habbie also spotted this last Friday. I will create a new PR that fixes this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Short description
Parameterize workflow
build-and-test-all
for running different container images as runners.This enables solving the request introduced in #13530 for testing also using
libssl1.1
.Before backporting to branches
rel/auth-4.9.x
andrel/dnsdist-1.9.x
, it is required to approve and merge the PR PowerDNS/base-pdns-ci-image#12 inPowerDNS/base-pdns-ci-image
Checklist
I have: