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

Port 7700 not binding in production with Docker Swarm #648

Closed
SiroDiaz opened this issue Apr 30, 2020 · 7 comments
Closed

Port 7700 not binding in production with Docker Swarm #648

SiroDiaz opened this issue Apr 30, 2020 · 7 comments

Comments

@SiroDiaz
Copy link

Hi!

i can't create my index and schema in production via Postman/curl because the port 7700 is not being published. I have in my docker-compose.yml in the ports option 7700:7700 but i can't access to that port.
No log errors are thrown in the container with docker logs container_id. I'm using v0.10.
I had to downgrade to a previous version because search and indexing was not working.

Thanks.

@tpayet
Copy link
Member

tpayet commented Apr 30, 2020

Hello @SiroDiaz,

If you run docker run -p 7700:7700 --rm getmeili/meilisearch:v0.10, can you reach the 7700 port?

Can you show me your docker-compose.yml?

@SiroDiaz
Copy link
Author

If you run docker run -p 7700:7700 --rm getmeili/meilisearch:v0.10, can you reach the 7700 port?

Not working. I have checked out the ufw (firewall) and i have allowed port 7700 so that can't be a problem. DB and app services are shared but only MeiliSearch doesn't publish it port.

Can you show me your docker-compose.yml?

service definition:

search:
    image: getmeili/meilisearch:v0.10
    container_name: meilisearch
    restart: always
    depends_on:
      - mysqldb
    volumes:
      - meilidata:/data.ms
    environment:
      MEILI_MASTER_KEY: "xxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx"
      MEILI_ENV: production
    ports:
      - 7700:7700
    networks:
      - app_net

Network definition:

networks:
  app_net:
    external:
      name: app_net

@tpayet
Copy link
Member

tpayet commented May 4, 2020

Not working. I have checked out the ufw (firewall) and i have allowed port 7700 so that can't be a problem. DB and app services are shared but only MeiliSearch doesn't publish it port.

That is weird. I am sorry to ask you something again but could you reach localhost:7700 when running docker run --rm -p 7700:80 -d nginxdemos/hello?

AFAIK MeilISearch image does not expose its 7700 port in the Dockerfile and it is something I am about to fix (#654), but it shouldnt stop you from reaching it when you're specifically asking docker to forward ports.

@Kerollmops
Copy link
Member

Closed by #654, Thank you all!

@Kerollmops
Copy link
Member

Whoops, thought #654 was fixing the bug you get, reopening.

@Kerollmops Kerollmops reopened this May 4, 2020
@SiroDiaz
Copy link
Author

@tpayet docker run --rm -p 7700:80 -d nginxdemos/hello works fine in my production server. But too late, i have moved to manticoresearch.

@Kerollmops
Copy link
Member

Thank you for reporting this issue anyway, have fun!

bors bot added a commit that referenced this issue Jan 16, 2023
662: Enhance word splitting strategy r=ManyTheFish a=akki1306

# Pull Request

## Related issue
Fixes #648 

## What does this PR do?
- [split_best_frequency](https://github.com/meilisearch/milli/blob/55d889522b8b5b6e3dc6d5cf8d14a4f7f62c956b/milli/src/search/query_tree.rs#L282-L301) to use frequency of word pairs near together with proximity value of 1 instead of considering the frequency of individual words. Word pairs having max frequency are considered.

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Co-authored-by: Akshay Kulkarni <akshayk.gj@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants