Add westside-admin to overlay registry in woodpecker-update-tag-step.yaml #312

New issue

Closed

opened 2026-04-28 04:41:42 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-04-28 04:41:42 +00:00

Contributor

Type

Chore

Lineage

Discovered scope from Wave 2B QA review (PR #11, 2026-04-26). The shared CI script scripts/woodpecker-update-tag-step.yaml carries an overlay registry header listing services that have overlays in pal-e-deployments. westside-admin's overlay was created by pal-e-deployments PR #134 (merged) but never registered in this header. PR #11 references the script via OVERLAY=westside-admin — works at runtime regardless of the header (the header is documentation), but missing from the registry creates discoverability/maintenance gap.

Repo

forgejo_admin/pal-e-platform

Type

Chore (documentation)

User Story

story:platform-stability — As a future agent or operator reading the shared CI tag-update script's documentation header, I need westside-admin listed alongside the other registered overlays so I don't assume it's missing/unsupported.

Architecture

arch:ci-pipeline

Context

The script scripts/woodpecker-update-tag-step.yaml documents an overlay-name-to-OVERLAY-env-var mapping in its header comment. Westside-admin is currently absent from this list despite the overlay existing in pal-e-deployments and being actively wired by westside-admin#11's .woodpecker.yaml.

File Targets

scripts/woodpecker-update-tag-step.yaml — add westside-admin -> OVERLAY=westside-admin line in the header comment (between westside-app and basketball-api alphabetically, or per existing ordering)

Acceptance Criteria

Header comment lists westside-admin
Ordering matches existing pattern (alphabetical or by pal-e-deployments overlay creation order)
No functional change to the script itself

Test Expectations

N/A (documentation change)
Verification: grep westside-admin scripts/woodpecker-update-tag-step.yaml returns the new line

Constraints

Documentation only — do not modify the actual step logic or environment variables
Don't reorder existing entries

Checklist

Reviewed via /review-ticket and moved backlog → todo (skip for trivial doc edit if Ava handles directly)
Edit applied
PR opened with Closes #THIS
Merged via standard hook

Triggered by: Wave 2B QA review on westside-admin#11
Related: westside-admin#11 (the consumer that needs this overlay registered)
Memory: feedback_discovered_scope_always_tracked

### Type Chore ### Lineage Discovered scope from Wave 2B QA review (PR #11, 2026-04-26). The shared CI script `scripts/woodpecker-update-tag-step.yaml` carries an overlay registry header listing services that have overlays in pal-e-deployments. westside-admin's overlay was created by pal-e-deployments PR #134 (merged) but never registered in this header. PR #11 references the script via `OVERLAY=westside-admin` — works at runtime regardless of the header (the header is documentation), but missing from the registry creates discoverability/maintenance gap. ### Repo forgejo_admin/pal-e-platform ### Type Chore (documentation) ### User Story story:platform-stability — As a future agent or operator reading the shared CI tag-update script's documentation header, I need westside-admin listed alongside the other registered overlays so I don't assume it's missing/unsupported. ### Architecture arch:ci-pipeline ### Context The script `scripts/woodpecker-update-tag-step.yaml` documents an overlay-name-to-OVERLAY-env-var mapping in its header comment. Westside-admin is currently absent from this list despite the overlay existing in pal-e-deployments and being actively wired by westside-admin#11's `.woodpecker.yaml`. ### File Targets - `scripts/woodpecker-update-tag-step.yaml` — add `westside-admin -> OVERLAY=westside-admin` line in the header comment (between westside-app and basketball-api alphabetically, or per existing ordering) ### Acceptance Criteria - [ ] Header comment lists westside-admin - [ ] Ordering matches existing pattern (alphabetical or by pal-e-deployments overlay creation order) - [ ] No functional change to the script itself ### Test Expectations - N/A (documentation change) - Verification: `grep westside-admin scripts/woodpecker-update-tag-step.yaml` returns the new line ### Constraints - Documentation only — do not modify the actual step logic or environment variables - Don't reorder existing entries ### Checklist - [ ] Reviewed via /review-ticket and moved backlog → todo (skip for trivial doc edit if Ava handles directly) - [ ] Edit applied - [ ] PR opened with `Closes #THIS` - [ ] Merged via standard hook ### Related - Triggered by: Wave 2B QA review on westside-admin#11 - Related: westside-admin#11 (the consumer that needs this overlay registered) - Memory: `feedback_discovered_scope_always_tracked`