On Thu, 2021-03-18 at 17:11 +0100, Erik Skultety wrote:
> + msg = textwrap.dedent(f"""
> + The following images are stale and can be purged from the
> registry:
> +
> + STALE_DETAILS
> +
> + You can delete the images listed above using this shell
> snippet:
> +
> + $ for image_id in {stale_ids}; do
> + curl --request DELETE --header "PRIVATE-TOKEN:
> <access_token>" \\
> + {registry_uri}/$image_id;
There should be one additional leading space here, to align things
better.
Please also move the check_stale_image() function after the
refresh_cirrus() image - basically all action_*() functions should
continue being grouped together.
With those tweaks,
Reviewed-by: Andrea Bolognani <[email protected]>
--
Andrea Bolognani / Red Hat / Virtualization