buildd maintainers need to be informed of build failures

Bug #31609 reported by Adam Conrad
8
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Celso Providelo

Bug Description

I need to be able to get build logs from any unsuccessful builds mailed to me as the builds fail, so I don't spend my whole day refreshing a web page to see what's recently failed.

Whether this is in the form of an entire build log mailed to me (as we did with buildd/wanna-build), or just a stub email telling me that the build of package "foo" failed, with a URL to click on to see the log, I'm not too picky.

Revision history for this message
Celso Providelo (cprov) wrote :

Actually, the idea of "Builder Failure subscription" is very nice. I think we can implement it very quickly . It would be sending a report email containing the build name, times related it and a link to the buildlog url. Does it cover your request ?

Changed in soyuz:
assignee: nobody → cprov
status: Unconfirmed → Confirmed
Revision history for this message
Matt Zimmerman (mdz) wrote :

Yes, that sounds like it would be sufficient. Adam, do you agree?

Revision history for this message
Celso Providelo (cprov) wrote :

Anyway it needs a implementation plan, specification, to avoid code duplication.

Feel free to increase the priority of this bug if you think it's necessary, because it's not my schedulle for this week.

Revision history for this message
Matt Zimmerman (mdz) wrote : Re: [Bug 31609] Re: buildd maintainers need to be informed of build failures

On Wed, Apr 26, 2006 at 07:28:40PM -0000, Celso Providelo wrote:
> Anyway it needs a implementation plan, specification, to avoid code duplication.
>
> Feel free to increase the priority of this bug if you think it's necessary, because it's not my schedulle for this week.

Adam, how bad does it hurt?

--
 - mdz

Revision history for this message
Adam Conrad (adconrad) wrote :

The downside of a link URL rather than an actual copy of the buildlog is that I can't then use any automated procmail-driven recipes to search build logs for common failures or other such fun.

I can, on the flipside, see that you don't want everyone subscribed to build failures to get a full log, as you'd be generating a LOT of mail that way.

Perhaps a configurable option when subscribing to build failures, off by default, with a blinking warning that "sending the full build log can generate several megabytes of mail PER FAILURE, only do this if you're sure that's what you want and need" would suffice to make everyone happy?

As for urgency, I currently get by with sifting through recent failures on the web UI, which isn't ideal, but it works for now. I'd rather see this implemented correctly than quickly.

Revision history for this message
Christian Reis (kiko) wrote :

Configurable options are often more work than predicted, so avoid it if possible.

Revision history for this message
Matt Zimmerman (mdz) wrote :
Celso Providelo (cprov)
Changed in soyuz:
importance: Medium → Critical
Celso Providelo (cprov)
Changed in soyuz:
status: Confirmed → In Progress
Revision history for this message
Celso Providelo (cprov) wrote :

wip in `build-failure-notification`

Revision history for this message
Malcolm Cleaton (malcolmcleaton) wrote :

Nearly fix released; fix code is out there, but set to just mail us, for live testing. Will be released v.soon now.

Changed in soyuz:
status: In Progress → Fix Committed
Revision history for this message
Matt Zimmerman (mdz) wrote :

How has the testing progressed? Can we expect this to be deployed in an upcoming rollout?

Revision history for this message
Celso Providelo (cprov) wrote :

Not as fast as expected, because lot of things needs to be tested this time, the clean up done during the sprint affected very sensible parts of soyuz.

They can't be deployed w/o full end-to-end tests in dogfood (actually soyuz is severely broken and I'm glad we were exposed to the errors before flying into production)

So, build-failure-notification were affected by the pending changes, its deployment needs wait this major changes.

If you desperatelly need b-f-n, talk with Adam to include your or other almost-public mbox in the lp_buildd+admins@drescher alias. Remember it still spamming people when we violate P-a-s.

Celso Providelo (cprov)
Changed in soyuz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.