* [PATCH] tests/docker: Use lower case for centos8 powertools
@ 2020-12-18 1:11 Jiaxun Yang
2020-12-18 10:08 ` Alex Bennée
2020-12-18 13:10 ` Wainer dos Santos Moschetta
0 siblings, 2 replies; 3+ messages in thread
From: Jiaxun Yang @ 2020-12-18 1:11 UTC (permalink / raw)
To: qemu-devel; +Cc: fam, philmd, alex.bennee, Jiaxun Yang
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.
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
--
2.29.2
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] tests/docker: Use lower case for centos8 powertools
2020-12-18 1:11 [PATCH] tests/docker: Use lower case for centos8 powertools Jiaxun Yang
@ 2020-12-18 10:08 ` Alex Bennée
2020-12-18 13:10 ` Wainer dos Santos Moschetta
1 sibling, 0 replies; 3+ messages in thread
From: Alex Bennée @ 2020-12-18 10:08 UTC (permalink / raw)
To: Jiaxun Yang; +Cc: fam, philmd, qemu-devel
Jiaxun Yang <jiaxun.yang@flygoat.com> writes:
> 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.
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Thanks for you patch, we already have a similar fix in flight this PR:
Subject: [PULL v2 00/11] testing and configure updates
Date: Thu, 17 Dec 2020 09:43:30 +0000
Message-Id: <20201217094330.17400-1-alex.bennee@linaro.org>
> ---
> 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
--
Alex Bennée
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] tests/docker: Use lower case for centos8 powertools
2020-12-18 1:11 [PATCH] tests/docker: Use lower case for centos8 powertools Jiaxun Yang
2020-12-18 10:08 ` Alex Bennée
@ 2020-12-18 13:10 ` Wainer dos Santos Moschetta
1 sibling, 0 replies; 3+ messages in thread
From: Wainer dos Santos Moschetta @ 2020-12-18 13:10 UTC (permalink / raw)
To: Jiaxun Yang, qemu-devel; +Cc: fam, alex.bennee, philmd
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-12-18 13:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-18 1:11 [PATCH] tests/docker: Use lower case for centos8 powertools Jiaxun Yang
2020-12-18 10:08 ` Alex Bennée
2020-12-18 13:10 ` Wainer dos Santos Moschetta
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.