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

The Extension Binds the Ports Used by MindsDB #7

Open
MinuraPunchihewa opened this issue Apr 5, 2024 · 2 comments · Fixed by #12
Open

The Extension Binds the Ports Used by MindsDB #7

MinuraPunchihewa opened this issue Apr 5, 2024 · 2 comments · Fixed by #12
Assignees
Labels
enhancement New feature or request

Comments

@MinuraPunchihewa
Copy link

MinuraPunchihewa commented Apr 5, 2024

When the extension is running on Docker Desktop, the ports that are used by MindsDB (specifically 47334 and 47335), are bound on the host machine. The reason for this is the fact that the extension runs the MindsDB container under-the-hood. Binding these ports make it unfeasible to run MindsDB via another means, locally (through Docker, PIP or from source code).

This may not cause many problems for users as they will probably only try a single method of running MindsDB, but it might hinder development unless users are aware of this. It might also be a problem for someone who is exploring the various methods available for running MindsDB.

Another minor, but connected issue here is that the extension also binds port 8080 incorrectly. There is no requirement for this to use MindsDB and should be fixed.

@MinuraPunchihewa
Copy link
Author

Only partially resolved by #12

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
@MinuraPunchihewa and others