Phase 4: publish image to GHCR; retire legacy self-hosted deploy workflows#12
Conversation
Per user direction: all merges go through a PR validated by CodeRabbit; merge only when validation fully passes; resolve findings and re-review before merging.
Phase 4 of agent-cloud deployment alignment. - Add build-and-publish.yml: build + push to ghcr.io/uhstray-io/wisbot on merge to main and on v* tags (metadata-action tags: latest, short-sha, semver). Minimal permissions (contents: read, packages: write), persist-credentials: false, all actions pinned to commit SHAs. - Remove legacy self-hosted-runner workflows: deployment_prod.yml, deployment_refresh_prod.yml, shutdown.yml (replaced by GHCR publish + agent-cloud/Semaphore deploy; they also leaked the token via 'cat .env'). - Keep deploy-o11y.yml (observability) — migration to agent-cloud's o11y is a tracked follow-up. - Update README + CLAUDE.md deployment sections.
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
💤 Files with no reviewable changes (3)
📜 Recent review details🧰 Additional context used📓 Path-based instructions (1){CLAUDE.md,README.md}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🧠 Learnings (1)📓 Common learnings🪛 LanguageToolREADME.md[uncategorized] ~51-~51: The official name of this software platform is spelled with a capital “H”. (GITHUB) [uncategorized] ~51-~51: The official name of this software platform is spelled with a capital “H”. (GITHUB) 🔇 Additional comments (5)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR removes legacy self-hosted runner deployment workflows and introduces a new containerized CI/CD model. It documents a CodeRabbit PR validation policy, adds a ChangesCI/CD Migration to Docker & agent-cloud
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Phase 4 of agent-cloud deployment alignment
WisBot now publishes its image; deployment moves to the agent-cloud platform.
Changes
build-and-publish.yml(new) — builds + pushes toghcr.io/uhstray-io/wisboton merge tomainand onv*tags. Tags viametadata-action:latest, short-sha, semver. Hardened:permissions: { contents: read, packages: write },persist-credentials: false, all actions pinned to verified commit SHAs.deployment_prod.yml,deployment_refresh_prod.yml,shutdown.yml. These diddocker build+docker compose up/downon a self-hosted runner and leaked the token viacat .env. Replaced by GHCR publish + agent-cloud/Semaphore deploy.deploy-o11y.yml(Prometheus/Promtail monitoring) — migrating observability to agent-cloud's o11y service is a tracked follow-up.README.md+CLAUDE.mddeployment sections updated.Notes
GITHUB_TOKEN(no new secrets). First publish happens when this merges tomain.Depends on: Phases 1–3 (#9, #10, #11 — merged). Next: Phase 5 —
agents/wisbot/deployment dir in the agent-cloud repo.