Skip to content

fix(infra): restore production admin api ingress - #3721

Merged
tasoo-oos merged 1 commit into
mainfrom
hotfix-restore-admin-api-ingress
Aug 30, 2026
Merged

fix(infra): restore production admin api ingress#3721
tasoo-oos merged 1 commit into
mainfrom
hotfix-restore-admin-api-ingress

Conversation

@tasoo-oos

@tasoo-oos tasoo-oos commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Description

Gateway 전환 중 admin API 외부 경로가 중단되지 않도록 production overlay에 Ingress를 복구합니다.

Additional context

HTTPRoute는 유지하며 production과 stage overlay 렌더링을 확인했습니다.


Before submitting the PR, please make sure you do the following

Summary by CodeRabbit

  • New Features
    • Added secure HTTPS routing for the production admin API.
    • Enabled access through the /graphql and /admin-api paths.
    • Added automatic TLS certificate management for codedang.com.

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 59ed6342-f9c8-4a9e-b649-97de0a91a5c9

📥 Commits

Reviewing files that changed from the base of the PR and between d174f5d and 0c9c368.

📒 Files selected for processing (2)
  • infra/k8s/admin-api/overlays/production/ingress.yaml
  • infra/k8s/admin-api/overlays/production/kustomization.yaml

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The production admin API overlay adds a Traefik Ingress with TLS, HTTPS redirection, and routing for /graphql and /admin-api to admin-api-server on port 3000.

Changes

Admin API ingress

Layer / File(s) Summary
Production ingress routing
infra/k8s/admin-api/overlays/production/ingress.yaml, infra/k8s/admin-api/overlays/production/kustomization.yaml
The production overlay defines a Traefik Ingress for codedang.com. It configures TLS, HTTPS redirection, and routes /graphql and /admin-api to admin-api-server:3000. The Kustomization includes the Ingress resource.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 0c9c3

This change restores the production admin API ingress while preserving the existing gateway and TLS routing behavior; no actionable merge-blocking risk remains after normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant Client
  participant Traefik
  participant admin-api-server
  Client->>Traefik: HTTPS request to codedang.com
  Traefik->>Traefik: Apply TLS and HTTPS routing
  Traefik->>admin-api-server: Forward /graphql or /admin-api to port 3000
Loading
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: restoring the production admin API Ingress.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix-restore-admin-api-ingress

Comment @coderabbitai help to get the list of available commands.

@tasoo-oos tasoo-oos self-assigned this Aug 30, 2026
@tasoo-oos
tasoo-oos marked this pull request as ready for review August 30, 2026 13:51
@tasoo-oos
tasoo-oos merged commit 86dca24 into main Aug 30, 2026
33 checks passed
@tasoo-oos
tasoo-oos deleted the hotfix-restore-admin-api-ingress branch August 30, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant