Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ spec:
main:
image:
repository: ghcr.io/immich-app/immich-machine-learning
tag: v2.7.5@sha256:a2501141440f10516d329fdfba2c68082e19eb9ba6016c061ac80d23beadf7f3
tag: v3.1.0@sha256:5a0839dc5303cd7215bcd2180a26aed3af41675aefb3e75e5157e9f10ad16e6e
resources:
requests:
cpu: 15m
Expand Down
5 changes: 0 additions & 5 deletions kubernetes/apps/default/immich/server/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,7 @@ data:
NODE_ENV: production
REDIS_HOSTNAME: dragonfly.database.svc.cluster.local
REDIS_PORT: "6379"
IMMICH_WEB_URL: http://immich-web.default.svc.cluster.local:3000
IMMICH_SERVER_URL: http://immich-server.default.svc.cluster.local:3001
IMMICH_MACHINE_LEARNING_URL: http://immich-machine-learning.default.svc.cluster.local:3003
TYPESENSE_HOST: immich-typesense.default.svc.cluster.local
TYPESENSE_PORT: "8108"
TYPESENSE_PROTOCOL: http
TYPESENSE_DATA_DIR: /config
DB_VECTOR_EXTENSION: pgvector
IMMICH_IGNORE_MOUNT_CHECK_ERRORS: "true"
2 changes: 1 addition & 1 deletion kubernetes/apps/default/immich/server/helmrelease.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ spec:
main:
image:
repository: ghcr.io/immich-app/immich-server
tag: v2.7.5@sha256:c15bff75068effb03f4355997d03dc7e0fc58720c2b54ad6f7f10d1bc57efaa5
tag: v3.1.0@sha256:b434cb9287eea1471c9974845914d4dd328c9c2d652e446ed4930f99944f0ceb

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Verify Immich v3 migration prerequisites before rollout.

This change upgrades Immich from v2.7.5 to v3.1.0. The PR objective lists breaking changes, including removal of deprecated environment variables and pgvecto.rs support. This manifest only changes the image reference.

Confirm that database migration, PostgreSQL extension compatibility, backup, rollback, postgres-init:v18, and referenced configuration are handled by other Flux manifests before deployment. If they are not, block the rollout until the migration plan is committed.

#!/usr/bin/env bash
set -euo pipefail

rg -n -C 5 \
  'pgvecto|pgvector|postgres-init|DB_URL|envFrom|secretRef|configMapRef|migration|backup|rollback' \
  kubernetes/apps/default/immich
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@kubernetes/apps/default/immich/server/helmrelease.yaml` at line 37, Before
changing the image tag in the Immich HelmRelease, verify that the surrounding
Flux manifests define the required v3 migration prerequisites: database
migration handling, PostgreSQL extension compatibility, backup and rollback
procedures, postgres-init:v18, and updated referenced configuration with
deprecated variables removed. If any prerequisite is missing, block the image
rollout and commit the necessary manifest changes first.

resources:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🛑 Blocker (bug): The init-db container uses postgres-init:v18; verify compatibility with Immich v3.0.2 database schema changes.

Automated finding from AI PR review.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Still open after this push; carried forward. (as of f9b6ac8)

requests:
cpu: 100m

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Info (question): Resource requests and limits are unchanged; verify they remain appropriate for Immich v3.0.3 workload characteristics.

Automated finding from AI PR review.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Still open after this push; carried forward. (as of aeed5cc)

Expand Down
8 changes: 8 additions & 0 deletions kubernetes/apps/security/crowdsec/app/helmrelease.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,14 @@ spec:
reason: "envoy bouncer response to already banned ips"
expression:
- "evt.Meta.http_status == '418'"
jellyfin-crawl-whitelist.yaml: |-
name: hydaz/jellyfin-crawl-whitelist
description: "Jellyfin clients fan out to dozens of unique /HomeScreen/CachedImage URIs per page load, which reads as http-crawl-non_statics"
filter: "evt.Meta.service == 'http' && evt.Meta.log_type == 'http_access-log' && evt.Meta.target_fqdn == 'jellyfin.${SECRET_DOMAIN}'"
whitelist:
reason: "successful jellyfin api traffic"
expression:
- "evt.Meta.http_status startsWith '2' or evt.Meta.http_status startsWith '3'"
config.yaml.local: |-
api:
server:
Expand Down
Loading