Conversation
* fix: 피버타임 검증 시 점수가 두배가 되지 않는 문제 해결 * refactor: 바뀐 dto에 맞게 BIZ 및 test 수정
* fix: 피버타임 검증 시 점수가 두배가 되지 않는 문제 해결 * refactor: 바뀐 dto에 맞게 BIZ 및 test 수정 * fix: 피버타임중 일시정지 시 피버타임도 멈추게 한다 * refactor: 인터페이스로 묶는다
* fix: 점수 증가 로직 오류 해결 * fix: 스트릭 발생 시각을 받는다 * refactor: 네이밍 수정 * fix(test): 스트릭 발생 시각 추가
* fix: 치팅 및 네트워크 지연 해결 * comment: 디버깅을 위한 로그 추가 * fix: session과 fevertime 일시정지 동기화
* fix: 치팅 및 네트워크 지연 해결 * comment: 디버깅을 위한 로그 추가 * fix: session과 fevertime 일시정지 동기화 * fix: 세션 예외 오류 해결 * feat: multipart 예외 핸들링 * refactor: CD 파이프라인 OCI로 마이그레이션 * fix: end response 분리
* fix: 치팅 및 네트워크 지연 해결 * comment: 디버깅을 위한 로그 추가 * fix: session과 fevertime 일시정지 동기화 * fix: 세션 예외 오류 해결 * feat: multipart 예외 핸들링 * refactor: CD 파이프라인 OCI로 마이그레이션 * fix: end response 분리 * fix: oci 경로 수정
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds shebangs to the deployment scripts and updates the PATH environment variable in deploy-rolling.sh. It also simplifies the OCI NLB backend update commands by removing several flags. Feedback suggests restoring the --is-backup false and --is-offline false flags to ensure that backends are explicitly set to an active state and can receive traffic immediately after deployment, preventing potential service disruptions if those states were previously set manually.
| --backend-set-name snackgame-http \ | ||
| --backend-name "$HTTP_BACKEND" \ | ||
| --is-drain "$is_drain" \ | ||
| --is-backup false \ | ||
| --is-offline false \ | ||
| --wait-for-state SUCCEEDED \ | ||
| --force | ||
| --wait-for-state SUCCEEDED |
There was a problem hiding this comment.
백엔드 업데이트 시 --is-backup false 및 --is-offline false 옵션이 제거되었습니다. 이 옵션들은 배포 과정에서 해당 백엔드가 백업용이 아니며 오프라인 상태도 아님을 명시적으로 보장하여, 배포 완료 후 즉시 정상적인 트래픽을 받을 수 있도록 합니다. 만약 관리자가 수동으로 백엔드를 오프라인이나 백업 상태로 설정해 두었다면, 이 옵션들이 없는 현재의 스크립트는 배포 후에도 해당 상태를 그대로 유지하게 되어 서비스 투입이 되지 않을 수 있습니다. 배포의 안정성과 자동화 측면에서 해당 옵션들을 명시적으로 유지하는 것이 좋습니다.
| --backend-set-name snackgame-http \ | |
| --backend-name "$HTTP_BACKEND" \ | |
| --is-drain "$is_drain" \ | |
| --is-backup false \ | |
| --is-offline false \ | |
| --wait-for-state SUCCEEDED \ | |
| --force | |
| --wait-for-state SUCCEEDED | |
| --backend-set-name snackgame-http \ | |
| --backend-name "$HTTP_BACKEND" \ | |
| --is-drain "$is_drain" \ | |
| --is-backup false \ | |
| --is-offline false \ | |
| --wait-for-state SUCCEEDED |
| --backend-set-name snackgame-https \ | ||
| --backend-name "$HTTPS_BACKEND" \ | ||
| --is-drain "$is_drain" \ | ||
| --is-backup false \ | ||
| --is-offline false \ | ||
| --wait-for-state SUCCEEDED \ | ||
| --force | ||
| --wait-for-state SUCCEEDED |
There was a problem hiding this comment.
snackgame-https 백엔드 세트 업데이트 시에도 --is-backup false 및 --is-offline false 옵션을 유지하여, 배포 완료 후 백엔드가 정상적으로 활성화되어 트래픽을 받을 수 있는 상태임을 보장하는 것이 좋습니다.
| --backend-set-name snackgame-https \ | |
| --backend-name "$HTTPS_BACKEND" \ | |
| --is-drain "$is_drain" \ | |
| --is-backup false \ | |
| --is-offline false \ | |
| --wait-for-state SUCCEEDED \ | |
| --force | |
| --wait-for-state SUCCEEDED | |
| --backend-set-name snackgame-https \ | |
| --backend-name "$HTTPS_BACKEND" \ | |
| --is-drain "$is_drain" \ | |
| --is-backup false \ | |
| --is-offline false \ | |
| --wait-for-state SUCCEEDED |
1 participant
No description provided.