-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add new Kaniko Dockerfile for PDNS builder #68
base: master
Are you sure you want to change the base?
Conversation
d24736c
to
d69bbde
Compare
Do note that all users of I'm not proposing "just edit the file", because that might lead to invisible and subtle problems. I am proposing "add the new file, and delete the old file, and users will notice, very clearly, at some point". |
He said, before reading the bit about "it's published to Docker Hub". |
I would like to get rid of Kaniko in all our build systems eventually, but preferably without causing any external users too much inconvenience. Having two of these Dockerfiles just seems confusing. Perhaps remove the old Dockerfile, move the new one to an Why would anyone want to continue to use the old Dockerfile-kaniko? |
I'm just not 100% sure the ZoneControl build process is the only one using it. |
While I thought I laid out a safe alternative path above (and was wrong), this is not a thing that worries me. I suggest just updating the existing file & docker hub image. |
I'm assuming I can ignore the failed tests? |
They are not related to this PR, but do need fixing. |
Put it besides `Dockerfile-kaniko` until it is clear that one can be removed.
Move Kaniko Dockerfile to `docker` directory. Remove old Kaniko Dockerfile.
72b7f3d
to
f55a2cf
Compare
I took the liberty of rebasing this, although I'm not sure we need it any more. |
ZoneControl is currently not using this. |
let's close this and at some point remove it all |
(I'm not sure @wojas needs to review this. Possibly the Kaniko build process was added after @pieterlexis took ZoneControl over.)
A custom Kaniko image for PDNS Builder, because the official Kaniko debug image only offers Busybox and the PDNS Builder also needs Bash, Git, Perl, rsync and perhaps more.
For now we put it besides
Dockerfile-kaniko
until it is clear that one can be removed.AFAIK this file is only used for ZoneControl builds but I'm not entirely sure, that's why I do not want to remove
Dockerfile-kaniko
yet.This new Dockerfile has been successfully used for the latest ZoneControl builds as part of the process to fix the flaky builds.
It is available at https://hub.docker.com/repository/docker/powerdns/kaniko-pdns-builder/ which is where the ZoneControl builds pull it from.