forgejo_admin/westside-landing
1
0
Fork 1
Code Issues 16 Pull requests 3 Projects Releases Packages Wiki Activity Actions

fix: align CI push target to westsidekingsandqueens Harbor project #41

Closed
forgejo_admin wants to merge 1 commit from 110-fix-harbor-push-target into main
pull from: 110-fix-harbor-push-target
merge into: forgejo_admin:main
Conversation 2 Commits 1 Files changed 1 +1 -1
forgejo_admin commented 2026-03-18 19:37:36 +00:00
Owner
Copy link

Summary

Convention: service key = Harbor project name. CI was pushing to westside-app/app but the service key is westsidekingsandqueens, so the correct target is westsidekingsandqueens/app.

Changes

  • .woodpecker.yaml: changed repo: westside-app/app to repo: westsidekingsandqueens/app in the build-and-push step

Test Plan

  • Merge and trigger a CI pipeline on main -- verify Kaniko pushes to harbor.tail5b443a.ts.net/westsidekingsandqueens/app
  • Verify ArgoCD picks up the new image from the correct Harbor project
  • No regressions in CI pipeline

Review Checklist

  • Passed automated review-fix loop
  • No secrets committed
  • No unnecessary file changes
  • Commit messages are descriptive

Related

## Summary Convention: service key = Harbor project name. CI was pushing to `westside-app/app` but the service key is `westsidekingsandqueens`, so the correct target is `westsidekingsandqueens/app`. ## Changes - `.woodpecker.yaml`: changed `repo: westside-app/app` to `repo: westsidekingsandqueens/app` in the build-and-push step ## Test Plan - [ ] Merge and trigger a CI pipeline on main -- verify Kaniko pushes to `harbor.tail5b443a.ts.net/westsidekingsandqueens/app` - [ ] Verify ArgoCD picks up the new image from the correct Harbor project - [ ] No regressions in CI pipeline ## Review Checklist - [x] Passed automated review-fix loop - [x] No secrets committed - [x] No unnecessary file changes - [x] Commit messages are descriptive ## Related - Closes forgejo_admin/pal-e-platform#110 - `plan-pal-e-platform`
Convention: service key = Harbor project name. The service key is
westsidekingsandqueens, so CI must push to
harbor.tail5b443a.ts.net/westsidekingsandqueens/app instead of
harbor.tail5b443a.ts.net/westside-app/app.

Related: forgejo_admin/pal-e-platform#110

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
forgejo_admin commented 2026-03-18 19:37:52 +00:00
Author
Owner
Copy link

Review

Status: LGTM

Single-line change, exactly scoped to the issue. Diff replaces westside-app/app with westsidekingsandqueens/app in the Kaniko push step. No other files touched, no secrets, no regressions possible beyond the target change itself.

Verified:

## Review **Status: LGTM** Single-line change, exactly scoped to the issue. Diff replaces `westside-app/app` with `westsidekingsandqueens/app` in the Kaniko push step. No other files touched, no secrets, no regressions possible beyond the target change itself. Verified: - [x] Diff matches issue spec - [x] No unrelated changes - [x] No secrets committed - [x] PR body follows template - [x] Cross-repo issue link present (forgejo_admin/pal-e-platform#110)
forgejo_admin closed this pull request 2026-03-27 04:56:03 +00:00
forgejo_admin commented 2026-03-27 04:56:05 +00:00
Author
Owner
Copy link

Closing — 33 commits behind main, too much scope (removes clone step, deletes Capacitor, switches port). Will create a surgical replacement that only fixes the repo: line in .woodpecker.yaml.

Closing — 33 commits behind main, too much scope (removes clone step, deletes Capacitor, switches port). Will create a surgical replacement that only fixes the `repo:` line in `.woodpecker.yaml`.

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend
No labels
domain:backend
domain:devops
domain:frontend
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
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
forgejo_admin/westside-landing!41
No description provided.