Re: [PATCH] tests/docker: Use lower case for centos8 powertools

[Wainer dos Santos Moschetta]

Hi,

On 12/17/20 10:11 PM, Jiaxun Yang wrote:
> Our gitlab amd64-centos8-container pipeline constantly fail at:
>
> 15.36 Error: No matching repo to modify: PowerTools.
>
> Fix it by convert it to lower case.

They recently changed the name of some repositories:

https://wiki.centos.org/Manuals/ReleaseNotes/CentOS8.2011#Yum_repo_file_and_repoid_changes

Thanks for working on this fix!

Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>

> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>   tests/docker/dockerfiles/centos8.docker | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/docker/dockerfiles/centos8.docker 
> b/tests/docker/dockerfiles/centos8.docker
> index 54bc6d54cd..06b67962fd 100644
> --- a/tests/docker/dockerfiles/centos8.docker
> +++ b/tests/docker/dockerfiles/centos8.docker
> @@ -31,6 +31,6 @@ ENV PACKAGES \
>       zlib-devel
>   
>   RUN dnf install -y dnf-plugins-core && \
> -  dnf config-manager --set-enabled PowerTools && \
> +  dnf config-manager --set-enabled powertools && \
>     dnf install -y $PACKAGES
>   RUN rpm -q $PACKAGES | sort > /packages.txt