* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).