finn/homelab
Watch
2
0
Fork
You've already forked homelab
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Enable public path-based PR previews #15

Merged
finn merged 1 commit from public-preview-routing into main 2026-10-26 23:53:52 +00:00
Conversation 0 Commits 1 Files changed 36 +1976 -20
finn commented 2026-10-26 23:51:35 +00:00
Owner

Summary

  • Reconcile the deployed Argo CD and Orchard Core preview infrastructure with Terraform source, including the shared preview Tunnel/DNS route and prefix middleware.
  • Route previews at https://preview.113lab.dev/// and request a cert-manager TLS certificate for the shared host.
  • Keep the legacy per-preview hostname mode available and document the public, no-Access exposure.
  • Include the related deployed application infrastructure changes so a future Terraform apply does not remove live resources.

Validation

  • Node tests: 16/16 pass.
  • Terraform formatting and validation pass.
  • Helm lint and path-mode rendering pass.
  • Full Terraform plan: no creates or destroys; two unrelated in-place updates remain (AppProject content refresh and RomM restartedAt annotation cleanup).

Live status

  • Shared Cloudflare DNS/Tunnel route, CoreDNS rewrite, and Traefik prefix middleware are already applied.
  • The public shared hostname currently returns 502 because the active OrchardCore preview has not yet been redeployed with the TLS-enabled path-mode chart. After this PR is merged, redeploy the preview to trigger cert-manager issuance and retest.
## Summary - Reconcile the deployed Argo CD and Orchard Core preview infrastructure with Terraform source, including the shared preview Tunnel/DNS route and prefix middleware. - Route previews at https://preview.113lab.dev/<repo-key>/<pr-number>/ and request a cert-manager TLS certificate for the shared host. - Keep the legacy per-preview hostname mode available and document the public, no-Access exposure. - Include the related deployed application infrastructure changes so a future Terraform apply does not remove live resources. ## Validation - Node tests: 16/16 pass. - Terraform formatting and validation pass. - Helm lint and path-mode rendering pass. - Full Terraform plan: no creates or destroys; two unrelated in-place updates remain (AppProject content refresh and RomM restartedAt annotation cleanup). ## Live status - Shared Cloudflare DNS/Tunnel route, CoreDNS rewrite, and Traefik prefix middleware are already applied. - The public shared hostname currently returns 502 because the active OrchardCore preview has not yet been redeployed with the TLS-enabled path-mode chart. After this PR is merged, redeploy the preview to trigger cert-manager issuance and retest.
Add public path-based PR previews
All checks were successful
terraform / check (pull_request) Successful in 19s
Details
c32100dad4 
Co-authored-by: Copilot <[email protected]>
finn merged commit 173057ab31 into main 2026-10-26 23:53:52 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
preview-bootstrap
Provision repository-scoped PR preview access

No labels
preview-bootstrap
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
finn/homelab!15
No description provided.