Last run time not updated on individual job run

Strangely, I have exactly the opposite problem: The "Last Run" time is updated when I run the Cron job manually, but never when it is run as part of a "drush cron-run". A new issue or the same?

We shouldn't do this manually.

I've been dropping those methods and moving things around, but we're not quite there yet.

I think we should try to move most of \Drupal\ultimate_cron\Plugin\ultimate_cron\Launcher\SerialLauncher::launch() into run(). It probably makes sense to expose $init_message as an optional argument for run() as that is tied to the threading stuff of the launcher. But locking and logging seems like sometihng that should always happen.

Yes, that fail is unrelated. #2804919: Rows in past_event_data not deleted? has the fix for the same test fail in past, please update ultimate_cron accordingly in a separate issue.

1. +++ b/src/CronJobInterface.php
   @@ -224,8 +224,11 @@ interface CronJobInterface extends ConfigEntityInterface {
   +   * @param string $init_message
   +   *   The launch message.
   

   As discussed, despite adding specific messages, lets make the argument optional, default to NULL and in the method, if null, use the launched manually string. Then document as (optional) and mention the default string.

2. +++ b/ultimate_cron.drush.inc
   @@ -330,7 +330,8 @@ function drush_ultimate_cron_cron_run($name = NULL) {
          if ($force || $job->isScheduled()) {
   -        $job->run();
   +        $init_message = t('Launched by drush');
   +        $job->run($init_message);
   

   You can just pass t() directly, you don't need a variable.

Thanks, committed.