[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 8/8] run-coverity-scan: support --update-tools-only --docker
From: |
Peter Maydell |
Subject: |
Re: [PATCH 8/8] run-coverity-scan: support --update-tools-only --docker |
Date: |
Mon, 27 Apr 2020 13:53:30 +0100 |
On Wed, 22 Apr 2020 at 18:26, Paolo Bonzini <address@hidden> wrote:
>
> Just build the container when run-coverity-scan is invoked with
> --update-tools-only --docker. This requires moving the "docker build"
> logic into the update_coverity_tools function.
>
> The only snag is that --update-tools-only --docker requires access to
> the dockerfile. For now just report an error for --src-tarball, and
> "docker build" will fail if not in a source tree. Another possibility
> could be to host our container images on a public registry, and use
> "FROM qemu:fedora" to make the Dockerfile small enough that it can be
> included directly in the run-coverity-scan script.
FWIW, one reason I didn't use 'FROM qemu:fedora' (or the local
equivalent) was because of your report that newer Fedora versions
broke the coverity tools.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- [PATCH 7/8] run-coverity-scan: download tools outside the container, (continued)
- [PATCH 7/8] run-coverity-scan: download tools outside the container, Paolo Bonzini, 2020/04/22
- [PATCH 1/8] docker.py/build: support -t and -f arguments, Paolo Bonzini, 2020/04/22
- [PATCH 4/8] run-coverity-scan: use docker.py, Paolo Bonzini, 2020/04/22
- [PATCH 2/8] docker.py/build: support binary files in --extra-files, Paolo Bonzini, 2020/04/22
- [PATCH 8/8] run-coverity-scan: support --update-tools-only --docker, Paolo Bonzini, 2020/04/22
- Re: [PATCH 8/8] run-coverity-scan: support --update-tools-only --docker,
Peter Maydell <=
- [PATCH 3/8] run-coverity-scan: get Coverity token and email from special git config section, Paolo Bonzini, 2020/04/22
- [PATCH 5/8] run-coverity-scan: add --no-update-tools option, Paolo Bonzini, 2020/04/22