Re: [libvirt PATCH 0/2] ci: Enable Cirrus CI integration

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

 



On Mon, 2020-06-08 at 19:07 +0200, Andrea Bolognani wrote:
> On Mon, 2020-06-08 at 17:34 +0100, Daniel P. Berrangé wrote:
> > On Mon, Jun 08, 2020 at 05:42:31PM +0200, Andrea Bolognani wrote:
> > > To see this in action, check out the
> > > 
> > >   https://gitlab.com/abologna/libvirt/-/pipelines/153963978
> > > 
> > > pipeline, and in particular the
> > > 
> > >   https://gitlab.com/abologna/libvirt/-/jobs/585678408
> > >   https://gitlab.com/abologna/libvirt/-/jobs/585678413
> > 
> > Ideally the actual build output from Cirrus CI would be pulled into
> > the build output of the GitLab CI job. That way email notifications
> > from failed builds will include the actual compile failure details.
> > This also avoids having to cut+paste non-clickable links to the
> > Cirrus CI job.
> 
> I agree that this would be desirable.
> 
> Earlier today, while reviewing the code for cirrus-run, I found a
> reference to a CIRRUS_LOG_FILE environment variable that should
> allow us to do just that, and was intending to try it out. If that
> doesn't work out, we should still be able to fetch the full job
> output from Cirrus CI after the fact.

I tried setting CIRRUS_LOG_FILE and it doesn't seem to work the way
we want. Getting the taks log directly from Cirrus CI is definitely
possible - you can do so from the Web UI already - but obtaining the
correct URL requires calling the GraphQL API and I need some more
time to figure out how that works. Possibly make some changes to
cirrus-run as well.

Since it's already possible, although not very convenient, to get to
the full log, I think this should not block the series and it can
come as a follow-up instead. I'll post a v2 that addresses the other
issue you pointed out.

-- 
Andrea Bolognani / Red Hat / Virtualization




[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