Ticket #458: group-root-to-0.diff

src/lib/Server/Plugins/Hostbase.py

@@ -80 +80 @@
         fname = entry.get('name').split('/')[-1]
         if not self.filedata.has_key(fname):
             raise PluginExecutionError
-        perms = {'owner':'root', 'group':'root', 'perms':'644'}
+        perms = {'owner':'root', 'group':'0', 'perms':'644'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
         entry.text = self.filedata[fname]
 

src/lib/Server/Plugins/SSHbase.py

@@ -123 +123 @@
         for hostkey in hostkeys:
             entry.text += "localhost,localhost.localdomain,127.0.0.1 %s" % (
                 self.repository.entries[hostkey].data)
-        permdata = {'owner':'root', 'group':'root', 'perms':'0644'}
+        permdata = {'owner':'root', 'group':'0', 'perms':'0644'}
         [entry.attrib.__setitem__(key, permdata[key]) for key in permdata]
 
     def build_hk(self, entry, metadata):
@@ -138 +138 @@
             self.logger.error("%s still not registered" % filename)
             raise Bcfg2.Server.Plugin.PluginExecutionError
         keydata = self.repository.entries[filename].data
-        permdata = {'owner':'root', 'group':'root'}
+        permdata = {'owner':'root', 'group':'0'}
         permdata['perms'] = '0600'
         if entry.get('name')[-4:] == '.pub':
             permdata['perms'] = '0644'

src/lib/Server/Plugins/Account.py

@@ -34 +34 @@
         fname = entry.attrib['name'].split('/')[-1]
         entry.text = self.repository.entries["static.%s" % (fname)].data
         entry.text += self.repository.entries["dyn.%s" % (fname)].data
-        perms = {'owner':'root', 'group':'root', 'perms':'0644'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0644'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
 
     def gen_limits_cb(self, entry, metadata):
@@ -43 +43 @@
         superusers = self.repository.entries["superusers"].data.split()
         useraccess = [line.split(':') for line in self.repository.entries["useraccess"].data.split()]
         users = [user for (user, host) in useraccess if host == metadata.hostname.split('.')[0]]
-        perms = {'owner':'root', 'group':'root', 'perms':'0600'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0600'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
         entry.text += "".join(["%s hard maxlogins 1024\n" % uname for uname in superusers + users])
         if "*" not in users:
@@ -56 +56 @@
         superusers += [user for (user, host) in rootlike if host == metadata.hostname.split('.')[0]]
         rdata = self.repository.entries
         entry.text = "".join([rdata["%s.key" % user].data for user in superusers if rdata.has_key("%s.key" % user)])
-        perms = {'owner':'root', 'group':'root', 'perms':'0600'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0600'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
 
     def gen_sudoers(self, entry, metadata):
@@ -66 +66 @@
         superusers += [user for (user, host) in rootlike if host == metadata.hostname.split('.')[0]]
         rdata = self.repository.entries
         entry.text = self.repository.entries['static.sudoers'].data%",".join(superusers)
-        perms = {'owner':'root', 'group':'root', 'perms':'0400'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0400'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]

src/lib/Server/Plugins/Cfg.py

@@ -88 +88 @@
         self.path = path
         self.repopath = repopath
         self.fragments = []
-        self.metadata = {'encoding': 'ascii', 'owner':'root', 'group':'root', 'perms':'0644'}
+        self.metadata = {'encoding': 'ascii', 'owner':'root', 'group':'0', 'perms':'0644'}
         self.paranoid = False
         self.interpolate = False
         

src/lib/Server/Plugins/Vhost.py

@@ -69 +69 @@
             entry.text = 'NO_START=0\n'
         else:
             entry.text = 'NO_START=1\n'
-        perms = {'owner':'root', 'group':'root', 'perms':'0644'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0644'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
                 
     def generateApacheSvc(self, entry, metadata):
@@ -99 +99 @@
             line = line.replace("XXchoiceXX", choice)
             config += line
         entry.text = base64.encodestring(config)
-        perms = {'owner':'root', 'group':'root', 'perms':'0644', 'encoding':'base64'}
+        perms = {'owner':'root', 'group':'0', 'perms':'0644', 'encoding':'base64'}
         [entry.attrib.__setitem__(key, value) for (key, value) in perms.iteritems()]
 
 class Vhost(Plugin):

src/lib/Server/Plugins/TCheetah.py

@@ -14 +14 @@
         self.name = name
         self.properties = properties
         self.states = {'template': False, 'info': False}
-        self.metadata = {'owner': 'root', 'group': 'root', 'perms': '644'}
+        self.metadata = {'owner': 'root', 'group': '0', 'perms': '644'}
 
     def HandleEvent(self, event):
         '''Handle all fs events for this template'''

src/lib/Client/Tools/POSIX.py

@@ -95 +95 @@
         except (OSError, KeyError):
             self.logger.error('User/Group resolution failed for path %s' % (entry.get('name')))
             owner = 'root'
-            group = 'root'
+            group = '0'
         finfo = os.stat(entry.get('name'))
         perms = oct(finfo[ST_MODE])[-4:]
         if entry.get('mtime', '-1') != '-1':