Ticket #378 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

force'd builds from irc don't shout unless specific event happens

Reported by: dannyb Owned by: nhemingway
Priority: major Milestone: 0.7.10
Version: 0.7.9 Keywords:
Cc: thijs, nhemingway

Description

Forced builds from irc create an ircbuildrequest that calls parent.buildFinished when done.

buildFinished has this code:

if not self.notify_for('finished', 'failed', 'success', 'exception', 'failedToSuccess', 'successToFailed'):

return

Thus, if you don't have a specific notify set, a forced irc build will never tell you when it is finished.

This is a regression from previous versions.

The easiest way to fix it is to change IRCBuildRequest to do:

d.addCallback(self.parent.watchedBuildFinished)

instead of

d.addCallback(self.parent.buildFinished)

This causes it to shout no matter what.

Change History

comment:1 Changed 4 years ago by thijs

  • Cc thijs added

comment:2 Changed 4 years ago by thijs

Thanks dannyb, that patch worked for  us using 0.7.9!

comment:3 Changed 4 years ago by dustin

  • Cc nhemingway added

Neil, do you mind taking a look at this? I think it was your patch that introduced this change.

comment:4 Changed 4 years ago by nhemingway

  • Owner set to nhemingway
  • Status changed from new to assigned

comment:6 Changed 4 years ago by dustin

  • Keywords review removed
  • Status changed from assigned to closed
  • Resolution set to fixed

merged - thanks!

comment:7 Changed 4 years ago by dustin

  • Milestone changed from undecided to 0.7.10
Note: See TracTickets for help on using tickets.