fix(ci): image publish path #11

Merged

rcsheets merged 2 commits from fix/harbor-login-openbao into main 2026-04-18 04:57:52 +00:00

Conversation 1 Commits 2 Files changed 3 +4 -4

rcsheets commented 2026-04-18 04:51:18 +00:00

Owner

Copy link

No description provided.

rcsheets added 2 commits 2026-04-18 04:51:18 +00:00

Fetch Harbor push creds from OpenBao, not Forgejo secrets ... 4e88fd18d5

The workflow referenced secrets.HARBOR_USERNAME/HARBOR_PASSWORD which
aren't configured on this repo. Switch to the same Kubernetes-auth +
OpenBao pattern used by pr-reviewer and front-door: the runner pod's
ServiceAccount token logs into OpenBao via the shared
harbor-push-brooktrails role and reads creds from
secret/data/forgejo-runners/brooktrails/harbor-push.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Publish images under the brooktrails/ Harbor project ... 2fc130e653

Match the convention used by pr-reviewer and front-door
(harbor.brooktrails.org/brooktrails/<app>). Image paths become
brooktrails/directory-pacifier/{controlloop,rpcserver}; placeholder
image refs in the k8s manifests are updated to match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

pr-reviewer-bot commented 2026-04-18 04:51:19 +00:00

Collaborator

Copy link

Automated review by pr-reviewer v0.17.0 | Safety Check | Claude Sonnet 4 | tracking id r-e30dc7-bb5d98
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

---

✅ Verdict: LGTM — This is a straightforward CI/CD fix to correct image publish paths by adding the organization prefix.
Complexity: trivial

Safety Check

This PR fixes the Docker image publish path by consistently adding the brooktrails/ organization prefix across CI configuration and Kubernetes manifests. The changes are straightforward path corrections with no security implications or breaking changes - all placeholder/comment updates are consistent with the actual configuration change.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.17.0 | Safety Check | Claude Sonnet 4 | tracking id `r-e30dc7-bb5d98`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- **✅ Verdict: LGTM** — This is a straightforward CI/CD fix to correct image publish paths by adding the organization prefix. **Complexity:** trivial ## Safety Check This PR fixes the Docker image publish path by consistently adding the `brooktrails/` organization prefix across CI configuration and Kubernetes manifests. The changes are straightforward path corrections with no security implications or breaking changes - all placeholder/comment updates are consistent with the actual configuration change.

rcsheets force-pushed fix/harbor-login-openbao from 2fc130e653 to d212a38ad2 2026-04-18 04:51:25 +00:00 Compare

rcsheets merged commit 3cecad9662 into main 2026-04-18 04:57:52 +00:00

rcsheets deleted branch fix/harbor-login-openbao 2026-04-18 04:57:52 +00:00

rcsheets referenced this pull request from a commit 2026-04-18 04:57:52 +00:00

Merge pull request 'fix(ci): image publish path' (#11) from fix/harbor-login-openbao into main

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

2 participants

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

brooktrails/directory-pacifier!11

No description provided.