Re: Fedora Modular bikeshed compose report: changes

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

 



On Mon, 2017-12-04 at 13:19 -0800, Adam Williamson wrote:
> On Mon, 2017-12-04 at 16:01 -0500, Przemek Klosowski wrote:
> > On 12/02/2017 01:15 PM, Fedora Rawhide Report sent this report, quoted 
> > in its entirety:
> > > _______________________________________________
> > > devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
> > > To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
> > 
> > Could someone associated with Fedora Modular bikeshed look into 
> > this---the automated reports come out empty for as many cycles as I can 
> > remember. Maybe if there aren't any changes, don't send anything?
> 
> It's more likely that this happens because the code that's intended to
> produce reports for other composes also fires for Bikeshed composes,
> but doesn't know how to analyze modular composes properly. That code is
> https://pagure.io/compose-utils/blob/master/f/compose_utils/changelog.py
> .

Well, now I refresh my memory, it's a bit more than that. I've just
dived in and debugged this, along with puiterwijk. Long story short,
it's basically because the nightly modular compose script doesn't bail
out when the compose fails, but continues with all the steps it should
only do for successful composes - including sending out this
'changelog' mail. The nightly non-modular compose script, by
comparison, checks the return code from pungi-koji and bails out if
it's non-zero, skipping all the steps that should only happen for a
successful compose.

We'll get that fixed somehow.
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux