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

DO NOT MERGE: Empty commit #603

Closed
wants to merge 2 commits into from
Closed

DO NOT MERGE: Empty commit #603

wants to merge 2 commits into from

Conversation

mrdavidburns
Copy link
Member

Checking Tugboat Build

@github-actions github-actions bot temporarily deployed to pantheon-pr-603 July 9, 2024 14:56 Destroyed
@github-actions github-actions bot temporarily deployed to pantheon-pr-603 July 9, 2024 15:13 Destroyed
@mrdavidburns
Copy link
Member Author

Confirming that sessions table doesn't exist.

MariaDB [tugboat]> show tables;
+----------------------------------+
| Tables_in_tugboat                |
+----------------------------------+
| block_content                    |
| block_content__body              |
| block_content_field_data         |
| block_content_field_revision     |
| block_content_revision           |
| block_content_revision__body     |
| cache_bootstrap                  |
| cache_config                     |
| cache_container                  |
| cache_default                    |
| cache_discovery                  |
| cachetags                        |
| comment                          |
| comment__comment_body            |
| comment_entity_statistics        |
| comment_field_data               |
| config                           |
| file_managed                     |
| file_usage                       |
| help_search_items                |
| history                          |
| key_value                        |
| menu_link_content                |
| menu_link_content_data           |
| menu_link_content_field_revision |
| menu_link_content_revision       |
| menu_tree                        |
| node                             |
| node__body                       |
| node__comment                    |
| node__field_image                |
| node__field_tags                 |
| node_access                      |
| node_field_data                  |
| node_field_revision              |
| node_revision                    |
| node_revision__body              |
| node_revision__comment           |
| node_revision__field_image       |
| node_revision__field_tags        |
| path_alias                       |
| path_alias_revision              |
| router                           |
| search_dataset                   |
| search_index                     |
| search_total                     |
| semaphore                        |
| sequences                        |
| shortcut                         |
| shortcut_field_data              |
| shortcut_set_users               |
| taxonomy_index                   |
| taxonomy_term__parent            |
| taxonomy_term_data               |
| taxonomy_term_field_data         |
| taxonomy_term_field_revision     |
| taxonomy_term_revision           |
| taxonomy_term_revision__parent   |
| user__roles                      |
| user__user_picture               |
| users                            |
| users_data                       |
| users_field_data                 |
| watchdog                         |
+----------------------------------+
64 rows in set (0.002 sec)

@deviantintegral
Copy link
Member

deviantintegral commented Jul 10, 2024

# task --summary sync
task: sync

Sync a database from production and import it

commands:
 - ./vendor/bin/drush site:install -y
 - echo "🧹 Sanitising database"
 - ./vendor/bin/drush sql:sanitize --yes

@mrdavidburns I think this is a Drupal 10.3 change: https://www.drupal.org/node/3431286

\Drupal\Core\Session\SessionHandler now creates the table the first time as needed. Since we're calling sanitize before trying to log in, the table isn't there.

I think Drush needs to be updated so sql:sanitize will fail gracefully if the sessions table hasn't been created yet.

@penyaskito
Copy link
Member

@penyaskito
Copy link
Member

This has been backported to drush 12.x, maybe not released yet? drush-ops/drush#6041

@beto-aveiga
Copy link
Collaborator

@mrdavidburns, here is the PR for Drush 13. PR Checks are passing.
#607
I'm unclear if that is what you need, and if so, what would be next.

@mrdavidburns
Copy link
Member Author

Closing in favor of #608

@mrdavidburns mrdavidburns deleted the DNM--test-tugboat branch July 16, 2024 11:57
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.

4 participants