-
Notifications
You must be signed in to change notification settings - Fork 8
chore: 지원 학기 변경 #487
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: 지원 학기 변경 #487
Conversation
Walkthrough
Estimated code review effort🎯 1 (사소) | ⏱️ ~2 minutes Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/main/resources/secret (1)
1-1: 런타임에서 해당 yml이 우선순위로 로딩되는지 한 번만 점검해 주세요.
1) 외부 ConfigMap/환경변수/프로필용 application-*.yml이 있으면 우선순위에 의해 값이 덮일 수 있습니다.
2) 빌드 시 src/main/resources/secret 이하 파일이 classpath에 포함되는지와 프로필별 로딩 전략을 확인하면 운영 리스크를 줄일 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
src/main/resources/secret(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (2)
src/main/resources/secret (2)
1-1: 서브모듈 커밋 포인터 갱신은 목적(지원 학기 2026-1 반영)에 부합합니다.
1) src/main/resources/secret 서브모듈 포인터가 fd0d80ad28d28698e3e27160d9d27bf4e5462238 커밋으로 업데이트되었습니다.
2) 코드/퍼블릭 API에는 영향이 없어 보입니다.
3) application-variable.yml 변경을 배포 아티팩트에 반영하려는 목적과 정합합니다.
1-1: 서브모듈 초기화 및 2026-1 반영 여부 재검증 요청
스크립트 실행 중src/main/resources/secret서브모듈을 찾을 수 없어 초기화에 실패했습니다.
아래 내용을 확인해 주세요:
.gitmodules URL 검증
–src/main/resources/secret서브모듈의 원격 URL이 올바르게 설정되어 있는지 확인합니다.
– HTTPS 대신 SSH를 사용하거나, 접근 권한 문제는 없는지 점검합니다.2026-1 문자열 수동 확인
– 서브모듈 디렉토리 내application-variable.yml파일을 직접 열어 ‘2026-1’ 문자열이 포함되었는지 확인합니다.CI 재현성 확보
– 올바른 URL 또는 인증 방식을 적용해 동일 스크립트로 CI 환경에서도 서브모듈 초기화가 통과하도록 수정합니다.위 사항을 점검하신 뒤 결과를 알려주세요!
| @@ -1 +1 @@ | |||
| Subproject commit bb3bf0f4122d10ddacab279a368cf9f06d6f6dbd | |||
| Subproject commit fd0d80ad28d28698e3e27160d9d27bf4e5462238 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
CI/CD에서 서브모듈 체크아웃이 보장되는지 확인해 주세요.
1) 빌드 파이프라인이 actions/checkout의 submodules 옵션 또는 명시적 git submodule update를 수행하는지 검증이 필요합니다.
2) 보장이 없으면 jar/war에 최신 application-variable.yml이 포함되지 않을 수 있습니다.
다음 스크립트로 워크플로우에 서브모듈 초기화가 있는지 확인할 수 있습니다.
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "[1/2] GitHub Actions에서 submodules 설정/명령 검색"
rg -n -C2 -g '.github/workflows/**' -e 'submodules\s*:\s*true|git submodule update|actions/checkout@.*with:'
echo "[2/2] Jenkins/GitLab/CircleCI 설정에서도 검색"
fd -HI -a -t f -E node_modules -E build -E dist 'Jenkinsfile|jenkins|gitlab-ci|\.gitlab-ci\.yml|\.circleci|circle' \
| xargs -r rg -n -C2 -e 'submodules|git submodule update'Length of output: 374
안녕하세요! CI 파이프라인에서 서브모듈 체크아웃 보장이 제대로 작동하는지 아래 3단계 스크립트로 재확인해 주세요.
- GitHub Actions 워크플로우 파일에서
actions/checkout사용 여부 검색
-.github/workflows/**/*.yml내부의 모든actions/checkout호출을 탐지합니다. - GitHub Actions 워크플로우 파일에서
submodules옵션 검색
- 동일 워크플로우 파일 내with:블록에submodules:설정이 있는지 확인합니다. - 기타 CI 구성 파일(Jenkinsfile,
.gitlab-ci.yml, CircleCI)에서git submodule update명령 검색
- 루트 디렉터리 아래 모든 관련 CI 설정 파일에서git submodule update실행 여부를 점검합니다.
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "[1/3] GitHub Actions 워크플로우에서 actions/checkout 호출 검색"
rg -n -C2 -g '.github/workflows/**/*.yml' 'actions/checkout'
echo "[2/3] GitHub Actions 워크플로우에서 submodules 옵션 검색"
rg -n -C2 -g '.github/workflows/**/*.yml' 'submodules\s*:\s*(true|false)'
echo "[3/3] 기타 CI 구성 파일에서 git submodule update 검색"
fd -HI -a -t f -E node_modules -E build -E dist 'Jenkinsfile|\.gitlab-ci\.yml|\.circleci/config\.yml' \
| xargs -r rg -n -C2 'git submodule update'Length of output: 342
CI 설정 파일이 확인되지 않아 자동 검증이 불가능합니다. 아래 3단계로 CI 구성 자체가 존재하는지 먼저 확인해 주세요.
- .github/workflows 디렉터리의 yml 파일 확인
- 경로:.github/workflows/*.yml - 루트 디렉터리의 CI 설정 파일 확인
- 파일명:Jenkinsfile,.gitlab-ci.yml - CircleCI 구성 파일 확인
- 경로:.circleci/config.yml
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "[1/3] .github/workflows 디렉터리 파일 목록"
find .github/workflows -maxdepth 1 -type f -name '*.yml' 2>/dev/null || echo " → 없음"
echo "[2/3] 루트 디렉터리 CI 설정 파일 목록"
find . -maxdepth 1 \( -name Jenkinsfile -o -name '.gitlab-ci.yml' \) -type f 2>/dev/null || echo " → 없음"
echo "[3/3] .circleci/config.yml 존재 여부"
find .circleci -maxdepth 1 -type f -name 'config.yml' 2>/dev/null || echo " → 없음"Length of output: 533
다음으로 실제 워크플로우 파일에서 체크아웃·서브모듈 설정을 점검해 볼게요.
- actions/checkout 호출 검색
-.github/workflows/ci.yml,dev-cd.yml,prod-cd.yml내에actions/checkout라인이 있는지 확인합니다. - submodules 옵션 검색
- 동일 파일들에서submodules:설정(true/false)이 명시되어 있는지 확인합니다. - git submodule update 검색
- 혹시 수동으로 서브모듈을 업데이트하는git submodule update명령이 있는지 검사합니다.
이 결과를 보고 CI/CD에서 서브모듈 체크아웃이 제대로 보장되는지 함께 검증해 주세요.
🏁 Script executed:
#!/bin/bash
set -euo pipefail
WORKFLOWS=(.github/workflows/ci.yml .github/workflows/dev-cd.yml .github/workflows/prod-cd.yml)
echo "[1/3] actions/checkout 호출 검색"
rg -n 'actions/checkout' "${WORKFLOWS[@]}" || echo " → 호출 없음"
echo "[2/3] submodules 옵션 검색"
rg -n 'submodules\s*:\s*(true|false)' "${WORKFLOWS[@]}" || echo " → 설정 없음"
echo "[3/3] git submodule update 검색"
rg -n 'git submodule update' "${WORKFLOWS[@]}" || echo " → 명령 없음"Length of output: 995
CI 워크플로우 서브모듈 체크아웃 설정 보강 필요
안녕하세요! CI 워크플로우에서 서브모듈 체크아웃 상태를 점검했습니다.
dev-cd.yml과 prod-cd.yml에는 submodules: true가 설정되어 있지만 ci.yml에는 빠져 있습니다.
이로 인해 빌드된 jar/war에 최신 application-variable.yml이 포함되지 않을 수 있습니다.
따라서 아래와 같이 수정이 필요합니다:
- ci.yml에 submodules 옵션 추가
-uses: actions/checkout@v4아래에
yaml with: submodules: true - (선택) 수동 서브모듈 업데이트 명령 추가
- 필요 시git submodule update --init --recursive를 후속 스텝으로 추가 검토
위 설정을 반영하시면 모든 빌드 파이프라인에서 서브모듈 체크아웃이 보장됩니다.
🤖 Prompt for AI Agents
In src/main/resources/secret around lines 1 to 1, the CI workflow is missing the
submodules checkout option causing builds to potentially exclude updated
submodule content; update the ci.yml workflow to add the submodules option to
the actions/checkout@v4 step (i.e., add a with: submodules: true block under the
uses line) and optionally add a follow-up step to run git submodule update
--init --recursive if you want explicit submodule initialization in the
pipeline.
작업 내용
지원 학기를 2026-1로 변경한 application-variable.yml 가 반영되도록
서브모듈을 업데이트 합니다.