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

9.2.0 broke sql-sync #3417

Closed
grasmash opened this issue Feb 27, 2018 · 6 comments
Closed

9.2.0 broke sql-sync #3417

grasmash opened this issue Feb 27, 2018 · 6 comments

Comments

@grasmash
Copy link
Contributor

My test suites for acquia/blt began failing immediately after drush/drush 9.2.0 was released.

Executing:

./vendor/bin/drush sql-sync @default.clone @self --structure-tables-key=lightweight --create-db --no-interaction --ansi

Outputs:

You will destroy data in drupal and replace with data from drupal3.
 [notice] Starting to create database on target.
Creating database drupal. Any existing database will be dropped!
 // Do you really want to continue?: yes.                                       
 [notice] Starting to dump database on source.
 [success] Database dump saved to 1.gz
 [notice] Starting to import dump file onto target database.
 [error]  File(s) not found: 1.gz 
 [error]  Failed to import 1.gz into target. 
@grasmash
Copy link
Contributor Author

grasmash commented Feb 27, 2018

I see that some related things changed in 9.1.0...9.2.2

@weitzman
Copy link
Member

Compare link is broken. Perhaps add -vvv for more debug info. A bit hard to know whats wrong from the provided log. Also helpful to share output of drush sa @default.clone and drush status

@rodrigoaguilera
Copy link
Contributor

I have been trying to debug this for the last hours I have nothing except that "1.gz" should be DATABASENAME_DATE.gz

@greg-1-anderson
Copy link
Member

sql-sync should be getting the filename from the result of the sql-dump command.

@rodrigoaguilera
Copy link
Contributor

For more context the sites that I am trying to sync both have local aliases

@weitzman
Copy link
Member

That filename was a good clue!. Fix in #3418.

FYI the behavior of --no-interaction was fixed in this release as well so please be on the lookout for problems there.

Closing this in favor of #3418

weitzman added a commit that referenced this issue Feb 27, 2018
* Fix sql-sync dump filename.

* Disable opcache as it doesn't help on the CLI.
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

No branches or pull requests

4 participants