Problem/Motivation

For Core MVP we had initially planned to only support patch level updates.

I chatted with @xjm recently who suggested we could have a non-UI config setting to allow minor updates.

I think this sounds great.

Currently we disallow updates that have database updates required during cron. We should not allow minor updates via cron. They will likely need database update anyways.

Proposed resolution

  1. add allow_core_minor_updates config switch. This switch is core specific to allow core have a different switch once we support config updates in core
  2. Add warning at the start of minor update with reasons why the user should be more careful. Is there an existing documentation page for this?
  3. Change \Drupal\automatic_updates\Validator\UpdateVersionValidator to allow minor updates if config is set and not in cron
  4. Add warning in update apply form if an update DB update is detected.
  5. re-route the user to update.php after the update if there are DB updates.

4 and 5 are existing problems because only stop updates with DB updates in cron so we should solve those first.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

tedbow created an issue. See original summary.

tedbow’s picture

Status: Active » Needs review
Issue tags: +Needs tests
tedbow’s picture

Issue summary: View changes

phenaproxima made their first commit to this issue’s fork.

phenaproxima’s picture

Issue tags: -Needs tests
phenaproxima’s picture

Title: Add non-UI config option to allow attended minor core updates » [PP-1] Add non-UI config option to allow attended minor core updates
Status: Needs review » Postponed

After discussion with @tedbow, we agreed to postpone this on #3260368: Improve the UX for updates that require database changes.

phenaproxima’s picture

Title: [PP-1] Add non-UI config option to allow attended minor core updates » Add non-UI config option to allow attended minor core updates
Status: Postponed » Needs review

Blocker is in.

tedbow’s picture

Is there a documentation page we link to if I minor update is recommended? Something to explain the differences between patch and minor updates and why a minor update could be more disruptive?

Maybe since this is hidden feature this could be a follow up

tedbow’s picture

Status: Needs review » Reviewed & tested by the community

  • phenaproxima committed 10b7481 on 8.x-2.x authored by tedbow
    Issue #3259664 by tedbow: Add non-UI config option to allow attended...
phenaproxima’s picture

Status: Reviewed & tested by the community » Fixed

Forgot to credit myself; whoops!

Status: Fixed » Closed (fixed)

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