Skip to content

build/도커 이미지 배포 #4

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.

Sign up for GitHub

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

Merged
GoGradually merged 2 commits into from
Dec 20, 2025
Merged

build/도커 이미지 배포 #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
767dfe2
build: Dockerfile 및 .dockerignore 파일 추가
GoGradually Dec 20, 2025
02495b5
build: envsubst 설치 및 이미지 풀 비밀 추가
GoGradually Dec 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .dockerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
**

!Dockerfile
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

(1) 문제점: .dockerignore 파일에서 Dockerfile 자체를 포함(!Dockerfile)하고 있는데, 이는 불필요합니다. Dockerfile은 Docker 빌드 과정에서 자동으로 처리되며 이미지 내부에 복사할 필요가 없습니다.

(2) 영향: 보안 및 이미지 크기 관점에서 불필요한 파일이 포함될 수 있습니다.

(3) 수정 제안: !Dockerfile 라인을 제거하세요.

Suggested change
!Dockerfile

Copilot uses AI. Check for mistakes.
!k8s/**
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

(1) 문제점: .dockerignore 파일이 k8s/** 디렉토리를 포함하도록 설정되어 있지만, Dockerfile에서는 k8s 디렉토리를 사용하지 않습니다. 불필요한 파일이 Docker 빌드 컨텍스트에 포함됩니다.

(2) 영향: Docker 빌드 컨텍스트 크기가 불필요하게 증가하여 빌드 속도가 느려질 수 있습니다.

(3) 수정 제안: k8s/** 라인을 제거하세요. Dockerfile에서 실제로 필요한 build/libs/*.jar 파일만 포함하도록 .dockerignore를 단순화하는 것이 좋습니다.

Suggested change
!k8s/**

Copilot uses AI. Check for mistakes.

!build/
!build/libs/
!build/libs/*.jar
3 changes: 3 additions & 0 deletions .github/workflows/cd.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,9 @@ jobs:

echo "KUBECONFIG=$PWD/kubeconfig" >> $GITHUB_ENV

- name: Install envsubst
run: sudo apt-get update && sudo apt-get install -y gettext-base

- name: Deploy (apply manifest with GITHUB_SHA substitution)
shell: bash
run: |
Expand Down
28 changes: 28 additions & 0 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
# syntax=docker/dockerfile:1

FROM eclipse-temurin:21-jre-jammy

WORKDIR /app

# 비루트 실행(쿠버네티스 securityContext와도 정합성 좋음)
RUN useradd -r -u 10001 -g root appuser \
&& mkdir -p /app \
&& chown -R 10001:0 /app

# GitHub Actions에서 ./gradlew clean test build 로 생성된 산출물 사용
# (주의) build/libs 에 plain.jar 와 bootJar가 같이 생길 수 있어 bootJar를 선택하도록 처리
COPY build/libs/*.jar /app/

RUN set -eux; \
JAR="$(ls /app/*.jar | grep -v -- '-plain\.jar$' | head -n 1)"; \
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

(1) 문제점: JAR 파일 선택 로직에서 여러 개의 non-plain JAR 파일이 존재할 경우 head -n 1을 사용하여 첫 번째 파일만 선택합니다. 이는 예측 불가능한 동작을 유발할 수 있습니다.

(2) 영향: 빌드 시 여러 버전의 JAR 파일이 존재하거나 예상치 못한 JAR 파일이 선택되어 잘못된 애플리케이션이 실행될 수 있습니다.

(3) 수정 제안: JAR 파일이 정확히 하나만 있는지 검증하는 로직을 추가하거나, 특정 패턴(예: *-SNAPSHOT.jar)을 명시적으로 지정하세요. 예: JAR_COUNT를 확인하여 1개가 아닌 경우 에러를 발생시키는 방식을 고려하세요.

Suggested change
JAR="$(ls /app/*.jar | grep -v -- '-plain\.jar$' | head -n 1)"; \
JARS="$(ls /app/*.jar | grep -v -- '-plain\.jar$' || true)"; \
JAR_COUNT="$(printf '%s\n' "$JARS" | sed '/^$/d' | wc -l)"; \
if [ "$JAR_COUNT" -ne 1 ]; then \
echo "ERROR: Expected exactly 1 non-plain JAR in /app, but found $JAR_COUNT." >&2; \
echo "Non-plain JAR candidates:" >&2; \
printf '%s\n' "$JARS" >&2; \
exit 1; \
fi; \
JAR="$(printf '%s\n' "$JARS" | sed '/^$/d')"; \

Copilot uses AI. Check for mistakes.
mv "$JAR" /app/app.jar; \
rm -f /app/*-plain.jar || true; \
chown 10001:0 /app/app.jar
Comment on lines +14 to +20
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

(1) 문제점: build/libs 디렉토리의 모든 JAR 파일을 복사한 후 불필요한 파일을 삭제하는 방식은 비효율적입니다. Docker 레이어 캐싱 관점에서도 최적화되어 있지 않습니다.

(2) 영향: 빌드 과정에서 불필요한 파일 복사 및 삭제 작업이 발생하며, Docker 레이어가 불필요하게 증가합니다.

(3) 수정 제안: COPY 명령 전에 plain.jar를 제외하고 복사하거나, gradle 설정에서 plain jar 생성을 비활성화(jar { enabled = false })하는 것을 권장합니다.

Copilot uses AI. Check for mistakes.

USER 10001

# Spring Boot 기본 포트가 8080인 경우가 많아 문서화 목적(EXPOSE는 필수 아님)
EXPOSE 8080

# JVM 옵션은 Kubernetes 매니페스트에서 JAVA_TOOL_OPTIONS로 주입 권장
ENTRYPOINT ["java","-jar","/app/app.jar"]
2 changes: 2 additions & 0 deletions k8s/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ spec:
labels:
app: pinit-auth
spec:
imagePullSecrets:
- name: ghcr-pull-secret
terminationGracePeriodSeconds: 30
volumes:
- name: keys
Expand Down