Skip to content

[feat] Firebase Admin Sdk 볼륨 마운트 형태로 주입#1339

Merged
lepitaaar merged 2 commits intodevelop/befrom
feature/#1338-file-mount-firebase-admin-sdk-MOA-750
Mar 18, 2026
Merged

[feat] Firebase Admin Sdk 볼륨 마운트 형태로 주입#1339
lepitaaar merged 2 commits intodevelop/befrom
feature/#1338-file-mount-firebase-admin-sdk-MOA-750

Conversation

@lepitaaar
Copy link
Copy Markdown
Contributor

@lepitaaar lepitaaar commented Mar 18, 2026

🚀 릴리즈 PR인 경우 릴리즈 템플릿으로 전환해 주세요. (Preview 탭에서 클릭)

#️⃣연관된 이슈

📝작업 내용

현재 도커이미지에 민감한 Admin SDK 설정파일이 주입되어있었음.
그부분을 K8s 배포시 볼륨 마운트를 시켜 기록이 남지 않게 변경함.

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

  • Chores
    • Firebase 설정 단계가 릴리스 워크플로우에서 제거되었습니다.
    • Firebase 구성 파일 경로를 환경 설정으로 관리할 수 있도록 개선되었습니다. 누락된 설정 파일에 대한 오류 로깅이 추가되었습니다.

@lepitaaar lepitaaar self-assigned this Mar 18, 2026
@lepitaaar lepitaaar added ✨ Feature 기능 개발 💾 BE Backend labels Mar 18, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
moadong Ready Ready Preview, Comment Mar 18, 2026 6:23am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

CI/CD 워크플로우에서 firebase.json 생성 단계를 제거하고, FcmInitializer의 Firebase 설정을 하드코딩된 ClassPathResource에서 @Value를 통한 의존성 주입 방식으로 변경하여 리소스 경로를 설정 가능하게 수정했습니다.

Changes

Cohort / File(s) Summary
CI/CD 워크플로우
.github/workflows/server_cd_release.yml
릴리즈 워크플로우의 application.properties 생성 직후 firebase.json 생성 단계를 제거했습니다.
Firebase 초기화 설정
backend/src/main/java/moadong/global/util/FcmInitializer.java
하드코딩된 ClassPathResource를 @Value("${firebase.config.path:classpath:firebase.json}")로 주입받는 방식으로 변경하여 Firebase 설정 파일 경로를 외부에서 설정 가능하게 개선했습니다. 리소스 존재 여부 검증 및 에러 로깅 로직을 추가했습니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • [cicd] firebase.json 추가 #814: CI/CD 워크플로우에서 firebase.json 생성 단계를 제거한 이번 PR와 firebase.json 생성을 추가한 해당 PR이 Firebase 설정 관리 방식을 대조적으로 수정합니다.

Suggested labels

📦 CI/CD

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

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.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 Firebase Admin SDK를 볼륨 마운트 형태로 주입하는 변경사항을 정확히 설명하며, 제공된 raw_summary와 pr_objectives의 주요 목표(Docker 이미지에 민감한 설정 파일 직접 포함 제거, Kubernetes 배포 시 볼륨 마운트로 주입)와 일치합니다.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/#1338-file-mount-firebase-admin-sdk-MOA-750
📝 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 generate a title for your PR based on the changes with custom instructions.

Set the reviews.auto_title_instructions setting to generate a title for your PR based on the changes in the PR with custom instructions.

@github-actions
Copy link
Copy Markdown

Test Results

100 tests   100 ✅  20s ⏱️
 25 suites    0 💤
 25 files      0 ❌

Results for commit 5b1c9b9.

@lepitaaar lepitaaar merged commit 180b6e8 into develop/be Mar 18, 2026
4 of 5 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💾 BE Backend ✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant