428407 - Sisyphus - should use unittest account on unittest machines

Status: RESOLVED FIXED

(Testing Graveyard :: Sisyphus, defect)

Description

[Bob Clary [:bc] (inactive)]

Currently, Sisyphus has anonymous pserver set in set-build-env.sh for checking out from cvs-mirror. This causes the unittest machines to lag the actual checkins by 15+ minutes. set-build-env.sh should be able to detect whether it has a unittest private key and use that when running on the unittest boxes.

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: patch

If CVSROOT already set use that, else if "unittest" account exists in id_dsa.pub, then use unittest else use anonymous.

Comment 2

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 315258 [details] [diff] [review]
patch

This makes me a little uneasy since you could see a test failure due to cvs update lag, but since they're automated, they would clear on the next update.

Comment 3

[Bob Clary [:bc] (inactive)]

Actually, the current setup _always_ uses the anon pserver. This just allows it to use unittest if it is available.

/cvsroot/mozilla/testing/sisyphus/bin/set-build-env.sh,v  <--  set-build-env.sh
new revision: 1.7; previous revision: 1.6

Comment 4

[(not currently active) Ted Mielczarek]

Mass move of Sisyphus bugs to Testing : Sisyphus. Filter on SisyphusMassMove to ignore.