Migrate "study" dir to root #11
Conversation
| Jenkinsfile | ||
| tools | ||
| .git | ||
| .gitignore |
There was a problem hiding this comment.
크 .dockerignore 파일까지 챙기시다니.. 꼼꼼하시네요 👍
Dockerfile
Outdated
| @@ -1,19 +1,20 @@ | |||
| # config-service-build stage | |||
There was a problem hiding this comment.
아래에 as config-service-build 로 stage 이름이 명시되어있어 해당 주석은 불필요해보입니다.
코드와 중복되는 의미를 갖는 주석은 유지보수시에 두번고쳐야하는 단점이 있습니다.
There was a problem hiding this comment.
오 그렇네요. 그럼 지우도록 하죠
Dockerfile
Outdated
| ADD pom.xml pom.xml | ||
| RUN mvn clean dependency:go-offline | ||
|
|
||
| ADD src src | ||
| RUN mvn -o package | ||
|
|
||
|
|
||
| # running stage |
There was a problem hiding this comment.
이 주석도 불필요해보입니다.
이보다 document 링크 주석이 더 올바르지 않나 생각됩니다.
https://docs.docker.com/develop/develop-images/multistage-build/
There was a problem hiding this comment.
# Ref: https://docs.docker.com/develop/develop-images/multistage-build/
| COPY --from=config-service-build /app/target/*.jar . | ||
|
|
||
| EXPOSE 8088 | ||
|
|
||
| CMD java -jar *.jar | ||
| ENTRYPOINT ["sh", "-c"] |
There was a problem hiding this comment.
entrypoint 에 sh -c를 두는 이유가있나요?
There was a problem hiding this comment.
기존 Dockerfile의 내용을 가져다 사용했습니다.
sh -c 를 쓰는 이유는 변수를 읽어들이기 위해 사용했었습니다.
뭐 현재는 필요없지만, 원본 Dockerfile의 경우
ENTRYPOINT ["sh", "-c"]
CMD ["exec java -jar $APP_FILE"]이렇게 돼있어서 sh -c가 없다면 $APP_FILE을 변수로 대체하지 않고
그냥 java -jar $APP_FILE 명령으로 실행합니다. 그래서 에러가 나게 되고요.
docker: Error response from daemon: OCI runtime create failed: container_linux.go:345: starting container process caused "exec: \"java -jar $APP_FILE\": executable file not found in $PATH": unknown.재영님이 jar파일을 변수명 처리 하지 않고 그냥 와일드카드로 줘서 굳이 필요 없긴 하지만, 놔뒀습니다.. ㅎ
|
|
||
| CMD java -jar *.jar | ||
| ENTRYPOINT ["sh", "-c"] | ||
| CMD ["exec java -jar *.jar"] |
There was a problem hiding this comment.
이것도 그냥 원본 갖다 쓴거 입니다.
이와 별개로 CMD에 재영님의 shell form(CMD java -jar *.jar) 을 사용하지 않고 exec form(CMD ["exec java -jar *.jar"])을 사용한 이유는 Docker 공식 레퍼런스에 그걸 추천한다고 되어 있어서 입니다.
README.md
Outdated
|
|
||
| - For java run(`Dockerfile`) | ||
| - To get the docker image, run: | ||
| `docker build -t tiny:tiny .` |
There was a problem hiding this comment.
naming은 그냥 작은거라서 tiny로 했습니다.
config-service 같은 걸로 고쳐야 겠네요.
- add annotations to Dockerfile - add `.dockerignore` file to reduce docker build context - add README
.dockerignorefile to reduce docker build context#10