Closed (fixed)
Project:
Hosting
Version:
6.x-2.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
8 Jul 2013 at 10:53 UTC
Updated:
12 Jun 2014 at 08:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
helmo commentedComment #2
ergonlogicThis has made the entire log grey for me.
Comment #3
ergonlogicfyi, these are all the task types I see:
Comment #4
ergonlogicOddly, all the rows are getting a class of 'hosting-info', but I don't see why...
Comment #5
ergonlogicSorry, almost everything was set to 'hosting-info'. I think 'message' should remain a 'hosting-success', no?
Comment #6
ergonlogicRevised patch for #5 attached.
If we go with what's in _drush_print_log(), then we should probably change all instances of
drush_log(..., 'message')todrush_log(..., 'success'), or something.Comment #7
ergonlogicFixed in Provision with d5a8862, and Hosting with 72f7db1.
As per #5, I switched to using 'success' rather than 'message' in Provision. This change means that 'status-info' is the default, so, for example, memory rows are no longer green. However, this better highlights the operations that are actually successful, and reduces the noise.
I think we could still tweak these a bit. For example, I think 'rollback' messages could be flagged as hosting-warning, and maybe 'debug' too.
Comment #8
helmo commentedI think you forgot to push to provision... I can't find d5a8862 there.
Drush also uses the cancel type... in 8.x-6.x ... so that's not really aegir specific anymore.
includes/drush.inc: drush_log($msg ? $msg : dt('Aborting.'), 'cancel');Comment #9
helmo commentedThe hosting-dispatch cron-job now started mailing:
Updated task status to "Successful" [success]Changing it to 'info' seems better:
Comment #10
ergonlogicOops, I'd missed a push from Provision. I also cleaned up the rest, as per your suggestions.
Comment #11
anarcat commentedGreat!
I think having rollbacks as warnings is a great idea, although they will probably always end up in red because a rollback usually happens after an error, which should be red. Debug should be inconspicuous.
Comment #12
anarcat commentedoups, didn't mean to change status.
Comment #13
ergonlogicok, I made rollbacks orange, even though the overall task will remain red.
Fixed in 7e31ccc