A few ci jobs use release names (hiruste) while others use date tag.
This pull request aligns Ubuntu image date release tags in ci/test/*.sh
ci, refactor: Use Ubuntu release date tags instead of release names #23503
pull dougEfresh wants to merge 1 commits into bitcoin:master from dougEfresh:ci-ubuntu-date-tag changing 4 files +11 −11-
dougEfresh commented at 9:20 AM on November 13, 2021: contributor
-
f35917f8f8
ci, refactor: Use docker release date tags instead of release names
A few ci jobs use release names (hiruste) while others use date tag.
-
dougEfresh commented at 9:24 AM on November 13, 2021: contributor
The digest are the same:
docker image inspect ubuntu:focal ubuntu:20.04 | jq -r '.[].RepoDigests[0]' ubuntu@sha256:626ffe58f6e7566e00254b638eb7e0f3b11d4da9675088f4781a50ae288f3322 ubuntu@sha256:626ffe58f6e7566e00254b638eb7e0f3b11d4da9675088f4781a50ae288f3322docker image inspect ubuntu:hirsute ubuntu:21.04 | jq -r '.[].RepoDigests[0]' ubuntu@sha256:26cd4ff32a9c031eaca3d6f589a7799f28b34a539e1bd81acbf1a6efeec4b1ce ubuntu@sha256:26cd4ff32a9c031eaca3d6f589a7799f28b34a539e1bd81acbf1a6efeec4b1ceI did not change the image names in
.cirrus.yml, only in ci/test/*.sh -
hebasto commented at 9:44 AM on November 13, 2021: member
I'd do not touch CI task
names as they already are long enough and not readable in the GitHub web-interface:
-
dougEfresh commented at 9:56 AM on November 13, 2021: contributor
I'd do not touch CI task
names as they already are long enough and not readable in the GitHub web-interface:True, but are readable here What about changing the ubuntu image names to all dates, i.e. align the
.cirrus.yamlwithci/test/*.sh? -
dougEfresh commented at 10:51 AM on November 13, 2021: contributor
wait for #23504
- DrahtBot added the label Tests on Nov 13, 2021
-
DrahtBot commented at 3:17 AM on November 14, 2021: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #23504 (ci: Replace soon EOL hirsute with jammy by MarcoFalke)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
- dougEfresh closed this on Nov 14, 2021
- DrahtBot locked this on Nov 14, 2022