Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GOVCMSD10-1163] Fix local mysql import error #444

Open
wants to merge 1 commit into
base: 3.x-develop
Choose a base branch
from

Conversation

jackwrfuller
Copy link
Contributor

The Simplesamlphp and drupal database tables use different collations.

Simplesaml uses utf8mb4_0900_ai_ci but drupal uses utf8mb4_general_ci.

This causes errors in local test sites when attempting to import lagoon database backups to a local site. In particular, the following error is received:

ahoy mysql-import ~/Downloads/vanilla-govcms-10-beta-test-mariadb-prebackuppod.mariadb.sql

Do you really want to drop all tables in the database drupal? (yes/no) [yes]:
> yes

> ERROR 1273 (HY000) at line 25: Unknown collation: 'utf8mb4_0900_ai_ci'

In Process.php line 270:

  The command "mysql --defaults-file=/tmp/drush_ONjodh --database=drupal --ho  
  st=mariadb --port=3306 -A" failed.                                           

  Exit Code: 1(General error)                                                  

  Working directory: /app/web                                                  

  Output:                                                                      
  ================                                                             

  Error Output:                                                                
  ================                                                             
  ERROR 1273 (HY000) at line 25: Unknown collation: 'utf8mb4_0900_ai_ci'

The change I propose to fix this is to explicitly tell simplesamlphp to use the utf8mb4_general_ci collation.

@pandaskii pandaskii changed the title Fix local mysql import error [GOVCMSD10-1163] Fix local mysql import error Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants