-
Notifications
You must be signed in to change notification settings - Fork 476
docs: Update SearXNG Web Search Guide for Docker Setup #851
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
Conversation
ℹ️ Documentation PR Guidelines👋 Welcome! This is an automated message posted on all new documentation PRs to help guide our contributors. Just because this comment appeared doesn't mean you have done anything wrong! Please ensure you're using the correct branches: Target branch (where you're merging TO):
Source branch (where you're merging FROM):
If your docs PR depends on a pending PR in open-webui/open-webui:
Please adjust your PR target branch, source branch, and/or draft status accordingly if needed. |
docs/features/web-search/searxng.md
Outdated
|
|
||
| 7. Bring up the container momentarily to generate a fresh settings.yml file: | ||
|
|
||
| If the container name caddy, redis, searxng is already in use, change it to another name of your choice in the docker-compose.yaml file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redis:
container_name: redis
image: docker.io/valkey/valkey:8-alpine
command: valkey-server --save 30 1 --loglevel warning
restart: unless-stopped
networks:
- searxng
volumes:
- valkey-data2:/data
logging:
driver: "json-file"
options:
max-size: "1m"
max-file: "1"
When I first cloned the repo, the docker-compose.yaml had the service and container named redis but was actually using the valkey image (They are similar but not same). Since I already had another container running with the name redis, I changed the container name here to avoid conflicts when running docker-compose. I thought mentioning this might help beginners who are not familiar with Docker errors.
I mentioned all the names just in case there might be containers with the same name running.
Do you think it would be better to remove it? Feel free to give me any feedback!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aha. I would perhaps clarify that.
"If you have multiple containers... then you need to..."
"If a container name is already in use, change it to another name of your choice..."
|
In Step 3, you instruct the user to configure the port to 0.0.0.0:8080. However, in Step 7, the docker-compose.yaml code block introduces a completely new port binding: ports:
You might want to mention that if the user is running Open WebUI on the same Docker network, they don't necessarily need to bind 0.0.0.0; they can just use the container name searxng. |
|
LGTM! THANKS @andrewbbaek |
Updated the SearXNG Web Search documentation to improve the Docker setup guide.
This update reflects recent changes made in the searxng-docker repository and removes unnecessary steps to make it easier for the community to follow.
Key changes include:
I ran the setup multiple times following the updated documentation to ensure it works as expected.