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

[enh]: kubeflow address 접속 시 트러블슈팅 가이드 추가 #66

Merged
merged 7 commits into from
Jan 12, 2022

Conversation

RyanKor
Copy link
Contributor

@RyanKor RyanKor commented Dec 31, 2021

Changes?

  • content/kor/contributors 경로에 PR을 작성한 사람의 seungtae-kim 폴더가 추가 되었습니다.
  • content/kor/docs/setup-components/install-components-kf.md 파일에 클라우드 환경에서 실습 시, 접근할 수 있는 address 옵션과 방화벽 포트 설정에 대해 안내하는 사항을 추가했습니다.

(위의 2가지 사항을 각각 한 개의 커밋으로 쪼개 PR 진행)

Why we need?

  • kubectl port-forward svc/ml-pipeline-ui -n kubeflow 8888:80 커맨드 이용 시, http://localhost:8888/#/pipelines/ 파이프라인 주소로 접근할 수 있는 내용이 있습니다.
  • 해당 내용을 동일하게 클라우드 상에서 진행 시, (작성자는 구글 클라우드 사용) 커맨드에 --address 옵션을 주는 것과 동시에 방화벽 포트를 열어줘야하는 내용이 필요한 것을 알게 되었습니다.
  • 따라서, 클라우드를 사용하면 운영 체제 등 (윈도우, 맥북 등을 사용하는 개발자들)의 환경에 영향받지 않고, 자유롭게 클라우드 환경에서 mlops 실습을 진행할 수 있음에 따라 사용자들이 동일한 에러를 피하게끔 하기 위해 해당 내용의 PR을 작성하게 되었습니다.

Test?

  • npm run start 를 수행하여 로컬에서 렌더링된 페이지를 확인하셨나요?
  • npm test 를 통과하였나요?

Anything Else? (Optional)

  • 특정 클라우드 환경에 대한 설명으로 치중이 될까봐 구글 클라우드 인스턴스의 방화벽 설정 이미지 등은 문서에 포함시키지 않은 점을 참고로 말씀드립니다.

Copy link
Member

@anencore94 anencore94 left a comment

Choose a reason for hiding this comment

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

안녕하세요! 모두의 mlops 문서에 관심을 가져주셔서 감사합니다.
한 가지 문의만 확인 부탁드립니다!

content/kor/docs/setup-components/install-components-kf.md Outdated Show resolved Hide resolved

만약 다음과 같이 `localhost에서 연결을 거부했습니다` 라는 에러가 출력될 경우, 커맨드로 address 설정을 통해 접근하는 것이 가능합니다.

보안상의 문제가 되지 않는다면, 아래와 같이 `0.0.0.0` 로 모든 주소의 bind를 열어주는 방향으로 ml-pipeline UI가 정상적으로 접속되는지 확인합니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
보안상의 문제가 되지 않는다면, 아래와 같이 `0.0.0.0` 로 모든 주소의 bind를 열어주는 방향으로 ml-pipeline UI가 정상적으로 접속되는지 확인합니다.
**보안상의 문제가 되지 않는다면,** 아래와 같이 `0.0.0.0` 로 모든 주소의 bind를 열어주는 방향으로 ml-pipeline UI가 정상적으로 접속되는지 확인합니다.

Copy link
Member

Choose a reason for hiding this comment

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

이것만 bold 체로 변경해주시면 감사하겠습니다!

@anencore94 anencore94 changed the title 컨트리뷰터 이름 폴더 추가 및 install-components-kf 파일에 클라우드 환경에서 kubeflow 주소 접근 방법 내용 추가 [enh]: kubeflow address 접속 시 트러블슈팅 가이드 추가 Jan 10, 2022
@RyanKor RyanKor requested a review from a team as a code owner January 11, 2022 13:48
@RyanKor RyanKor requested review from zamonia500 and removed request for a team January 11, 2022 13:48
@RyanKor
Copy link
Contributor Author

RyanKor commented Jan 12, 2022

@zamonia500 아고, 평상시에 제가 이미지 넣는 방식을 그대로 사용해버렸네요.

지금 바로 커밋하겠습니다.

확인 감사합니다!

Copy link
Member

@Aiden-Jeon Aiden-Jeon left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@zamonia500 zamonia500 left a comment

Choose a reason for hiding this comment

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

LGTM :)

@Aiden-Jeon Aiden-Jeon merged commit d267e34 into mlops-for-all:main Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants