Re: [jenkins-ci PATCH v5 1/5] lcitool: use subparsers for commands

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

 



On Tue, 2019-02-26 at 11:00 +0000, Daniel P. Berrangé wrote:
> +++ b/guests/README.markdown
[...]
> @@ -58,7 +58,7 @@ You can add more git repositories by tweaking the `git_urls` dictionary
>  defined in `playbooks/build/jobs/defaults.yml` and then build arbitrary
>  branches out of those with
>  
> -    lcitool -a build -h all -p libvirt -g github/cool-feature
> +    lcitool build -g github/cool-feature all libvirt
>  
>  
>  Host setup

You forgot to update the line

  0 0 * * * ~/libvirt-jenkins-ci/guests/lcitool -a update -h all -p all

in this section of the document. With that fixed,

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>


I'm not sure why you decided to post v5 before I got around to review
the remaining two patches, but since you have R-b's for these first
three patches now and they make sense even on their own, feel free to
push them without waiting any further.

-- 
Andrea Bolognani / Red Hat / Virtualization

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