Skip to content
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.

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

[#285] Add API Dockerfile #286

Conversation

skgndi12
Copy link
Collaborator

#285

Changes

  • Add API Dockerfile

@skgndi12 skgndi12 added api infra Infrastructure-related tasks and updates. labels Mar 14, 2024
@skgndi12 skgndi12 requested review from isutare412 and 2wheeh March 14, 2024 11:45
@skgndi12 skgndi12 self-assigned this Mar 14, 2024
api/Dockerfile Outdated Show resolved Hide resolved
@isutare412 isutare412 self-requested a review March 15, 2024 00:27
Copy link
Collaborator

@isutare412 isutare412 left a comment

Choose a reason for hiding this comment

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

.dockerignore 파일도 추가 부탁드립니다~

@isutare412 isutare412 self-requested a review March 15, 2024 05:21
@skgndi12
Copy link
Collaborator Author

d5ca4c7 에서 .dockerignore 파일에 이미지 빌드시 불필요한 폴더 및 파일들 목록을 작성하여 추가했습니다.

@isutare412
Copy link
Collaborator

d5ca4c7 에서 .dockerignore 파일에 이미지 빌드시 불필요한 폴더 및 파일들 목록을 작성하여 추가했습니다.

node_modules 가 포함되지 않은 것 같은데, 다음 내용에 대해 정리해주신 후에 node_modules 도 추가해주시겠어요?

  1. docker build context 란 무엇인지
  2. .dockerignore 파일이 하는 역할

@skgndi12
Copy link
Collaborator Author

1573552 에서 node_modules 폴더도 .dockerignore 파일에 추가했습니다.

  1. docker build context 란 무엇인지

도커의 빌드 컨텍스트란 도커가 이미지를 빌드하기 위해 사용할 작업 디렉토리를 의미합니다. 빌드 명령어에 컨텍스트를 명시적으로 설정할 수 있으며 이 컨텍스트는 로컬 디렉토리 위치뿐 아니라 URL 도 가능합니다.

2 .dockerignore 파일이 하는 역할

.dockerignore 파일은 도커의 빌드 컨텍스트에서 제외할 파일 및 폴더 목록들을 작성한 파일입니다. 이미지 경량화 및 보안을 위해서 사용할 파일 및 폴더들을 명시적으로 관리하기 위해 사용합니다.

Copy link
Collaborator

@isutare412 isutare412 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 👍

@skgndi12 skgndi12 force-pushed the feature/issue-285/create-api-dockerfile branch from 1573552 to 9771f29 Compare March 15, 2024 07:44
@skgndi12 skgndi12 merged commit 5f3f49a into MovieReviewComment:develop Mar 15, 2024
1 check passed
@skgndi12 skgndi12 deleted the feature/issue-285/create-api-dockerfile branch March 15, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api infra Infrastructure-related tasks and updates.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants