← Back to team overview

yellow team mailing list archive

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. 

Trapping the error (re-raised inside the generator) using a try/finally statement should solve this.
E.g. http://pastebin.ubuntu.com/836410/

-- 
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.


References