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

Request: More options for MySQL preset #215

Open
zdunecki opened this issue Jul 6, 2021 · 2 comments
Open

Request: More options for MySQL preset #215

zdunecki opened this issue Jul 6, 2021 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@zdunecki
Copy link

zdunecki commented Jul 6, 2021

Hey,

I see awesome MySQL preset but at case in my company there are still missing some features. We need three things:

  • Run in root mode.
  • Don't choose database at start because we have multiple db's in single MySQL instance.
  • Mount Docker volumes. We use internal cache for seeds and it's saved in Docker volume.

Here's a quick sneak peek what I'm talking about in code:
https://github.com/orlangure/gnomock/compare/master...zdunecki:new-options-for-mysql-preset?expand=1

If you like this idea I'm able to finish this PR and add more tests, comments etc.

@orlangure
Copy link
Owner

Hi @zdunecki and thanks for the ideas (and for using Gnomock 😻)

Numbers 1 and 2 are great and fairly simple, I think these 2 features can be merged very soon. Everything just needs to be tested properly, at least the happy flows, and documented (including swagger spec for non-Go users).

Number 3 is also very useful, but requires a deeper look. There is a way to run gnomock inside a container to access it from other languages using the API, and I'm not sure implementing volumes would be straightforward. I think it would "just work", but it needs a bit more attention. If you could confirm that it is possible to mount volumes in the same way using gnomock container as it is in Go, that would make adding this feature even easier 😼

@zdunecki
Copy link
Author

zdunecki commented Jul 6, 2021

@orlangure

Thanks for reply.

Number 3 looks very easy to implement, I did it here.

Gnomock itself require Docker engine and volumes looks like passing volume id/name similar to bind mount but it's not a local path but Docker area path. I'll test above features in Go and API mode, add tests and make code more robust.

Currently I've switched to another ticket at my company but I hope I'll back to this soon because we need this features internally.

@orlangure orlangure added the enhancement New feature or request label Jul 6, 2021
@orlangure orlangure added good first issue Good for newcomers help wanted Extra attention is needed labels Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants