Re: [PATCH] automation: Add missing and drop obsoleted aliases from containerize

[Michal Orzel <michal.orzel@xxxxxxx>]

On 03/03/2023 12:59, Andrew Cooper wrote:
> 
> 
> On 01/03/2023 9:06 am, Michal Orzel wrote:
>> Add missing aliases for:
>>  - debian:unstable-cppcheck
>>  - debian:unstable-arm64v8-arm32-gcc
>>  - ubuntu:bionic
>>
>> Remove aliases for no longer used containers:
>>  - centos:7.2
>>  - debian:unstable-arm32-gcc
>>
>> Modify docs to refer to CentOS 7 instead of 7.2 not to create confusion.
>>
>> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
>> ---
>> Open questions related to the CI cleanup (@Andrew, @Anthony):
>>  - Why do we keep suse:sles11sp4 dockerfile?
>>  - Why do we keep jessie dockefiles?
> 
> Because we don't yet have a process for retirement of such things.
> 
> Right now, all branches other than staging are using the jessie
> containers.  While it's still in use on any branch, we need some way to
> rebuild the container (potentially with modifications - see the recent
> HTTPS fun), and standard practice is "patches into staging".
> 
> An alternative could be to patch into the most recent staging branch
> containing the dockerfile.
> 
> 
> sles11sp4 is more complicated.  We specifically tried to get a SLES
> container working, but it was always in a weird state (only @suse people
> could rebuild the container).  We did eventually replace it with
> OpenSUSE containers, but I can't currently locate any evidence in the
> gitlab yml that we wired sles11sp4 up.   Which is confusing because I
> swear we did have it running at some point in the past...

Ok, thanks for answering. Stefano wanted me to remove these unused dockefiles.
Are you ok with that or only to remove sles for now?

~Michal