Code review comment for ~cgrabowski/maas:fix_config_permissions_on_init

Revision history for this message
Christian Grabowski (cgrabowski) wrote :

> just asking you:
>
> did you check these other changes to permissions of /etc/maas* ?
>
>
> $ git grep "chown.*etc/maas"
>
> debian/maas-common.postinst: chown -R maas:maas /etc/maas/certificates
> utilities/remote-reinstall:ssh_run "chown -R maas:maas /etc/maas"

Yeah I noticed those, and regiond.conf and rackd.conf have root:maas as well, thinking about it, we may want maas:maas instead.

« Back to merge proposal