scripts/coverity-scan: Use Fedora project Docker container registry

[PATCH] scripts/coverity-scan: Use Fedora project Docker container registry

Philippe Mathieu-Daudé posted 1 patch 4 years, 3 months ago

Download mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211103150407.3459877-1-philmd@redhat.com

scripts/coverity-scan/coverity-scan.docker | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[PATCH] scripts/coverity-scan: Use Fedora project Docker container registry

Posted by Philippe Mathieu-Daudé 4 years, 3 months ago

Similarly to commit 102cd5c294d ("tests/docker: use project
specific container registries"), use the Fedora registry to
reduce pull rate limits enforced by Docker Hub.

Inspired-by: John Snow <jsnow@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 scripts/coverity-scan/coverity-scan.docker | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/coverity-scan/coverity-scan.docker b/scripts/coverity-scan/coverity-scan.docker
index ecff6ac5b4b..529c0a28566 100644
--- a/scripts/coverity-scan/coverity-scan.docker
+++ b/scripts/coverity-scan/coverity-scan.docker
@@ -15,7 +15,7 @@
 # The work of actually doing the build is handled by the
 # run-coverity-scan script.
 
-FROM fedora:30
+FROM registry.fedoraproject.org/fedora:30
 ENV PACKAGES \
     alsa-lib-devel \
     bc \
-- 
2.31.1

Re: [PATCH] scripts/coverity-scan: Use Fedora project Docker container registry

Posted by John Snow 4 years, 3 months ago

On Wed, Nov 3, 2021 at 11:04 AM Philippe Mathieu-Daudé <philmd@redhat.com>
wrote:

> Similarly to commit 102cd5c294d ("tests/docker: use project
> specific container registries"), use the Fedora registry to
> reduce pull rate limits enforced by Docker Hub.
>
> Inspired-by: John Snow <jsnow@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  scripts/coverity-scan/coverity-scan.docker | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/coverity-scan/coverity-scan.docker
> b/scripts/coverity-scan/coverity-scan.docker
> index ecff6ac5b4b..529c0a28566 100644
> --- a/scripts/coverity-scan/coverity-scan.docker
> +++ b/scripts/coverity-scan/coverity-scan.docker
> @@ -15,7 +15,7 @@
>  # The work of actually doing the build is handled by the
>  # run-coverity-scan script.
>
> -FROM fedora:30
> +FROM registry.fedoraproject.org/fedora:30


Should we bump to F34? F30 is EOL. Switching to an LTS distro is also an
option.


>
>  ENV PACKAGES \
>      alsa-lib-devel \
>      bc \
> --
> 2.31.1
>
>