search

mlops-for-all / mlops-for-all.github.io

169 stars 38 forks source link

docs: add kor/eng metallb.md #76

Closed kosehy closed 2 years ago

kosehy commented 2 years ago

Changes?

Why we need?

MetalLB를 사용하면 kubectl port-forward 를 사용하지 않고 사용자가 원하는 IP 주소로 쿠버네티스 서비스에 접근할 수 있습니다.

Test?

Anything Else? (Optional)

anencore94 commented 2 years ago

npm test 확인해주시면 감사하겠습니다!

또한 package-lock.json 은 삭제하신 이유가 있으실까요? 제가 frontend 에 대해 잘 이해하고 있지는 않지만, package-lock.json 은 정확한 node package 버전 관리를 위해서는 git diff 가 많이 발생하더라도 소스코드와 함께 관리하는 방향이 best practice 인 것으로 주워들었습니다..ㅎㅎ 그래서 현재 github action 에서도 packge.json 과 함께 package-lock.json 을 바탕으로 패키지 버전을 고정시켜 페이지 렌더링이 되는 것을 확인하고 있습니다.(https://github.com/mlops-for-all/mlops-for-all.github.io/runs/4717966022?check_suite_focus=true) 혹시 제가 잘못 이해하고 있다면 말씀해주세요!

kosehy commented 2 years ago

npm test 확인해주시면 감사하겠습니다!

또한 package-lock.json 은 삭제하신 이유가 있으실까요? 제가 frontend 에 대해 잘 이해하고 있지는 않지만, package-lock.json 은 정확한 node package 버전 관리를 위해서는 git diff 가 많이 발생하더라도 소스코드와 함께 관리하는 방향이 best practice 인 것으로 주워들었습니다..ㅎㅎ 그래서 현재 github action 에서도 packge.json 과 함께 package-lock.json 을 바탕으로 패키지 버전을 고정시켜 페이지 렌더링이 되는 것을 확인하고 있습니다.(https://github.com/mlops-for-all/mlops-for-all.github.io/runs/4717966022?check_suite_focus=true) 혹시 제가 잘못 이해하고 있다면 말씀해주세요!

이 부분은 제가 착각해서 package-lock.json 파일을 삭제했네요... 해당 파일은 다시 추가하겠습니다.

좋은 의견 감사합니다! 여러 사용자를 위해 appendix 로 추가하신 부분도 센스가 넘치시네요 👍 몇몇 코멘트를 남겨두었으니 확인해주시면 감사하겠습니다!

@anencore94 자세히 리뷰해 주셔서 정말 감사합니다! @anencore94 님의 의견을 반영해서 다시 커밋하겠습니다.

Aiden-Jeon commented 2 years ago

@kosehy 안녕하세요! 이미지 경로가 틀려서 노출되지 않는 문제가 있습니다. 현재 이미지가 아래 경로에 있는데 images/docs/metallb/login-after-istio-ingressgateway-setting.png 이 쪽으로 옮겨주시면 될 것 같습니다! static/images/docs/metallb/login-after-istio-ingressgateway-setting.png

kosehy commented 2 years ago

프로젝트의 다른 글들과의 톤앤 매너를 위해 몇가지 코멘트 했습니다. 제가 영문 버전에 코멘트를 달았는데 국문 버전도 동일하게 부탁드립니다! 감사합니다!

@Aiden-Jeon 자세히 리뷰해 주셔서 너무 감사합니다! @Aiden-Jeon 님의 의견을 반영해서 다시 커밋하겠습니다.