yellow team mailing list archive
-
yellow team
-
Mailing list archive
-
Message #00412
Re: [Merge] lp:~gary/charms/oneiric/buildbot-master/run-as-buildbot into lp:~yellow/charms/oneiric/buildbot-master/trunk
All those helpers are taken from setuplxc.py, btw.
The su contextmanager has an issue: the euid and egid are not set back properly when there is an error in the with block, for some reason. This can cause some problems: for some reason the juju log command hangs when the buildbot user tries to use it. Benji and I took a quick look at it and did not see a reason why they would do this. Perhaps it takes a moment to propagate?
I have not yet run the existing tests. I will do this before landing, resolving any issues. That said, this will bring up a master and slave for lpbuildbot, and at least check out the branch. Subsequently there's an error within Twisted/Buildbot itself, but it's on the right track.
--
https://code.launchpad.net/~gary/charms/oneiric/buildbot-master/run-as-buildbot/+merge/92399
Your team Launchpad Yellow Squad is requested to review the proposed merge of lp:~gary/charms/oneiric/buildbot-master/run-as-buildbot into lp:~yellow/charms/oneiric/buildbot-master/trunk.
Follow ups
References