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

[CHORE] 환경 분리를 위한 yml 파일 변경 #106

Merged
merged 8 commits into from
Sep 24, 2023

Conversation

ddongseop
Copy link
Member

@ddongseop ddongseop commented Sep 22, 2023

📌 관련 이슈

#105

✨ 어떤 이유로 변경된 내용인지

환경 분리를 위해 로컬 환경과 배포 서버에서 사용하는 yml 형식을 통일하였음

해당 yml 파일은 로컬 환경과 배포 서버에서 모두 사용할 수 있으며,
어떤 환경 변수를 넣어주는지와 active profile에 따라서 환경이 분리될 것임

로컬 환경에서는 Intellij의 설정을 통해 환경변수에 값을 넣어줄 것이며,
배포 서버에서는 AWS Secret Manager을 통해 환경변수에 값을 넣어주는 원리
(기존에 yaml파일을 아예 생성해줬던 것에서 변경)

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

@ddongseop ddongseop added 🔥 Pull Request PR 날림 ddongseop🍑 🧹Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 🚀Deploy 배포 관련 labels Sep 22, 2023
@ddongseop ddongseop requested a review from jun02160 September 22, 2023 18:25
@ddongseop ddongseop self-assigned this Sep 22, 2023
@ddongseop ddongseop changed the title [CHORE] 환경 분리를 위한 yaml 파일 변경 [CHORE] 환경 분리를 위한 yml 파일 변경 Sep 24, 2023
Copy link
Member

@jun02160 jun02160 left a comment

Choose a reason for hiding this comment

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

오랜만에 approve 합니다~ 완전 수고했어 !!!!!

Comment on lines +45 to +54
# 3) AWS Secrets Manger 환경변수 사용
- name: Read secrets from AWS Secrets Manager into environment variables
uses: abhilash1in/[email protected]
with:
aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY }}
aws-secret-access-key: ${{ secrets.AWS_SECRET_KEY }}
aws-region: ap-northeast-2
secrets: /secret/umbba-dev
parse-json: false

Copy link
Member

Choose a reason for hiding this comment

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

오오 이거 쓴거 너무 좋은데?

Comment on lines +40 to +80
cloud:
aws:
credentials:
accessKey: ${CLOUD_ACCESS}
secretKey: ${CLOUD_SECRET}
region:
static: ${CLOUD_REGION}
s3:
bucket: ${BUCKET_NAME}
stack:
auto: false
sqs:
notification:
name: ${SQS_NOTIFICATION_NAME}
url: ${SQS_NOTIFICATION_URL}

kakao:
client-id: ${KAKAO_ID}
authorization-grant-type: authorization_code
redirect-uri: ${KAKAO_REDIRECT}
apple:
iss: ${APPLE_ISS}
client-id: ${APPLE_ID}
# nonce:

jwt:
secret: ${JWT_SECRET}

slack:
webhook:
url: ${SLACK_URL}

fcm:
key:
path: ${FCM_JSON_PATH}
scope: ${FCM_SCOPE}
# firebase-create-scoped: "https://www.googleapis.com/auth/firebase.messaging"
api:
url: ${FCM_API_URL}
topic:
"qna_notification"
Copy link
Member

Choose a reason for hiding this comment

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

훨씬 깔꼼&굿 이네요

Comment on lines +88 to +135

# local_setting
---
spring:
config:
activate:
on-profile: local_setting

server:
port: 9091

# local_notification_setting
---
spring:
config:
activate:
on-profile: local_notification_setting

server:
port: 9092

# set1_setting
---
spring:
config:
activate:
on-profile: set1_setting

server:
port: 8081

# set2_setting
---
spring:
config:
activate:
on-profile: set2_setting

server:
port: 8082

# notification_setting
---
spring:
config:
activate:
on-profile: notification_setting

Copy link
Member

Choose a reason for hiding this comment

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

환경 분리 넘 최고다 감사함다 ㅠㅠ

@ddongseop ddongseop merged commit 74069be into develop Sep 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 ddongseop🍑 🚀Deploy 배포 관련 🔥 Pull Request PR 날림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants