[libvirt PATCH] ci: Disable verbose mode for cirrus-run

Andrea Bolognani posted 1 patch 3 years, 7 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200903171058.70274-1-abologna@redhat.com
.gitlab-ci.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[libvirt PATCH] ci: Disable verbose mode for cirrus-run
Posted by Andrea Bolognani 3 years, 7 months ago
It was temporarily enabled to help the upstream author validate
some changes, but now that 0.3.0 is out we can safely disable it
again.

This reverts commit 9b5622d31f8c3036425721ab9c6b23067aa87034.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
 .gitlab-ci.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 4a42eb479f..bd42c95602 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -104,7 +104,7 @@ stages:
           -e "s|[@]PYPI_PKGS@|$PYPI_PKGS|g"
       <ci/cirrus/build.yml >ci/cirrus/$NAME.yml
     - cat ci/cirrus/$NAME.yml
-    - cirrus-run -v --show-build-log always ci/cirrus/$NAME.yml
+    - cirrus-run --show-build-log always ci/cirrus/$NAME.yml
   only:
     variables:
       - $CIRRUS_GITHUB_REPO
-- 
2.26.2

Re: [libvirt PATCH] ci: Disable verbose mode for cirrus-run
Posted by Daniel P. Berrangé 3 years, 7 months ago
On Thu, Sep 03, 2020 at 07:10:58PM +0200, Andrea Bolognani wrote:
> It was temporarily enabled to help the upstream author validate
> some changes, but now that 0.3.0 is out we can safely disable it
> again.
> 
> This reverts commit 9b5622d31f8c3036425721ab9c6b23067aa87034.
> 
> Signed-off-by: Andrea Bolognani <abologna@redhat.com>
> ---
>  .gitlab-ci.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 4a42eb479f..bd42c95602 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -104,7 +104,7 @@ stages:
>            -e "s|[@]PYPI_PKGS@|$PYPI_PKGS|g"
>        <ci/cirrus/build.yml >ci/cirrus/$NAME.yml
>      - cat ci/cirrus/$NAME.yml
> -    - cirrus-run -v --show-build-log always ci/cirrus/$NAME.yml
> +    - cirrus-run --show-build-log always ci/cirrus/$NAME.yml
>    only:
>      variables:
>        - $CIRRUS_GITHUB_REPO

I think it might be premature - last week we hit a new problem with
Cirrus CI we'd not seeen before and debug logs were useful

https://github.com/sio/cirrus-run/issues/7

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Re: [libvirt PATCH] ci: Disable verbose mode for cirrus-run
Posted by Andrea Bolognani 3 years, 7 months ago
On Thu, 2020-09-03 at 18:16 +0100, Daniel P. Berrangé wrote:
> On Thu, Sep 03, 2020 at 07:10:58PM +0200, Andrea Bolognani wrote:
> > -    - cirrus-run -v --show-build-log always ci/cirrus/$NAME.yml
> > +    - cirrus-run --show-build-log always ci/cirrus/$NAME.yml
> 
> I think it might be premature - last week we hit a new problem with
> Cirrus CI we'd not seeen before and debug logs were useful
> 
> https://github.com/sio/cirrus-run/issues/7

Alright, let's wait a bit longer then :)

-- 
Andrea Bolognani / Red Hat / Virtualization