[ytdlp] Remove ollama_host variable and fallback env vars

## Context
Stage 3 of ollama decommission. `ytdlp` had an Ollama fallback path for when
OpenRouter models failed. With ollama going away, that fallback is
inoperable — removing the variable and two env entries prevents pods from
ever attempting to hit a service that no longer exists.

## This change
- Drops `variable "ollama_host"` from stacks/ytdlp/main.tf.
- Drops the two env entries `OLLAMA_URL` and `OLLAMA_MODEL` (plus their
  preceding comment) from the yt-highlights container.
- Apply: `0 added, 4 changed, 0 destroyed` — deployments rolled out fresh
  env, plus benign Kyverno ndots drift (already accepted).
- Verified `kubectl get deploy -n ytdlp` no longer exposes OLLAMA_URL.

## What is NOT in this change
- OpenRouter primary path unchanged.
- config.tfvars `ollama_host` still present (stage 7 removes it).

## Test plan
### Automated
- `scripts/tg plan` → 4 in-place updates, 0 destroy.
- `scripts/tg apply` → "Apply complete! Resources: 0 added, 4 changed, 0 destroyed."

### Manual Verification
1. `kubectl get deploy -n ytdlp -o yaml | grep OLLAMA` → empty.
2. yt-highlights continues processing via OpenRouter (check container logs for
   successful OpenRouter responses).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Viktor Barzin 2026-04-18 11:13:42 +00:00
parent cc44bccfaa
commit c175cfd69b

View file

@ -33,7 +33,6 @@ resource "kubernetes_manifest" "external_secret" {
}
variable "redis_host" { type = string }
variable "ollama_host" { type = string }
resource "kubernetes_namespace" "ytdlp" {
@ -285,15 +284,6 @@ resource "kubernetes_deployment" "yt_highlights" {
name = "TORCH_HOME"
value = "/data/cache/torch"
}
# Ollama fallback for when OpenRouter models fail
env {
name = "OLLAMA_URL"
value = "http://${var.ollama_host}:11434"
}
env {
name = "OLLAMA_MODEL"
value = "qwen2.5:14b"
}
volume_mount {
name = "data"
mount_path = "/data"