fix: apply some building best practices
Following https://docs.docker.com/build/building/best-practices
-
useADD
instead ofgit clone
, for more precise build cache- doesn't seem to work here, and we only care about cache usage on MR pipelines
- for long-lived MRs, we may want to rebuild with no-cache, or we need to figure out how to cache bust both
git clone
andpacman -S
usages
- assign GID and UID 10001 to default user, and follow other recommended options
- use
WORKDIR
once to be in/build
instead ofcd /build
in every instruction - set
CMD
to an interactive shell - clone QADB with
--recurse-submodules
(might as well fix this now)
Edited by Christopher Dilks