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.

Sign up for GitHub

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

Jump to bottom

Remove portmapping for elasticsearch and add proxy hint to readme #102

Open
michael-82 opened this issue Dec 2, 2024 · 0 comments
Open

Remove portmapping for elasticsearch and add proxy hint to readme #102

michael-82 opened this issue Dec 2, 2024 · 0 comments
Assignees
@michael-82
Labels
documentation Improvements or additions to documentation

Comments

@michael-82
Copy link
Contributor

Port 9200 is not needed from outside the docker network, so there is no need to expose it.

In case there is no central proxy configuration for the containers, it might be necessary to set env variables in the init eleasticsearch container. Add a hint to that in the readme
@michael-82 michael-82 added the documentation Improvements or additions to documentation label Dec 2, 2024
@michael-82 michael-82 self-assigned this Dec 2, 2024
michael-82 added a commit that referenced this issue Dec 2, 2024
@michael-82
#102 - Remove portmapping for elasticsearch and add proxy hint to readme
839ca68 
- Remove port mappings from dataportal-elastic service
- Add hint to proxy settings in readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michael-82 michael-82

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

feature/102-remove-portmapping-for-elasticsearch-and-add-proxy-hint-to-readme medizininformatik-initiative/feasibility-deploy
1 participant
@michael-82