RecordingSlave is missing some methods from BuilderSlave

Bug #509383 reported by William Grant
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Julian Edwards

Bug Description

RecordingSlave._sendFileToSlave is missing, while it is present on BuilderSlave and code expects it to be there.

The purpose of BuilderSlave should probably be rethought, and the missing method fixed somehow.

This prevents private builds from being dispatched.

Related branches

Changed in soyuz:
status: New → Triaged
importance: Undecided → High
milestone: none → 10.01
description: updated
tags: added: ppa soyuz-build
Revision history for this message
Julian Edwards (julian-edwards) wrote :

BuilderSlave and RecordingSlave are doing the same thing, essentially. We should combine them. In the meantime, I'm fixing this bug by adding the missing method.

Revision history for this message
Julian Edwards (julian-edwards) wrote :

I could just remove _sendFileToSlave completely and change its callsites to use ensurepresent instead, since they're doing the exact same thing.

However, it's a much better name, so we'll deprecate ensurepresent() in favour of sendFileToSlave().

Changed in soyuz:
status: Triaged → In Progress
assignee: nobody → Julian Edwards (julian-edwards)
tags: added: wellington
Revision history for this message
Julian Edwards (julian-edwards) wrote :

devel r10200

Changed in soyuz:
status: In Progress → Fix Committed
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Correction: devel r10205

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.