[김창우] sprint8 #391

qwertyuiop4m · 2025-04-14T09:11:17Z

요구사항

기본

애플리케이션 컨테이너화
BinaryContentStorage 고도화 (AWS S3)
AWS를 활용한 배포 (AWS RDS, ECR, ECS)

심화

이미지 최적화하기
GitHub Actions를 활용한 CI/CD 파이프라인 구축
CI
CD

스크린샷

env 파일

제출용.txt

RDS

ECR

ECS

VPC

IAM

배포

멘토에게

셀프 코드 리뷰를 통해 질문 이어가겠습니다.

youngxpepp

창우님 고생하셨습니다!

CI/CD 전반적으로 잘해주셔서 크게 고쳐야 될건 없다고 생각해요.
피드백 드린 부분도 사실 정보성 리뷰라 과제 채점에 큰 영향을 주진 않았어요ㅎㅎ
과제 양이 많았는데 요구사항 대부분 다 지켜주셨네요👍

youngxpepp · 2025-04-22T15:01:19Z

.idea/gradle.xml

.gitignore에 .idea가 있는데도 불구하고 제외가 안되나 보네요!
아래 글 보시겠어요?
https://coding-groot.tistory.com/59

youngxpepp · 2025-04-22T15:56:11Z

Dockerfile

+# 필요한 거 먼저 복사
+COPY build.gradle settings.gradle gradlew /app/
+COPY gradle /app/gradle


Dockerfile을 작성할 땐 잘 안 바뀌는 파일을 먼저 COPY 해주시는게 좋습니다.
build.gradle, settings.gradle, gradlew, gradle
위 4개 파일을 잘 안 바뀔거 같은 순서대로 나열해보세요ㅎㅎ

이미지 레이어에 대해선 아래 도커 공식문서 글 보시면 좋습니다!

https://docs.docker.com/get-started/docker-concepts/building-images/understanding-image-layers/

youngxpepp · 2025-04-22T16:05:34Z

Dockerfile

+# 의존성 미리 다운(캐시)
+RUN chmod +x gradlew && ./gradlew dependencies --no-daemon


gradlew는 내부적으로 gradle을 다운로드 합니다.
builder 스테이지의 베이스 이미지가 gradle:8.10-jdk17-alpine 네요.

두 가지중 하나를 선택하는게 어떨까요?

베이스 이미지 gradle

베이스 이미지 amazoncorretto + gradlew

youngxpepp · 2025-04-22T16:06:09Z

Dockerfile

+# 전체 프로젝트 복사 (.dockerignore로 불필요 파일 제외)
+COPY . /app


Suggested change

# 전체 프로젝트 복사 (.dockerignore로 불필요 파일 제외)

COPY . /app

# 전체 프로젝트 복사 (.dockerignore로 불필요 파일 제외)

COPY . .

WORKDIR /app 으로 설정해두셨기 때문에 경로 지정을 하지 않아도 되겠네요ㅎㅎ

youngxpepp · 2025-04-22T16:12:08Z

Dockerfile

+# jar 파일 실행 (환경변수 활용)
+CMD ["sh", "-c", "java $JVM_OPTS -Dspring.profiles.active=$SPRING_PROFILES_ACTIVE -jar /app/app.jar"]


이렇게 실행한다면 1번 프로세스가 쉘이 되겠네요.
컨테이너가 종료될 때 1번 프로세스에 종료 시그널이 전달됩니다.
자바 어플리케이션에 종료 시그널이 전달 되지 않으면 graceful shutdown 을 하기가 어려워요.

정말 중요한 내용이라 아래 글을 번역해서라도 읽어보셨으면 좋겠습니다.

https://engineering.pipefy.com/2021/07/30/1-docker-bits-shell-vs-exec/

youngxpepp · 2025-04-22T16:19:57Z

src/main/java/com/sprint/mission/discodeit/config/StorageConfig.java

+@Slf4j
+@Configuration
+public class StorageConfig {
+
+  @Bean
+  @ConditionalOnProperty(name = "discodeit.storage.type", havingValue = "S3")
+  public S3Client s3Client(
+      @Value("${discodeit.storage.s3.access-key}") String accessKey,
+      @Value("${discodeit.storage.s3.secret-key}") String secretKey,
+      @Value("${discodeit.storage.s3.region}") String region
+  ) {
+    return S3Client.builder()
+        .region(Region.of(region))
+        .credentialsProvider(StaticCredentialsProvider.create(
+            AwsBasicCredentials.create(accessKey, secretKey)
+        )).build();
+  }
+
+  @Bean
+  @ConditionalOnProperty(name = "discodeit.storage.type", havingValue = "s3")
+  public S3Presigner s3Presigner(
+      @Value("${discodeit.storage.s3.access-key}") String accessKey,


Suggested change

@Slf4j

@Configuration

public class StorageConfig {

@Bean

@ConditionalOnProperty(name = "discodeit.storage.type", havingValue = "S3")

public S3Client s3Client(

@Value("${discodeit.storage.s3.access-key}") String accessKey,

@Value("${discodeit.storage.s3.secret-key}") String secretKey,

@Value("${discodeit.storage.s3.region}") String region

) {

return S3Client.builder()

.region(Region.of(region))

.credentialsProvider(StaticCredentialsProvider.create(

AwsBasicCredentials.create(accessKey, secretKey)

)).build();

}

@Bean

@ConditionalOnProperty(name = "discodeit.storage.type", havingValue = "s3")

public S3Presigner s3Presigner(

@Value("${discodeit.storage.s3.access-key}") String accessKey,

@Slf4j

@Configuration

@ConditionalOnProperty(name = "discodeit.storage.type", havingValue = "local", matchIfMissing = true)

public class StorageConfig {

@Bean

public S3Client s3Client(

@Value("${discodeit.storage.s3.access-key}") String accessKey,

@Value("${discodeit.storage.s3.secret-key}") String secretKey,

@Value("${discodeit.storage.s3.region}") String region

) {

return S3Client.builder()

.region(Region.of(region))

.credentialsProvider(StaticCredentialsProvider.create(

AwsBasicCredentials.create(accessKey, secretKey)

)).build();

}

@Bean

public S3Presigner s3Presigner(

@Value("${discodeit.storage.s3.access-key}") String accessKey,

이건 어때요?

youngxpepp · 2025-04-22T16:28:49Z

src/main/resources/application.yml

+    type: ${STORAGE_TYPE:local}
    local:
-      root-path: /path/to/storage
+      root-path: ${STORAGE_LOCAL_ROOT_PATH:.discodeit/storage}
+    s3:
+      access-key: ${AWS_S3_ACCESS_KEY}
+      secret-key: ${AWS_S3_SECRET_KEY}
+      region: ${AWS_S3_REGION}
+      bucket: ${AWS_S3_BUCKET}
+      presigned-url-expiration: ${AWS_S3_PRESIGNED_URL_EXPIRATION:600}


spring boot에서는 환경변수를 properties로 바꿔주는 기능을 제공해요!
DISCODEIT_STORAGE_S3_ACCESSKEY -> discodeit.storage.s3.access-key

https://docs.spring.io/spring-boot/reference/features/external-config.html#features.external-config.typesafe-configuration-properties.relaxed-binding.environment-variables

youngxpepp · 2025-04-22T16:29:41Z

src/main/resources/application-prod.yml

+  datasource:
+    url: jdbc:postgresql://${DB_HOST}:${DB_PORT}/${DB_NAME}
+    username: ${DB_USER}
+    password: ${DB_PASSWORD}
+  driver-class-name: org.postgresql.Driver
+  hikari:
+    maximum-pool-size: 20


Suggested change

datasource:

url: jdbc:postgresql://${DB_HOST}:${DB_PORT}/${DB_NAME}

username: ${DB_USER}

password: ${DB_PASSWORD}

driver-class-name: org.postgresql.Driver

hikari:

maximum-pool-size: 20

datasource:

url: jdbc:postgresql://${DB_HOST}:${DB_PORT}/${DB_NAME}

username: ${DB_USER}

password: ${DB_PASSWORD}

driver-class-name: org.postgresql.Driver

hikari:

maximum-pool-size: 20

들여쓰기가 빠져있네요!

qwertyuiop4m added 19 commits April 10, 2025 16:10

feat: application container

e11d41c

feat: binaryContent 고도화

c8e0dfd

refactor: 멀티 스테이지

d05a8e7

feat: CI/CD

a914ca6

refactor: 오타 수정

dc6077a

refactor: 오타 수정

e6885a4

refactor: 권한 설정

9502eed

refactor: login 설정

cbaba5f

feat: render

97df812

refactor: 파일 위치 변경

9fc8367

refactor: task definition 등록

e506c22

feat: 수동 버튼 추가

b2beb4c

feat: port 설정

cc74aad

feat: port 설정

df8e6ef

feat: 태스크 설정

ff1d5e5

feat: 태스크 설정, port 수정

87ec38d

feat: 태스크

363cc0e

feat: 포트 설정

26461cc

refactor: 테스크 수정(프리티어)

b45d3e8

qwertyuiop4m requested a review from youngxpepp April 14, 2025 09:11

qwertyuiop4m added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Apr 14, 2025

qwertyuiop4m added 9 commits April 14, 2025 18:17

CI test

6f31815

CI test

506b750

CI test

d1f7b97

CI test2

895b92c

CI test

b78a357

CI test

1382329

CI test

fb37371

sprint8

39a8f72

sprint8

3b05c95

qwertyuiop4m added 2 commits April 14, 2025 19:12

sprint8

07fc674

sprint8

134fc8c

youngxpepp approved these changes Apr 22, 2025

View reviewed changes

youngxpepp merged commit 65716b3 into codeit-bootcamp-spring:part3-김창우 Apr 22, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[김창우] sprint8 #391

[김창우] sprint8 #391

Uh oh!

qwertyuiop4m commented Apr 14, 2025 •

edited

Loading

Uh oh!

youngxpepp left a comment

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

youngxpepp Apr 22, 2025

Uh oh!

Uh oh!

		# 의존성 미리 다운(캐시)
		RUN chmod +x gradlew && ./gradlew dependencies --no-daemon

		# 전체 프로젝트 복사 (.dockerignore로 불필요 파일 제외)
		COPY . /app

		# jar 파일 실행 (환경변수 활용)
		CMD ["sh", "-c", "java $JVM_OPTS -Dspring.profiles.active=$SPRING_PROFILES_ACTIVE -jar /app/app.jar"]

[김창우] sprint8 #391

[김창우] sprint8 #391

Uh oh!

Conversation

qwertyuiop4m commented Apr 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

요구사항

기본

심화

스크린샷

env 파일

RDS

ECR

ECS

VPC

IAM

배포

멘토에게

Uh oh!

youngxpepp left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

qwertyuiop4m commented Apr 14, 2025 •

edited

Loading