Close install-review gaps: vision model auto-pull, GHCR visibility, manual steps - #2
Merged
Merged
Conversation
…anual steps - Add qwen2.5vl:3b to apps/ollama/application.yaml's auto-pull list so the vision model ollama-chat routes image requests to (src/lib/conversations.js VISION_MODEL) is fetched on sync like the text models already are. - Document GHCR package visibility for the three git-source Applications (ollama-chat, homelab-gateway, piper): confirmed all three currently public via anonymous tags/list pulls, none have an imagePullSecret, and a fresh package defaults to private regardless of the source repo's visibility. Add ADR-0018 recording that as a deliberate decision (mirrors the example already called out in ollama-chat's own ADR-0006, but that was never carried over here or to homelab-gateway). - Add a "Manual post-install steps" checklist consolidating the MetalLB DHCP reservation, Grafana admin Secret, kube-prometheus-stack CRDs, GPU time-slicing ClusterPolicy patch, and GHCR visibility check with exact commands in one place. - Fill in README's Structure section for apps/ollama-chat, apps/whisper, apps/piper, apps/homelab-gateway, and apps/gpu-time-slicing, which were missing even though docs/architecture.md already documented them — verified piper's Application stays consistent with piper-tts-server now that it's documented as its own separate repo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
vision model ollama-chat routes image requests to (src/lib/conversations.js
VISION_MODEL) is fetched on sync like the text models already are.
(ollama-chat, homelab-gateway, piper): confirmed all three currently
public via anonymous tags/list pulls, none have an imagePullSecret, and a
fresh package defaults to private regardless of the source repo's
visibility. Add ADR-0018 recording that as a deliberate decision (mirrors
the example already called out in ollama-chat's own ADR-0006, but that
was never carried over here or to homelab-gateway).
DHCP reservation, Grafana admin Secret, kube-prometheus-stack CRDs, GPU
time-slicing ClusterPolicy patch, and GHCR visibility check with exact
commands in one place.
apps/piper, apps/homelab-gateway, and apps/gpu-time-slicing, which were
missing even though docs/architecture.md already documented them —
verified piper's Application stays consistent with piper-tts-server now
that it's documented as its own separate repo.