← Back to team overview

cloud-init-dev team mailing list archive

[Merge] ~rjschwei/cloud-init:setPerm into cloud-init:master

 

Robert Schweikert has proposed merging ~rjschwei/cloud-init:setPerm into cloud-init:master.

Requested reviews:
  cloud-init commiters (cloud-init-dev)

For more details, see:
https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/337666

On SUSE distributions the neither the "adm" nor the "wheel" group are set up by default thus the user will find a warning message in the log file:

2018-02-12 17:06:33,648 - stages.py[WARNING]: Failed changing perms on '/var/log/cloud-init.log'. tried: syslog:adm,root:adm,root:wheel. Unknown user or group: 'getgrnam(): name not found: wheel'

Change the logfile permissions to root:root in the cloud-init default configuration file.

https://bugzilla.suse.com/show_bug.cgi?id=1080595
-- 
Your team cloud-init commiters is requested to review the proposed merge of ~rjschwei/cloud-init:setPerm into cloud-init:master.
diff --git a/config/cloud.cfg.tmpl b/config/cloud.cfg.tmpl
index 32de9c9..3ab681e 100644
--- a/config/cloud.cfg.tmpl
+++ b/config/cloud.cfg.tmpl
@@ -5,6 +5,9 @@
 {% if variant in ["freebsd"] %}
 syslog_fix_perms: root:wheel
 {% endif %}
+{% if variant in ["suse"] %}
+syslog_fix_perms: root:adm
+{% endif %}
 # A set of users which may be applied and/or used by various modules
 # when a 'default' entry is found it will reference the 'default_user'
 # from the distro configuration specified below

Follow ups