Skip to content
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

fix: dev 데이터 환경 초기화 삭제 #153

Merged
merged 3 commits into from
Sep 8, 2024
Merged

Conversation

GaBaljaintheroom
Copy link
Contributor

@GaBaljaintheroom GaBaljaintheroom commented Sep 8, 2024

😋 작업한 내용

  • dev 환경 데이터 초기화 삭제
  • alarm 서버에서 prod 환경에서 APPLICATION_DATASOURCE_URL_PROD을 넣어주지 않아 github action에서 키를 넣어주었습니다.

🙏 PR Point

  • dev만 sql init을 삭제하면 될까요?
  • 추가로 prod 환경의 CI/CD는 prod 브랜치에 배포되었을 때 trigger로 발생되는데, prod 브랜치는 없애고, main 브랜치로 머지가 될 때 실행되는 건 어떻게 생각하시나용?

👍 관련 이슈


@GaBaljaintheroom GaBaljaintheroom added the fix fix feature label Sep 8, 2024
@GaBaljaintheroom GaBaljaintheroom self-assigned this Sep 8, 2024
@devmizz
Copy link
Contributor

devmizz commented Sep 8, 2024

@GaBaljaintheroom

generate-ddl: true
defer-datasource-initialization: true

두 개 옵션도 변경해야 할 거에요!

prod ci/cd의 경우, 저희 main 브랜치가 아니라 prod로 하기로 했었던 거 같은데, main이 일단 왜 남아있는지는 모르겠네요..?
그리고 default 브랜치도 develop말고 prod여야 할 거 같긴 한데...

@GaBaljaintheroom
Copy link
Contributor Author

generate-ddl은 false로 해두었고, defer-datasource-initialization의 경우 데이터베이스 연결 전에 먼저 다른 구성 파일이나 Bean이 준비되기를 원할 때 사용해서 true로 해도 될 것 같다고 생각했어용.

저희 main은 없애고 prod로 하기로 했었군요? 기억이 안나서 몰랐네용

@GaBaljaintheroom
Copy link
Contributor Author

prod 브랜치는 만들어져있는데, main 브랜치는 삭제하면 되는 걸까용?

@devmizz
Copy link
Contributor

devmizz commented Sep 8, 2024

defer-datasource-initialization을 true로 한다는 건 hibernate 초기화 이후 관련 스크립트(data.sql, schema.sql)를 사용하겠다는 옵션인 걸로 아는데, 혹시 다르게 아는 부분이 있을까요?
만약 준수님이 말한대로라고 하더라도, db 연결 전에 다른 구성 파일이나 bean이 준비되는게 지금 있나요? 혹은 그 가능성으로 염두해두는게 있을까요?

넹 main은 그냥 삭제해도 돼요

Copy link
Contributor

@devmizz devmizz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이슈는 크게 없을 거 같아서 approve 미리 눌러놓을게요!

@GaBaljaintheroom
Copy link
Contributor Author

아 착각했네용 false로 수정했습니당.

@GaBaljaintheroom GaBaljaintheroom merged commit 604a8cc into develop Sep 8, 2024
1 check passed
@GaBaljaintheroom GaBaljaintheroom deleted the fix/data_init branch September 8, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 데이터 초기화 설정 삭제
2 participants