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

Update databaseRefreshDdev() to be able to be run from within the local dev container #80

Closed
markdorison opened this issue Jul 5, 2022 · 4 comments · Fixed by #205
Closed
Assignees
Labels
enhancement New feature or request

Comments

@markdorison
Copy link
Contributor

markdorison commented Jul 5, 2022

Description

databaseRefreshDdev() must be run from outside ddev since we currently use the DDEVdb-import command to handle the import. robo dev:refresh also uses this method so currently we are forced to run this from outside the container which means dependencies like PHP must be installed locally.

Proposed Solution

Refactor databaseRefreshDdev() to be able to run from inside the ddev container.

Additional Context

@goranmiric mentioned on a back-end call that he

@markdorison markdorison added the enhancement New feature or request label Jul 5, 2022
@apotek
Copy link
Contributor

apotek commented Aug 16, 2023

I just wanted to note that I run my db refreshes from within Lando all the time by 1) making sure the Lando appserver image has the AWS cli installed and 2) configurating AWS environment variables in .lando/env (and setting the environment key in the Lando config to use that file).

@markdorison
Copy link
Contributor Author

@apotek I am not sure I follow. What is the next step to refresh the db this way? How do you trigger it?

Side note: I should probably rename this issue to refer to DDEV, but I didn't want to cause any confusion so I held off for now.

@apotek apotek changed the title Update databaseRefreshLando() to be able to be run from within the lando container Update databaseRefreshDdev() to be able to be run from within the lando container Jul 18, 2024
@apotek
Copy link
Contributor

apotek commented Jul 18, 2024

Removed references to Lando. We are no longer using Lando. The issue is updated to refer to ddev now.

@apotek apotek linked a pull request Sep 12, 2024 that will close this issue
@apotek apotek self-assigned this Sep 12, 2024
@apotek apotek changed the title Update databaseRefreshDdev() to be able to be run from within the lando container Update databaseRefreshDdev() to be able to be run from within the local dev container Sep 12, 2024
@apotek
Copy link
Contributor

apotek commented Sep 12, 2024

Gah. This is basically a duplicate of #204.

@apotek apotek closed this as completed Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants