sts-sponsors team mailing list archive

[Christian Grabowski <mp+434600@xxxxxxxxxxxxxxxxxx>]

Christian Grabowski has proposed merging ~cgrabowski/maas:fix_config_permissions_on_init into maas:master.

Commit message:
change ownership of /etc/maas to root:maas



Requested reviews:
  MAAS Maintainers (maas-maintainers)

For more details, see:
https://code.launchpad.net/~cgrabowski/maas/+git/maas/+merge/434600
-- 
Your team MAAS Maintainers is requested to review the proposed merge of ~cgrabowski/maas:fix_config_permissions_on_init into maas:master.

diff --git a/debian/maas-common.postinst b/debian/maas-common.postinst
index 67e9787..6e446f4 100755
--- a/debian/maas-common.postinst
+++ b/debian/maas-common.postinst
@@ -17,6 +17,10 @@ configure_logging() {
     chown maas:maas /var/log/maas/maas.log
 }
 
+configure_conf_dir() {
+    chown root:maas /etc/maas
+}
+
 configure_lib_dir() {
     # Ensure lib dir has correct permissions
     chown maas:maas /var/lib/maas/
@@ -42,6 +46,7 @@ add_user_group(){
 disable_nginx
 add_user_group
 configure_logging
+configure_conf_dir
 configure_lib_dir
 
 #DEBHELPER#