Opened 8 years ago

Closed 8 years ago

#2169 closed enhancement (fixed)

make umask on master configurable

Reported by: mverwijs Owned by:
Priority: minor Milestone: 0.8.+
Version: 0.8.5 Keywords: simple, sprint
Cc:

Description

In my setup, the master creates an APT and YUM repository using MasterShellCommands?. These commands result in files and folders that are inaccessable to any other user due to Twistd's restrictive settings.

I've modded lib/python2.6/site-packages/Twisted-11.1.0-py2.6-linux-x86_64.egg/twisted/scripts/_twistd_unix.py to have a less restrictive umask, as I was unable to find any other way to do this.

It would be useful if the method used for Slaves to set the umask was applied to the Master as well.

Change History (3)

comment:1 Changed 8 years ago by dustin

  • Keywords simple added
  • Milestone changed from undecided to 0.8.+

comment:2 Changed 8 years ago by dustin

  • Keywords sprint added

comment:3 Changed 8 years ago by dustin

  • Resolution set to fixed
  • Status changed from new to closed

bdbaddog just fixed this:

commit cfa13136a2a7f927b220320c381f9996644080c6
Merge: dbd8a86 c854a2f
Author: Dustin J. Mitchell <dustin@mozilla.com>
Date:   Mon Mar 12 14:53:02 2012 -0500

    Merge branch 'ticket_2169' of git://github.com/bdbaddog/buildbot
    
    * 'ticket_2169' of git://github.com/bdbaddog/buildbot:
      changed to have default umask None, which should default the umask to twistd's default (currently 077)
      Add umask to buildbot.tac which propagates and sets default umask for buildbot master. Ticket 2169
Note: See TracTickets for help on using tickets.