Skip to content

Commit

Permalink
Caddy 관련 flow 다시 수정합니다. (#15)
Browse files Browse the repository at this point in the history
* feat/fix: apply NAT IP only AWS to LOKI request by Caddy
1. codedang AWS에서 오는 요청만 LOKI에서 수집하기 위해 Caddy에서 특정 NAT IP만 프록시하도록 설정하였습니다.
2. github action flow 를 보완하였습니다.
- Caddy를 reload하는 과정은 static frontend file mount가 정상적으로 작동하지 않았을 때 해결하는 과정이므로 불필요하다고 생각합니다.
- 사용하지 않는 volume을 삭제하는 과정은 추후 로그 분석시 이전 log가 삭제될 가능성을 염두하였을 때 불필요하다고 생각합니다.
- container가 running인지 확인하는 과정은 log container에 국한되므로, 이를 분리합니다.

* fix(Caddy): reload caddy not running compose caddy up always

* fix(Caddy): edit env variable of Caddyfile

* fix(cd): set env by secret
  • Loading branch information
goathoon authored Jan 31, 2024
1 parent 500018a commit 3eacd39
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/update-stage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,8 @@ jobs:
docker compose --profile caddy up -d --no-recreate
- name: Set Caddyfile Environment Variables
env:
AWS_REQ_IP: ${{ secrets.AWS_REQ_IP}}
run: |
sed -i "s/{$AWS_REQ_IP}/$AWS_REQ_IP/" ./Caddyfile
sed -i 's/{$AWS_REQ_IP}/${{secrets.AWS_REQ_IP}}/' ./Caddyfile
- name: Copy Caddyfile into Caddy Container
run: |
Expand Down

0 comments on commit 3eacd39

Please sign in to comment.