Skip to content

release: develop → main (#686 유사퀴즈 제출 Clock 정합 · #687 오토스케일링 검증 대시보드) - #689

Merged
dlxodus02 merged 5 commits into
mainfrom
develop
Jul 24, 2026
Merged

release: develop → main (#686 유사퀴즈 제출 Clock 정합 · #687 오토스케일링 검증 대시보드)#689
dlxodus02 merged 5 commits into
mainfrom
develop

Conversation

@dlxodus02

Copy link
Copy Markdown
Contributor

📌 연관 이슈/PR


📝 작업 내용 (이번 릴리즈 포함분)


🖥️ 프론트엔드 연동 가이드 (API 명세)

⚠️ API 계약 변경 없음. #686은 서버 내부 시각 계산만 고친 동작 버그 픽스, #687은 모니터링 대시보드로 API 무관.

  • 참고: KST 새벽 제출 시 복습 완료가 반영되지 않던 문제가 이 릴리즈로 프로덕션에 반영됩니다.

🚀 배포 주의사항 (Release Notes)

  • DB 마이그레이션: 신규 없음 (main·develop db/migration 트리 동일 → Flyway 드리프트 없음).
  • main 전용 핫픽스 보존 확인: main의 fix(order): 결제 확정 후 지급 실패 시 재호출에서 재시도 #675(결제 확정 후 지급 실패 시 멱등 재시도, ConfirmOrderPaymentService)는 develop이 해당 파일을 건드리지 않아 이 머지로 되돌아가지 않음(검증 완료).
  • ⚠️ 후속 권장: 릴리즈 후 main → develop 백머지 필요. develop 히스토리엔 #675가 없어(구버전 결제 서비스), 하지 않으면 이후 develop 작업이 fix(order): 결제 확정 후 지급 실패 시 재호출에서 재시도 #675 이전 상태에서 출발함.
  • 환경변수/시크릿 변경 없음. 서버 타임존과 무관하게 동작하도록 Clock 주입으로 고친 것이라 별도 인프라 조치 불필요.

✅ 체크리스트

dlxodus02 and others added 5 commits July 24, 2026 15:59
복습 완료 전진의 '오늘'이 스케줄 조회(ScheduleController)의 KST '오늘'과
어긋나던 문제 수정. LocalDateTime.now() 는 JVM 기본 타임존(컨테이너 UTC)을
써서, KST 00:00~08:59 구간에 제출하면 today(UTC)=어제(KST)가 되어
오늘 due 복습 카드가 전진되지 않고(완료 무반영) due 날짜도 하루 밀렸다.
팀 표준 Clock(ClockConfig, Asia/Seoul)을 주입해 now(clock)로 계산한다.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
instance_name 라벨 기준으로 인스턴스별 RPS와 활성 인스턴스 수를
확인할 수 있는 패널 2개를 공통 대시보드에 추가한다.
고정 Clock을 KST 10:00 → UTC 23:30(KST 익일 08:30)으로 옮겨 버그가
재현되던 00:00~08:59 경계를 실제로 통과시키고, submittedAt·복습 전진 인자가
LocalDateTime.of(2026,5,12,8,30)인지 절대값으로 단언한다. 두 값의 동일성만
검사하던 기존 방식은 구현이 raw now()로 되돌아가도 통과해 회귀를 못 잡았다.
(CodeRabbit 리뷰 반영)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
fix(quiz): 유사퀴즈 제출 시각을 주입 Clock(Asia/Seoul)으로 계산
feat: 오토스케일링/로드밸런싱 검증용 Grafana 대시보드 패널 추가 (#687)
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 7db7c535-9282-41f1-8546-f1e7dbc36a1a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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.

@github-actions

Copy link
Copy Markdown

Gate 2 · 결정론 판정

88 tests   82 ✅  0s ⏱️
29 suites   6 💤
29 files     0 ❌

Results for commit d6fb130.

@github-actions

Copy link
Copy Markdown

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit d6fb130.

@github-actions

Copy link
Copy Markdown

Gate 1 · 결정론 규칙

16 tests   16 ✅  1s ⏱️
 3 suites   0 💤
 3 files     0 ❌

Results for commit d6fb130.

@dlxodus02
dlxodus02 merged commit 3aa22d0 into main Jul 24, 2026
10 checks passed
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.

2 participants