Re: [jenkins-ci PATCH] projects: switch libvirt-perl over to using Module::Build

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

 



On Tue, 2018-05-15 at 20:30 +0200, Andrea Bolognani wrote:
> On Tue, 2018-05-15 at 16:37 +0100, Daniel P. Berrangé wrote:
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> > ---
> >  jobs/perl-makemaker.yaml   | 136 -------------------------------------
> >  projects/libvirt-perl.yaml |   6 +-
> >  2 files changed, 3 insertions(+), 139 deletions(-)
> >  delete mode 100644 jobs/perl-makemaker.yaml
> > 
> > diff --git a/jobs/perl-makemaker.yaml b/jobs/perl-makemaker.yaml
> > deleted file mode 100644
> > index 84b368e..0000000
> > --- a/jobs/perl-makemaker.yaml
> > +++ /dev/null
> > @@ -1,136 +0,0 @@
> > -
> > -- job-template:
> > -    id: perl-makemaker-build-job
> > -    name: '{name}-{branch}-build{variant}'
> 
> [...]
> > -    publishers:
> > -      - email:
> > -          recipients: '{obj:spam}'
> > -          notify-every-unstable-build: true
> > -          send-to-individuals: false
> 
> This hunk should really be its own commit. Please split it off.
> 
> > diff --git a/projects/libvirt-perl.yaml b/projects/libvirt-perl.yaml
> > index 0a6f648..37b48ff 100644
> > --- a/projects/libvirt-perl.yaml
> > +++ b/projects/libvirt-perl.yaml
> > @@ -4,11 +4,11 @@
> >      machines: '{all_machines}'
> >      title: Libvirt Perl
> >      jobs:
> > -      - perl-makemaker-build-job:
> > +      - perl-modulebuild-build-job:
> >            parent_jobs: 'libvirt-master-build'
> > -      - perl-makemaker-test-job:
> > +      - perl-modulebuild-test-job:
> >            parent_jobs: 'libvirt-perl-master-build'
> >            test_args: 'TEST_MAINTAINER=1'
> 
> test_args will no longer work: you need to replace it with
> 
>   local_env: |
>     export TEST_MAINTAINER=1
> 
> so that it will be picked up. Which is a better approach to
> begin with, hence
> 
>   https://www.redhat.com/archives/libvir-list/2018-May/msg00691.html
> 
> 
> If you take care of all of the above, you can pick up my
> 
>   Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> 
> but you're not quite done yet: you also need to push
> 
>   https://www.redhat.com/archives/libvir-list/2018-May/msg01171.html
> 
> at the same time to ensure all required packages are installed
> on the CI workers.

I've pushed that series, so you can go ahead with your changes.

Note that I have also pushed

  https://www.redhat.com/archives/libvir-list/2018-May/msg00692.html

so, in addition to the fixes mentioned above, you need to make
sure your jobs use 'check' instead of 'test' in their names, and
delete the old jobs once you're done applying the changes.

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