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

Always clear the cache after db-import #22

Open
dalin- opened this issue Feb 29, 2024 · 4 comments
Open

Always clear the cache after db-import #22

dalin- opened this issue Feb 29, 2024 · 4 comments
Assignees

Comments

@dalin-
Copy link
Contributor

dalin- commented Feb 29, 2024

Otherwise you'll get random hard-to-debug errors.

@dalin-
Copy link
Contributor Author

dalin- commented Feb 29, 2024

Or maybe, even better yet, maybe we should always run fire cim to get local config installed.

@dalin-
Copy link
Contributor Author

dalin- commented Mar 8, 2024

Actually we need to do both.

fire import-db
fire cim

Results in no config to import because there's old data stored in Redis.

So we need to both drush cr && drush cim -y after a DB import.

@rigoucr
Copy link
Contributor

rigoucr commented Aug 1, 2024

Hi @dalin- @Yuvania the Idea/goal behind the db-import command is to import the DB as close as posible to the state that it has over the the selected env, this in case we need to run some debugging over DB data without the influence of any local config change , so , IMporting data Right after importing DB is not a option.

I think this change goes more into the the command src/Robo/Plugin/Commands/DrushDeployCommand.php and add just a drush cr at the very beggining of the command exec, thx

@dalin-
Copy link
Contributor Author

dalin- commented Aug 2, 2024

@rigoucr
I could be convinced not to do a cim after import. But I really suggest doing cr. I've often see people run into strange issues because they don't know how Redis works and don't realize that this step is required.

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

3 participants