dashboard: fix status reporting in init.py

circle/dashboard/management/commands/init.py

@@ -17,6 +17,7 @@
 
 
 from __future__ import unicode_literals, absolute_import
 from __future__ import unicode_literals, absolute_import
 
 
+import logging
 from optparse import make_option
 from optparse import make_option
 
 
 from django.contrib.auth.models import User
 from django.contrib.auth.models import User
@@ -28,6 +29,9 @@ from storage.models import DataStore
 from vm.models import Lease
 from vm.models import Lease
 
 
 
 
+logger = logging.getLogger(__name__)
+
+
 class Command(BaseCommand):
 class Command(BaseCommand):
     option_list = BaseCommand.option_list + (
     option_list = BaseCommand.option_list + (
         make_option('--force', action="store_true"),
         make_option('--force', action="store_true"),
@@ -49,18 +53,18 @@ class Command(BaseCommand):
         qs = model.objects.filter(**{field: value})[:1]
         qs = model.objects.filter(**{field: value})[:1]
         if not qs.exists():
         if not qs.exists():
             obj = model.objects.create(**kwargs)
             obj = model.objects.create(**kwargs)
-            self.changed.append('New %s: %s' % (model, obj))
+            logger.info('New %s: %s', model, obj)
+            self.changed = True
             return obj
             return obj
         else:
         else:
             return qs[0]
             return qs[0]
 
 
 # http://docs.saltstack.com/en/latest/ref/states/all/salt.states.cmd.html
 # http://docs.saltstack.com/en/latest/ref/states/all/salt.states.cmd.html
     def print_state(self):
     def print_state(self):
-        changed = "yes" if len(self.changed) else "no"
-        print "\nchanged=%s comment='%s'" % (changed, ", ".join(self.changed))
+        print "\nchanged=%s" % ("yes" if self.changed else "no")
 
 
     def handle(self, *args, **options):
     def handle(self, *args, **options):
-        self.changed = []
+        self.changed = False
 
 
         if (DataStore.objects.exists() and Vlan.objects.exists()
         if (DataStore.objects.exists() and Vlan.objects.exists()
                 and not options['force']):
                 and not options['force']):