launchpad-reviewers team mailing list archive

["Jeroen T. Vermeulen" <jtv@xxxxxxxxxxxxx>]

Jeroen T. Vermeulen has proposed merging lp:~jtv/maas/p-sru-pkg-bug-1089819 into lp:~maas-maintainers/maas/packaging.precise.sru.

Commit message:
Backport packaging r158: Export CLUSTER_UUID from the cluster controller's upstart jobs to their respective daemon child processes.

Requested reviews:
  Jeroen T. Vermeulen (jtv)
Related bugs:
  Bug #1089819 in MAAS: "CLUSTER_UUID not exported to pserv & cluster-controller processes"
  https://bugs.launchpad.net/maas/+bug/1089819

For more details, see:
https://code.launchpad.net/~jtv/maas/p-sru-pkg-bug-1089819/+merge/139659
-- 
https://code.launchpad.net/~jtv/maas/p-sru-pkg-bug-1089819/+merge/139659
Your team MAAS Maintainers is subscribed to branch lp:~maas-maintainers/maas/packaging.precise.sru.

=== modified file 'debian/maas-cluster-controller.maas-cluster-celery.upstart'
--- debian/maas-cluster-controller.maas-cluster-celery.upstart	2012-10-03 03:43:27 +0000
+++ debian/maas-cluster-controller.maas-cluster-celery.upstart	2012-12-13 10:46:23 +0000
@@ -21,6 +21,10 @@
 end script
 
 script
-    . $CONFIG_FILE  # sets MAAS_URL
+    # Prepare settings.
+    . $CONFIG_FILE
+    # Allow the cluster-controller process to read CLUSTER_UUID as set
+    # in that config file.
+    export CLUSTER_UUID
     exec /usr/sbin/maas-provision start-cluster-controller $MAAS_URL -u maas -g maas
 end script

=== modified file 'debian/maas-cluster-controller.maas-pserv.upstart'
--- debian/maas-cluster-controller.maas-pserv.upstart	2012-12-12 07:14:04 +0000
+++ debian/maas-cluster-controller.maas-pserv.upstart	2012-12-13 10:46:23 +0000
@@ -23,6 +23,9 @@
 script
     # Prepare settings.
     . $CONFIG_FILE
+    # Allow the tftpd process to read CLUSTER_UUID as set in that config
+    # file.
+    export CLUSTER_UUID
     # To add options to your daemon, edit the line below:
     exec /usr/bin/twistd -n --uid=maas --gid=maas --pidfile=/run/maas-pserv.pid --logfile=/dev/null maas-pserv --config-file=/etc/maas/pserv.yaml
 end script