Add new setting for how unattended updates will be run

Updated the summary to use "background" instead of "webrequest"

\Drupal\automatic_updates\Validation\StatusChecker::run needs to be updated to run set the stage to the correct Drush stage or Cron stage or UpdateStage

This is not something we can do, because DrushUpdateStage only exists when running Drush (that's where the service is defined). I think it's okay if it stays the way it is -- the point is that it's checking for an unattended update, and CronUpdateStage fulfills that criterion.

Looking good. left some comments

@phenaproxima needs work. that change in CronHandler might affect the tests so not looking at them now

Review posted.

Turns out @phenaproxima already tried enums, didn't work out. Ah well.

Looks great! 😊

It's really good to see this getting done, because as became very explicitly clear at #3362143-20: Install rsync on DrupalCI and run build tests with it → and subsequent comments, we really need these checks to work reliably in all possible setups! 👍

Looks good just 1 problem. See MR comment

Looks good, if tests pass!!!!

Needs work.

The new function test points out a problem. Probably caused by the fact that we run the status checks when install modules.

We were going to do 📌 For web cron updates run each stage life cycle phase in a different request Closed: won't fix first but now we have closed that issue favor trying 📌 For web server dependent unattended updates run the entire life cycle in a separate process that will not be affected by hosting time limits Fixed but that will not before 3.0.x Alpha

So this issue needs to updated for changes in 📌 To prevent conflict and performance problems with other code that runs during cron, decorate the cron service instead of implementing hook_cron Needs work

The 2 ways to run unattended updates

1. drush cron tab
2. Cron route. see 📌 To prevent conflict and performance problems with other code that runs during cron, decorate the cron service instead of implementing hook_cron Needs work

This is looks very good!

I think 1 thing that is missing is that `automatic_updates_cron()` we should return early if method === cli

this because this could be cron is being run by drush cron or another CLI method then running the status checks might get results that don't match when run by web and we don't want to cache those or email.

\Drupal\automatic_updates\Commands\AutomaticUpdatesCommands::autoUpdate already runs status checks so if they selected cli they cron should never need to be run by cron

This also will need a test

Looks great! Thanks @phenaproxima

Automatically closed - issue fixed for 2 weeks with no activity.