shaal / DrupalPod

Start Drupal contributions with 1 click.
https://shaal.github.io/DrupalPod
MIT License
163 stars 54 forks source link

Database errors testing patch on Drupal 10.0.1 #104

Closed tyler36 closed 1 year ago

tyler36 commented 1 year ago

Describe the bug

I was testing some Drupal 10 readiness patches today and was sometimes was getting the following error.

Notice: running 'php vendor/bin/phpcs --config-set installed_paths vendor/drupal/coder/coder_sniffer' in ddev
Using config file: /var/www/html/vendor/squizlabs/php_codesniffer/CodeSniffer.conf

Config value "installed_paths" updated successfully; old value was "../../drupal/coder/coder_sniffer,../../sirbrillig/phpcs-variable-analysis,../../slevomat/coding-standard"

  Command pm:install was not found. Drush was unable to query the database. As a result, many commands are unavailable. Re-run your command with --debug to see relevant lo  
  g messages.                                                                                                                                                                

Failed to run drush en -y rename_admin_paths: exit status 1
Creating database snapshot drupalpod_20221114085254 
Created database snapshot drupalpod_20221114085254 
Your database state was locally saved, you can revert to it by typing:
ddev snapshot restore --latest
drupalpod-setup.sh script ran for 87 seconds

When the browser pops up, it showed the Drupal setup wizard. I tried different modules , but it seems to happen

To Reproduce

  1. Visit https://www.drupal.org/project/rename_admin_paths/issues/3297951
  2. Click DrupalPod extension and adjust following settings

d10

The error did not display when Drupal Core 10.1.x

Expected behavior

Screenshots

Version and configuration information (please complete the following information)

Additional context

tyler36 commented 1 year ago

Confirmed issue still occuring with: https://www.drupal.org/project/rename_admin_paths/issues/3297951

tyler36 commented 1 year ago

Confirmed on 2 other modules:

tyler36 commented 1 year ago

Tested potential fix with #102

@shaal was able to replicate the issue with Token + D10

From Discord thread ....

The ready-made environment (specifically of D10 + Standard) failed in setup for some reason. When it tries to restore ddev snapshot, it gets a website that is not installed yet