Doc review: connect-to-postgres, create-release-artifact-workflow, deploy-k8s-service #191

Merged

eblume merged 6 commits from docs/doc-review-feb-15-2026 into main 2026-02-15 07:42:01 -08:00

Conversation 0 Commits 6 Files changed 12 +43 -25

eblume commented 2026-02-15 07:38:09 -08:00

Owner

Copy link

Summary

Review session covering 3 docs, plus a codebase-wide cleanup:

Docs reviewed

• connect-to-postgres — verified end-to-end (psql connection tested), stamped
• create-release-artifact-workflow — clarified that build-blumeops.yaml is only a version bump example (not a packages API example)
• deploy-k8s-service — fixed stale repoURL (indri:2200 → forge.ops.eblu.me:2222), wrong Caddy config keys (upstream → backend, added missing host), updated Homepage group to "Services", added Tailscale tag documentation

Codebase cleanup

• Migrated all remaining op item get --fields calls to op read URI syntax across 7 files (docs, READMEs, YAML comments)
• Simplified the op read vs op item get guidance in CLAUDE.md

Side findings (not addressed)

• New immich-pg CNPG cluster not yet documented in the postgresql reference card

Test plan

• psql connection to pg.ops.eblu.me verified
• All pre-commit hooks pass
• docs-check-links, docs-check-index, docs-check-frontmatter pass

## Summary Review session covering 3 docs, plus a codebase-wide cleanup: ### Docs reviewed - **connect-to-postgres** — verified end-to-end (psql connection tested), stamped - **create-release-artifact-workflow** — clarified that `build-blumeops.yaml` is only a version bump example (not a packages API example) - **deploy-k8s-service** — fixed stale repoURL (`indri:2200` → `forge.ops.eblu.me:2222`), wrong Caddy config keys (`upstream` → `backend`, added missing `host`), updated Homepage group to "Services", added Tailscale tag documentation ### Codebase cleanup - Migrated all remaining `op item get --fields` calls to `op read` URI syntax across 7 files (docs, READMEs, YAML comments) - Simplified the `op read` vs `op item get` guidance in CLAUDE.md ## Side findings (not addressed) - New `immich-pg` CNPG cluster not yet documented in the postgresql reference card ## Test plan - [x] `psql` connection to `pg.ops.eblu.me` verified - [x] All pre-commit hooks pass - [x] `docs-check-links`, `docs-check-index`, `docs-check-frontmatter` pass

eblume added 5 commits 2026-02-15 07:38:09 -08:00

Review connect-to-postgres doc: verified and stamped ... 5d7515fdc9

Connection tested end-to-end via psql. All info accurate.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Review create-release-artifact-workflow doc: clarify and stamp ... c7bcbecdf2

Clarify that build-blumeops.yaml is only an example of the version
bump pattern, not the full generic packages upload flow.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Review deploy-k8s-service doc: fix repoURL, Caddy keys, add tag docs ... aa0b7d3a6c

Fixes:
- repoURL: indri.tail8d86e.ts.net:2200 → forge.ops.eblu.me:2222
- Caddy config: upstream → backend, add missing host field
- Homepage group: "Apps" → "Services" (matches real ingresses)

Adds documentation for tailscale.com/tags annotation:
- ProxyGroup default tag:k8s is inherited, no per-ingress annotation needed
- tag:flyio-target must be added explicitly for public access
- Setting tags overrides ProxyGroup defaults, so both must be specified

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Replace remaining op item get calls with op read ... 6ce6a1329f

Migrate all remaining `op item get --fields` usage to `op read` URI
syntax across docs, READMEs, and YAML comments. Simplify the guidance
note in CLAUDE.md now that the migration is complete.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Add changelog fragment for doc review session ... fbc82c3ea3

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

eblume added 1 commit 2026-02-15 07:40:12 -08:00

Add immich-pg cluster to postgresql reference card ... 2e3e243f90

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

eblume merged commit 22f418d0dc into main 2026-02-15 07:42:01 -08:00

eblume referenced this pull request from a commit 2026-02-15 07:42:03 -08:00

Doc review: connect-to-postgres, create-release-artifact-workflow, deploy-k8s-service (#191)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

eblume/blumeops!191

No description provided.