Skip to content

[INITOS-JIWON] - Github Actions & Discord - #19

Merged
yureutaejin merged 2 commits into
mainfrom
INITOS-JIWON/github-actions
Jul 3, 2025
Merged

[INITOS-JIWON] - Github Actions & Discord#19
yureutaejin merged 2 commits into
mainfrom
INITOS-JIWON/github-actions

Conversation

@yucori

@yucori yucori commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Description

Github actions로 build와 레지스트리 업로드를 자동화 해봤습니다.
웹훅으로 디스코드 alert까지 연동 시켰는데 actions 짜는게 생각보다 재밌더라고요
예전에 파이썬 처음 배울 때의 느낌이랄까요ㅋㅋ

지금까지 스터디 운영해주신 윤태님께 감사의 말씀 드립니다~!! 다들 수고하셨습니다!!

@github-actions
github-actions Bot requested review from window9u and yureutaejin July 2, 2025 13:40

@yureutaejin yureutaejin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

고생하셨습니다 👍

Comment on lines +22 to +24
- runner 등록 과정에서 Must not run with sudo 문제 발생
⇒ `export RUNNER_ALLOW_RUNASROOT="1"` 으로 문제 해결
- 원래 runner는 root 사용이 비권장되지만, bootc를 통해 os 이미지를 관리하기 위해서 root로 runner 실행하도록 함

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  1. RUNNER_ALLOW_RUNASROOT 레퍼런스 달면 좋을 것 같습니다.
  2. Runner 용 User를 새로 만들고 필요한 유저/그룹 권한 할당하는 것을 해보면 좋을 것 같아요

Comment on lines +56 to +57
- runner를 run.sh로 단순 실행했다가 터미널이 종료되면 runner가 listen을 못하기에 nohup을 통해 백그라운드로 실행
- `nohup ./run.sh & > nohup.out`

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

이거 홈서버 스터디 때 말한건데...😂
참고

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

secrets 말고도 vars도 있으니 다음에 사용할 땐 참고

Comment on lines +116 to +122
- name: 'Notify Discord - Fail'
if: steps.build-phase.outcome != 'success'
uses: rjstone/discord-webhook-notify@v1
with:
severity: error
details: 'Build - Fail'
webhookUrl: ${{ secrets.DISCORD_WEBHOOK }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

star 75개 Community action이라 유의해서 사용 (직접 만드는 거 권장)

@yureutaejin
yureutaejin merged commit 1a205e5 into main Jul 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants