[DrupalCon] Automatic Updates alpha test result

Created on 7 June 2023, over 1 year ago
Updated 17 August 2023, over 1 year ago

Goal: Update 10.0.0 to 10.1.0-beta1 via Automatic Updates

If you had a problem, what were the steps you took, starting from when you added the Automatic Updates module to your Drupal project (see testing instructions )? Include anything that seems relevant, including commands you ran, links you clicked on, output logs, relevant config files, screenshots, screen recordings, etc.

When updating from 10.0.0 to 10.1.0-beta1 using Automatic Updates, I get a Warning message:

Your site does not pass some readiness checks for automatic updates. Depending on the nature of the failures, it might affect the eligibility for automatic updates.
* Possible database updates have been detected in the following extensions. ...

My expectation is that there would be explicit instructions to trigger db updates (either through the UI or the command line) before proceeding with the core update. Instead, I get the statement

Drupal core will be updated to 10.1.0-beta1

and a strong recommendation to backup my database and site before proceeding.

<!--break-->

What was your environment like?
Please answer as many of the following questions as you can.

Was this a local development environment, or a remote server?: Gitpod

Are you using Lando, DDEV, or another Docker-based environment?: DDEV

What operating system were you using?

What web server are you using? Chrome

What PHP version are you using (see https://www.drupal.org/i/3275810#confirm-php-version )? PHP 8.1.3

What version of Composer are you using (see https://www.drupal.org/i/3275810#confirm-composer-version )? Composer 2.5.7

Did you accomplish your goal? Were the instructions clear? Did you observe any bugs or errors, or other issues? Did you need to do any workarounds?

No, I did not accomplish the goal of updating from 10.0.0 to 10.1.0-beta1. I went ahead with the core update without doing any database update, and I got an error message:

An AJAX HTTP error occurred.
HTTP Result Code: 500
Debugging information follows.
Path: /batch?id=7&op=do_nojs&op=do
StatusText: error
ResponseText: The website encountered an unexpected error. Please try again later.

I checked the recent log messages.
1st message:

Type php
Date Wednesday, June 7, 2023 - 18:35
User admin
Location /batch?_format=json&id=7&op=do
Referrer /batch?id=7&op=start
Message ArgumentCountError: Too few arguments to function Drupal\big_pipe\Render\BigPipeResponseAttachmentsProcessor::__construct(), 8 passed in /var/www/html/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 263 and exactly 9 expected in Drupal\big_pipe\Render\BigPipeResponseAttachmentsProcessor->__construct() (line 55 of /var/www/html/web/core/modules/big_pipe/src/Render/BigPipeResponseAttachmentsProcessor.php).
Severity Error
Hostname 192.168.125.70

2nd message:

Error message
There is a security update available for your version of Drupal. To ensure the security of your server, you should update immediately! See the available updates page for more information and to update your software.
Type theme
Date Wednesday, June 7, 2023 - 18:36
User admin
Location /admin/automatic-update-ready/lrTcuOdIn1Z7m5CaQ44FXYky3hDl7ffpiXa9SD6iN44
Referrer /batch?id=7&op=start
Message Theme hook big_pipe_interface_preview not found.
Severity Warning
Hostname 192.168.125.70

There were seven more Error messages, all the same.

Problem with these instructions? Anything else we should know?

Screen shots of update warning message without instructions to update database

🐛 Bug report
Status

Needs work

Version

3.0

Component

Code

Created by

🇺🇸United States uptil4music

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @uptil4music
  • 🇺🇸United States tedbow Ithaca, NY, USA

    It would good if someone could test the instructions https://www.drupal.org/project/automatic_updates/issues/3365152 🌱 3.0.0 Alpha Testing Instructions Active
    but updating directly from 10.0.0 to 10.1.0-alpha1, or whatever the current version of 10.1.x is. To see if we get the same error.

    I thought maybe BigPipeResponseAttachmentsProcessor changed in 10.1 but I couldn't find anything. But it is likely there might related change that affects this.

  • 🇺🇸United States tedbow Ithaca, NY, USA
  • 🇺🇸United States tedbow Ithaca, NY, USA

    another person got this problem with Big Pipe in 🐛 [DrupalCon] Automatic Updates alpha test result Active

    My guess would be there is some sort of change in 10.1 that doesn't work our batch update system.

  • 🇮🇳India kunal.sachdev

    I tried updating my site from 10.0.0 to 10.1.0-rc1 and I also get the warning message on ready to update page see and when I hit continue it shows me progress bar of applying updates and then update.php automatically runs and when I go back to site it is updated to 10.1.0-rc1.

  • 🇮🇳India omkar.podey

    I wasn't even able to complete my update,
    1.

    2.

    3.When i try to check the status report.

  • 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
    Message ArgumentCountError: Too few arguments to function Drupal\big_pipe\Render\BigPipeResponseAttachmentsProcessor::__construct(), 8 passed in /var/www/html/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 263 and exactly 9 expected in Drupal\big_pipe\Render\BigPipeResponseAttachmentsProcessor->__construct() (line 55 of /var/www/html/web/core/modules/big_pipe/src/Render/BigPipeResponseAttachmentsProcessor.php).
    

    🤯 Wow.

    Fortunately I'm a BigPipe maintainer! 🤓

    Unfortunately I'm puzzled. 😅

    BigPipe is installed by default. We all run it when developing Automatic Updates. NOPE, build tests use minimal!

    We should be running build tests with standard.

    #4:

    My guess would be there is some sort of change in 10.1 that doesn't work our batch update system.

    Nothing BigPipe-related changed in 10.1. But what did change significantly is how assets are generated and served.

    My bet would be that's what's causing this. And I bet we're hitting some sort of race condition with enabling maintenance mode that makes assets generation fail (while maintenance mode is on).

    #7: can you please check the watchdog table, where type = 'php'? 🙏 Because without a precise error message, we're nowhere.

  • Status changed to Needs work over 1 year ago
  • 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
  • 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺

    Actually, no, the root cause here is simpler: 🐛 Stampedes and cold cache performance issues with css/js aggregation Fixed landed a change in August 2022, which indeed added a new required argument to \Drupal\big_pipe\Render\BigPipeResponseAttachmentsProcessor::__construct().

    The fact that this error message appears then leaves only one possible conclusion: the dependency injection container is not being rebuilt immediately after copying the files from "stage" to "active": at least one request is getting through to "active" before the container is rebuilt, which is causing this error message. 🫣

  • Assigned to tedbow
  • 🇺🇸United States tedbow Ithaca, NY, USA

    Assigning to myself to make follow-up issues. Thanks @kunal.sachdev and @omkar.podey for testing and thanks @Wim Leers for the investigation

  • 🇺🇸United States tedbow Ithaca, NY, USA

    Created
    📌 In Core build tests have core module change a service class constructor Active
    📌 Change build tests to install Standard profile Needs work

    Still needs follow-up for messaging issues(or to find existing issues which I think there is)

    @UpTil4Music thanks for testing, credited you on those issues

  • 🇺🇸United States xjm
Production build 0.71.5 2024