-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
[FEAT] Documentation fix #16
Comments
Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid. |
Mysql-workbench is a separate product to mysql, much like it is advertised on mysql's website itself. It's a tool to run SQL queries against MySQL/MariaDB servers much like Microsoft's SQL Management Studio. There is no need to add adjust the compose to show how to deploy/connect to a MySQL server with this product because it's a desktop application (within a kasmvnc instance) which you would then add your connection details to for your various servers. I would suggest attempting to deploy the container first before asking for clarity/changes so you can see what the application is / how it works. |
mysql-workbench as a application does not bundle mysql, that fact should make it clear if it is included in the image |
The image has Don't get astonished if a potato peeler doesn't say whether it comes with potatoes or not. It's a potato peeler, that's all. You provide your own potatoes. |
FYI.
✏️ NOTE: the most recent MySQL version is not supported. 💥 And the critical part: Once you start MySQL-Workbench (ie. browsing to localhost:3000), you need to setup the database connection. HOWEVER "localhost:3306" does not wok. You have to use the container name "sqldb:3306" |
Is this a new feature request?
Wanted change
Your documentation does not say whether your mysql-workbench image includes MySQL or not. There's literally not one sentence that clarifies this. You could say one of the following:
Please add a
docker-compose.yml
that shows how to connect MySQL-workbench to MySQL server. (Assuming that it is not already included.)Thanks.
(Note: I have not yet downloaded it and tried it out to see exactly what the status is. I'll do that this evening. I'm just astonished about this omission in the documentation.)
Reason for change
Clarity. Need I say more?
Proposed code change
fixed up docker-compose.yml
The text was updated successfully, but these errors were encountered: