Skip to content

[KOBG-4] 기초 세팅#1

Merged
yerim123456 merged 22 commits intodevelopfrom
feature/KOBG-4/initial-setting
Aug 17, 2025
Merged

[KOBG-4] 기초 세팅#1
yerim123456 merged 22 commits intodevelopfrom
feature/KOBG-4/initial-setting

Conversation

@yerim123456
Copy link
Contributor

@yerim123456 yerim123456 commented Jul 26, 2025

🔑 JIRA 이슈 키

KOBR-4 기초 세팅


✒ 진행한 작업

  • spring boot initializar 통한 기초 파일 세팅
  • pr 템플릿 생성
  • 아키택처 적용
  • 공통 코드 작업
  • 배포 및 pipline 작업

💡 생겼던 문제 및 해결법

1️⃣ .gitattributes 파일 업로드 여부

  • 처음 보는 파일이 생겨서 해당 파일 업로드 여부를 고민했으나, os 간 줄바꿈 문제 방지, 바이너리 파일(ex .jar, .png, .mp4) 를 줄바꿈 처리하지 않도록 하여 파일 손상 방지 등의 장점을 갖기에 업로드하기로 설정

2️⃣ rds-ec2 연결 시, rds의 private subnet 설정

  • rds 를 private subnet 으로 사용할 수밖에 없게 강제된 상황에서 ssh 를 통해 intelij 에서 연결하는 방법을 배울 수 있었음
  • 이렇게 설정된 이유는 ec2 > rds 로 들어왔을 때 db를 public 인 상태로 두어 외부에서 쉽게 접근할 수 없도록 막아 기본적인 보안을 지키기 위해서임

3️⃣ docker compose ver2 사용
저번 cd pipeline 에서는 매번 docker compose up -d 를 ssh 에서 다시 해줘야 하는 문제가 있었어서 해당 문제 이유가 docker contanier 의 최신 버전을 캐시에서 가져와서 업데이트가 제대로 안되는 것일거란 피드백을 기반으로 cd pipeline 과 docker-compose.yml 을 수정하여 latest 로 최신화 되도록 우선 작업했다.
그 과정에서 보다 간단하게 작업하면서 최신의 docker를 사용할 수 있도록 도운 docker compose version v2.33.1 을 사용했다. 이로써 docker compose up 과 -pull alawys 2개의 라인을 1개로 줄일 수 있었다.

4️⃣ window 사용시, checkstyle 개행 문자 규칙 문제
현재 윈도우 사용 중이나, naver checkstyle 은

    <module name="RegexpMultiline">
        <property name="format" value="\r\n"/>
        <property name="message" value="[newline-lf] Line must end with LF, not CRLF"/>
    </module>

해당 부분으로 LF 만 강제하고 있다. 해당 부분을 삭제하는 방법도 있겠지만, 추후에도 사용하기 위해 경고를 없애고자 .gitattributes 를 통해 윈도우라도 push 시 알아서 LF 로 변경되도록 강제했다. 예외 규칙을 통해 window의 CRLF가 꼭 필요한 .bat 파일은 제외하고 모든 텍스트 파일은 LF로 통일 되도록 설정했다.


📢 아쉬운 부분 및 개선점

  • nginx 추가할 예정
  • github.sha 를 활용한 IMAGE TAG 를 추가 문제 해결

📚 개발에 참고한 자료 및 포인트


@yerim123456 yerim123456 self-assigned this Jul 26, 2025
@yerim123456 yerim123456 added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 26, 2025
@github-actions
Copy link

📊 Jacoco Test Coverage

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@github-actions
Copy link

📊 Jacoco Test Coverage

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@github-actions
Copy link

📊 Jacoco Test Coverage

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@yerim123456 yerim123456 merged commit 165c29a into develop Aug 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant