launchpad-reviewers team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

Julian Edwards has proposed merging lp:~julian-edwards/maas/cluster-celery-user into lp:~maas-maintainers/maas/packaging.

Commit message:
Ensure the user for the cluster controller processes is "maas".  Previously it ran as root.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~julian-edwards/maas/cluster-celery-user/+merge/127240
-- 
https://code.launchpad.net/~julian-edwards/maas/cluster-celery-user/+merge/127240
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~julian-edwards/maas/cluster-celery-user into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/maas-cluster-controller.maas-cluster-celery.upstart'
--- debian/maas-cluster-controller.maas-cluster-celery.upstart	2012-09-29 06:40:45 +0000
+++ debian/maas-cluster-controller.maas-cluster-celery.upstart	2012-10-01 11:30:49 +0000
@@ -7,6 +7,8 @@
 
 start on filesystem and net-device-up
 stop on runlevel [016]
+setuid maas
+setgid maas
 
 env CONFIG_FILE=/etc/maas/maas_cluster.conf