Skip to content

Sync db scheme development and production#30

Merged
roschaefer merged 4 commits intomasterfrom
29-sync-db-scheme-development-and-production
Mar 12, 2019
Merged

Sync db scheme development and production#30
roschaefer merged 4 commits intomasterfrom
29-sync-db-scheme-development-and-production

Conversation

@matboehm
Copy link
Copy Markdown
Contributor

@matboehm matboehm commented Mar 9, 2019

Close #29


Synchronize the Docker MySQL database with the Production live database.

@matboehm matboehm added the bug Something isn't working label Mar 9, 2019
@matboehm matboehm self-assigned this Mar 9, 2019
Copy link
Copy Markdown
Contributor

@roschaefer roschaefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did run the changes this time but I think this should be OK. @matthib can you double check the bash script please before merge?

Comment thread README.md Outdated
2. Go to the newly created Clock-of-Change-API directory (`cd Clock-of-Change-API` in the console)
3. Run `docker-compose up`. This will build the Docker container on first startup and run it. This can take a while, but after some time you should see the Clock of Change ticking.
4. To create an initial API key and sample entries, run `docker-compose exec db seed.sh`. The initial API key you can use with your requests will have the value `secret`.
4. To create an initial API key and sample entries, run `docker-compose exec db bash seed.sh`. The initial API key you can use with your requests will have the value `secret`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks wrong. You sync seed.sh in a folder under $PATH making seed.sh an executable program.

Copy link
Copy Markdown
Contributor Author

@matboehm matboehm Mar 10, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@roschaefer

Can change it back, if that is better.
Unfortunately I get the error:

OCI runtime exec failed: exec failed: container_linux.go:344: starting container process caused "no such file or directo
ry": unknown

when trying to run it from $PATH directly. With running first bash and then passing seed.sh as an argument, the seed.sh can successfully be found in the $PATH.

This could be a problem with my local environment, as I use the Windows Git Bash as my console.

@matboehm matboehm changed the title [WIP] Sync db scheme development and production Sync db scheme development and production Mar 11, 2019
@matboehm
Copy link
Copy Markdown
Contributor Author

@roschaefer Changed back the command in the Readme.md. If the rest of the code seems ok, please merge. Thanks! 👍

@roschaefer roschaefer merged commit ca45054 into master Mar 12, 2019
@roschaefer roschaefer deleted the 29-sync-db-scheme-development-and-production branch March 12, 2019 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working PR: merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants