Re: [jenkins-ci PATCH 03/20] quayadmin: Fix quiet mode for show-repo

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 17, 2019 at 01:53:52PM +0200, Andrea Bolognani wrote:
> Since the user is explicity asking for information to be
> displayed, we should present it to them whether or not quiet
> mode is enabled.

The reason why I honoured the "quiet" flag in this command
was because it serves as a "does the repo exist" check,
and when used for that purpose avoids need to redirect
stdout to /dev/null.

> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  guests/quayadmin | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/guests/quayadmin b/guests/quayadmin
> index 5dc5eff..980fc1b 100755
> --- a/guests/quayadmin
> +++ b/guests/quayadmin
> @@ -84,8 +84,8 @@ def run_show_repo(args):
>          return 1
>  
>      info = res.json()
> -    if not args.quiet:
> -        print("{}/{}: {}".format(args.namespace, args.repo, info["description"]))
> +
> +    print("{}/{}: {}".format(args.namespace, args.repo, info["description"]))

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 :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux