[김지선] sprint11#218
Hidden character warning
Conversation
|
아니.. 다 됬다고 생각했는데.. Github Action한테 발목 잡혔네요 🤣 |
|
멘토님, 현재 제 원격 레포에만 "secrets" 키 목록이 저장 되어 있어 여러 에러가 발생 한 것으로 확인 됩니다. 보통의 팀 작업은 공용 레포에 "secrets"를 저장해 사용하기 때문에 제가 초기에 셋팅해 둔 방식으로 한다면 문제가 되지 않지만, 하여, 제출했던 코드에서 push / pull_request 진행 범위를 좁히고, 제가 이해한 내용이 잘못되었거나, 해결 방식이 잘못 되었다면 조언 부탁드립니다 🙇♂️ |
KWANHYEONGLEE
left a comment
KWANHYEONGLEE
left a comment
There was a problem hiding this comment.
이전 미션 피드백 반영 및 CI/CD 파이프라인을 구축하시느라 정말 수고 많으셨습니다 👏
특히 여럿 테스트를 진행해보면서 나온 고민의 흔적이 보여서 좋았습니다.
스프린트 미션 진행하시느라 정말 수고 많으셨고 남은 연휴 잘 보내시길 바라겠습니다. LGTM 🍀
|
|
||
| const likeProductMember = await favoriteRepository.getFavoriteMember(productId); |
There was a problem hiding this comment.
불필요한 DB조회 로직을 잘 수정하셨네요! 👍
| env: | ||
| # DB 정보 | ||
| DATABASE_URL: ${{ secrets.DATABASE_URL }} | ||
| NODE_ENV: production | ||
|
|
||
| # JWT 설정 | ||
| JWT_ACCESS_TOKEN_SECRET: ${{ secrets.JWT_ACCESS_TOKEN_SECRET }} | ||
| JWT_REFRESH_TOKEN_SECRET: ${{ secrets.JWT_REFRESH_TOKEN_SECRET }} | ||
|
|
||
| # AWS 설정 | ||
| AWS_REGION: ${{ secrets.AWS_REGION }} | ||
| AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| AWS_BUCKET_NAME: ${{ secrets.AWS_BUCKET_NAME }} | ||
|
|
||
| # EC2 설정 | ||
| EC2_HOST: ${{ secrets.EC2_HOST }} | ||
| EC2_USER: ${{ secrets.EC2_USER }} | ||
| EC2_PRIVATE_KEY: ${{ secrets.EC2_PRIVATE_KEY }} |
There was a problem hiding this comment.
secrets 값을 직접 주입하지않고 env섹션에서 한 번 정의해서 사용하셨네요!
저는 직접 주입을 하고있어 수정해야하는 부분이 있으면 하나하나 다 찾아야했었는데 이렇게 사용하면 유지보수 측면에서 좋겠네요 🤩 지선님 덕분에 배웠습니다. 감사합니다!
|
|
||
| deploy: | ||
| # run 환경 | ||
| if: github.repository == 'KimDay366/6-sprint-mission' # 현재 개인 레포지토리에만 secret 설정이 되어있어 해당 조건을 추가하였습니다 |
There was a problem hiding this comment.
우선 오류 원인을 파악하시고 if를 사용하여 대응하신게 너무 좋네요!
말씀해주신것처럼 코드잇 레포지토리의 특성상 secrets값이 팀원별로 다르기때문에 secrets값을 주입하기가 어렵긴하겠네요 🤔 이번엔 CI/CD를 해보는 경험이 목표이기 때문에 지금도 충분히 좋습니다.
현업에서는 환경별로 다른 값을 넣어주어야한다면 다음 방법을 사용하곤합니다!
secrets 이름으로 분기
각 환경에 맞는 식별자를 붙인 secrets값을 등록합니다.
(ex. KIM_SSH_HOST, KWAN_SSH_HOST)
GitHub Environments
Github 레포지토리 설정에서 Environment를 생성하고 환경별로 sercret 값을 설정하는 방법입니다.
There was a problem hiding this comment.
오호!!! 해당 방법도 같이 공부 해 보겠습니다~ 감사합니다 😁
| pull_request : # 브랜치에 코드가 머지될 때 | ||
| branches : | ||
| - 김지선-sprint11 # action test용 | ||
| - 김지선 | ||
| - main |
There was a problem hiding this comment.
위처럼 작성하면 PR이 생성될때 실행이 되는걸로 알고있었는데 아니였나보군요? 😀
There was a problem hiding this comment.
저도 PR로 생각 했는데, PR을 확인 한 뒤 기존 브랜치에 머지 할 때 실행되는 거라고 하네요~ 😉
2 participants
요구사항
[ 목표 ]
[ 작업 내용 ]
1. 기본 요구사항
멘토에게