Skip to content

Comments

Fix Ollama deployment#371

Merged
kylegordon merged 4 commits intomasterfrom
fix-ollama
Feb 24, 2026
Merged

Fix Ollama deployment#371
kylegordon merged 4 commits intomasterfrom
fix-ollama

Conversation

@kylegordon
Copy link
Owner

No description provided.

@kylegordon kylegordon requested a review from Copilot February 23, 2026 14:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes the Ollama deployment by stabilizing the image version and adding explicit environment configuration to enable the web UI to properly connect to the Ollama backend service.

Changes:

  • Pins the open-webui image from :main tag to stable version v0.8.3
  • Adds environment variables to configure the web UI URL, disable authentication, and set Ollama backend connection
  • Includes commented configuration examples for alternative deployment scenarios

Comment on lines +95 to +96
# extra_hosts:
# - host.docker.internal:other_machine-like172.24.32.5
Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate extra_hosts configuration detected. Lines 87-88 define extra_hosts with host.docker.internal:host-gateway, and then lines 95-96 comment out another extra_hosts section. This creates confusion about which configuration is active. Consider removing the commented-out duplicate at lines 95-96 since the active configuration at lines 87-88 already provides the same functionality.

Suggested change
# extra_hosts:
# - host.docker.internal:other_machine-like172.24.32.5

Copilot uses AI. Check for mistakes.
- host.docker.internal:host-gateway
restart: unless-stopped
environment:
- WEBUI_AUTH=False
Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The WEBUI_AUTH environment variable is set to False, which disables authentication for the Ollama web UI. Since this service is exposed publicly via Traefik at https://ollama.viewpoint.house with no authentication middleware configured, this creates a security risk by allowing unrestricted access to the Ollama interface. Consider either enabling authentication (WEBUI_AUTH=True) or adding Traefik basicAuth middleware to protect this endpoint, especially since it's accessible from the internet.

Suggested change
- WEBUI_AUTH=False
- WEBUI_AUTH=True

Copilot uses AI. Check for mistakes.
- WEBUI_AUTH=False
- WEBUI_URL=https://ollama.viewpoint.house
- OLLAMA_BASE_URLS=http://ollama:11434
# - OLLAMA_BASE_URLS=http://host.docker.internal:7869 #comma separated ollama hosts
Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conflicting OLLAMA_BASE_URLS configuration. Line 93 sets the active value to http://ollama:11434, but line 94 comments out an alternative value http://host.docker.internal:7869. This commented line appears to be leftover configuration that should be removed to avoid confusion, as the comment suggests it's an alternative option but provides no guidance on when to use one versus the other.

Suggested change
# - OLLAMA_BASE_URLS=http://host.docker.internal:7869 #comma separated ollama hosts

Copilot uses AI. Check for mistakes.
@kylegordon kylegordon merged commit a67e390 into master Feb 24, 2026
5 checks passed
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

Successfully merging this pull request may close these issues.

1 participant