launchpad-reviewers team mailing list archive

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

Jeroen T. Vermeulen has proposed merging lp:~jtv/maas/bug-1051454 into lp:maas.

Requested reviews:
  MAAS Maintainers (maas-maintainers)
Related bugs:
  Bug #1051454 in MAAS: "report_boot_images task invoked on incorrect module"
  https://bugs.launchpad.net/maas/+bug/1051454

For more details, see:
https://code.launchpad.net/~jtv/maas/bug-1051454/+merge/124562

Scott noticed that this wasn't working, as per log: http://paste.ubuntu.com/1205716/

This is such a simple problem & fix, I think speed is more helpful than a pre-imp.

It is to be backported to 1.1 as well.


Jeroen
-- 
https://code.launchpad.net/~jtv/maas/bug-1051454/+merge/124562
Your team MAAS Maintainers is requested to review the proposed merge of lp:~jtv/maas/bug-1051454 into lp:maas.

=== modified file 'etc/celeryconfig.py'
--- etc/celeryconfig.py	2012-09-12 07:06:24 +0000
+++ etc/celeryconfig.py	2012-09-16 01:04:20 +0000
@@ -76,7 +76,7 @@
     # XXX JeroenVermeulen 2012-09-12, bug=1039366: this task should run
     # only on the master worker.
     'report-boot-images': {
-        'task': 'provisioningserver.report_boot_images',
+        'task': 'provisioningserver.tasks.report_boot_images',
         'schedule': timedelta(minutes=5),
     },
 }