Skip to content

oci 이전#131

Merged
mjy926 merged 1 commit into
developfrom
feat/oci
Mar 17, 2026
Merged

oci 이전#131
mjy926 merged 1 commit into
developfrom
feat/oci

Conversation

@mjy926

@mjy926 mjy926 commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@mjy926 mjy926 requested a review from a team as a code owner March 17, 2026 12:46
@coderabbitai

coderabbitai Bot commented Mar 17, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

AWS 기반 인프라에서 OCI(Oracle Cloud Infrastructure) 기반으로 마이그레이션하는 변경사항입니다. GitHub Actions 워크플로우를 ECR에서 OCIR로 전환하고, 비밀 관리를 AWSSecretManager에서 OCISecretManager로 교체했으며, 여러 설정 파일에서 OCI 통합을 추가했습니다.

Changes

Cohort / File(s) Summary
GitHub Actions 워크플로우
.github/workflows/deploy-dev.yaml, .github/workflows/deploy-prod.yaml
AWS ECR 기반 배포 흐름을 OCIR로 변경. Checkout 액션을 v3에서 v4로 업데이트, AWS 자격증명 로그인을 Docker/OCIR 로그인으로 교체, ECR 관련 환경변수를 OCIR 환경변수로 변경, 빌드 플랫폼을 linux/arm64로 조정.
핵심 비밀 관리 구현
wacruit/src/secrets.py
AWSSecretManager를 OCISecretManager로 완전히 교체. OCI SecretsClient 초기화, 인스턴스 기반 인증 및 설정 파일 폴백, Base64 디코딩된 시크릿 페이로드 처리, get_secret_bundle을 통한 OCI 비밀 조회로 변경.
비밀 관리자 소비처
wacruit/src/admin/config.py, wacruit/src/apps/common/security.py, wacruit/src/apps/hodu/config.py, wacruit/src/apps/judge/config.py, wacruit/src/apps/portfolio/file/aws/config.py, wacruit/src/database/config.py
각 설정 파일에서 AWSSecretManager를 OCISecretManager로 교체. 함수 시그니처 및 제어 흐름은 변경 없음, 시크릿 소스만 변경.
핵심 설정 업데이트
wacruit/src/settings.py
새로운 읽기 전용 프로퍼티 oci_secret_ocid 추가. 개발/프로덕션 환경에 따라 OCID 문자열 반환.
의존성
pyproject.toml
OCI 라이브러리 의존성 추가: "oci>=2.168.1".

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive 설명이 제공되지 않았습니다. 매우 관대한 기준이므로 설명 부재는 불완전하지만 변경 사항과 완전히 무관하지는 않습니다. AWS에서 OCI로 마이그레이션하는 변경 사항의 주요 목표, 영향 범위, 검증 방법을 설명하는 PR 설명을 추가하시기 바랍니다.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 변경 내용의 핵심을 명확하게 요약하고 있습니다. AWS에서 OCI로의 전환이라는 주요 변경 사항을 간결하게 표현하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/oci
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Tip

CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.

Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present.

@mjy926 mjy926 merged commit a3934e3 into develop Mar 17, 2026
2 of 3 checks passed
@mjy926 mjy926 deleted the feat/oci branch April 9, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant