[Buildbot] #3522: Gerrit Reporter summaryCB does not accept deferreds
Buildbot trac
trac at buildbot.net
Wed Apr 13 20:19:18 UTC 2016
#3522: Gerrit Reporter summaryCB does not accept deferreds
-----------------------+-----------------------
Reporter: ryansydnor | Owner:
Type: defect | Status: new
Priority: minor | Milestone: undecided
Version: master | Keywords:
-----------------------+-----------------------
I am trying to pull extra information about a particular build (steps, in
this case). I am not able to yield getDetailsForBuild(master, build,
wantSteps=True) because I cannot make my callback a
@defer.inlineCallbacks.
I see a couple of solutions:
1. Make sendBuildSetSummary a @defer.inlineCallbacks and yield the call to
self.summaryCB, which would break backwards compatibility
2. Take a new "summaryCBDeferred" in the constructor. Do the same as
above, if specified. This would retain backwards compatibility
3. Take "wantSteps", "wantLogs", "wantX" into the constructor, pass those
through to the "buildsetComplete" call to "getDetailsForBuildset" so the
consumer is none the wiser. This will retain backwards compatibility.
--
Ticket URL: <http://trac.buildbot.net/ticket/3522>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation
More information about the bugs
mailing list