fix: 도커파일 이미지 빌드 실패현상 수정

[ParkGyeongTae]

#️⃣ Issue Number

• [버그] [Dockerfile] 도커파일이 빌드되지 않는 현상 #148

📝 요약(Summary)

• docker compose up -d 실행시 Dockerfile로 이미지가 생성되지 않는 현상

# image에 없는 패키지로 이미지 버전 변경
E: Unable to locate package software-properties-common

# requirements.txt가 이 프로젝트에 없어 pyproject.toml로 수정
failed to solve: failed to compute cache key: 
failed to calculate checksum of ref ... "/requirements.txt": not found

# 아래 에러로 인해 복사 후 의존성 설치하는 것으로 수정
COPY pyproject.toml ./
RUN pip install --upgrade pip setuptools wheel \
    && pip install .

💬 To Reviewers (선택)

• docker compose up -d시 문제없시 실행되는지 확인

PR Checklist

• N/A

reference) How to Code Review

• 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
• 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
• 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
• 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

[ehddnr301]

💊
제가 아직 테스트를 못해봤는데 uv를 사용해서 설치하는 방향으로 변경하고 lang2sql 명령어를 쓸 수 있게 변경해보는건 어떠신가요!?
혹은 lang2sql run-streamlit 대신 lang2sql run-streamlit-docker 라던지요.

💬 Docker 부분도 방치되어있었는데 얼른 해결이 필요하겠네요ㅎㅎㅎㅎ;;; 항상 좋은기여 감사합니다.

[ParkGyeongTae]

@ehddnr301 님, 제가 제대로 이해했는지 모르겠지만...
확인 한번 부탁드립니다~
코멘트 주신 부분과 다르다면 가감없이 피드백 주세요~~!

1. pip -> uv
2. port 설정을 이미지에서 지정하지 않고, 컨테이너 실행시 환경변수로 설정
3. lang2sql run-streamlit-docker 명령어 생성 및 사용

[ehddnr301]

@ParkGyeongTae 님 확인이 늦었습니다아ㅜㅜ

👍 변경해주신 내용확인했고 테스트하다보니 제가 의견을 잘못드린것 같습니다아...!ㅜㅜ

docker run --rm -p 30859:8501 <image-name> lang2sql --datahub_server http://34.27.166.177:8080 run-streamlit

Docker로 환경만 제공하고 위 예시처럼 CMD 그냥 덮어써버리면 될 것 같은데 어떻게 생각하시나요!?

cli의 내용만 제거하면 바로 머지해도 될것같습니다아

[ParkGyeongTae]

@ehddnr301 님, 시간 괜찮으실 때 확인 한번 부탁드립니다~!

[ehddnr301]

👍 좋습니다아아 다음부터는 docker build 해서 명령어가 잘 되는지도 체크해봐야겠네요!!

💬 논외의 이야기입니다만 포함하는 패키지가 많아지다보니 설치하는게 너무 많아 느리긴하네요ㅜ!! 혹시 좋은 아이디어 있으시면 의견 부탁드립니다아

[ParkGyeongTae]

@ehddnr301 도커 이미지 내부에 패키지를 줄이는 수 밖에 없을 것 같은데... 흠.. 경량화는 한번 고민해보겠습니다~!~!