add cmd for showing BYOH-agent installation guide #27
Conversation
cmd/show_byoh_agent_guide.go
Outdated
| $ echo "127.0.0.1 $(hostname)" >> /etc/hosts | ||
|
|
||
| # Download agent | ||
| $ curl -LO https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/releases/download/v0.2.0/byoh-hostagent-linux-amd64 |
There was a problem hiding this comment.
이거 저희쪽 repo로 저장해서 다운로드 받게 할 수 있을까요?
There was a problem hiding this comment.
네 요즘 이런 니즈가 많아지니 다용도로 쓸 파일 서버를 하나 준비하는게 좋겠네요.
There was a problem hiding this comment.
네 요즘 이런 니즈가 많아지니 다용도로 쓸 파일 서버를 하나 준비하는게 좋겠네요.
우선 openinfradev github 공간을사용해도 되지 않을까요?
There was a problem hiding this comment.
네 일단 급한대로 사용할 수는 있습니다. https://github.com/robertchoi80/sample-petclinic/blob/main/spring-petclinic-2.7.0-SNAPSHOT.jar 파일이 제 개인 repo에 임시로 올린 것인데, 소스 코드 repo에 binary 파일 올리는게 적절한지 잘 모르겠어서요, 좀더 좋은 방안이 있지 않을까 생각했습니다.
There was a problem hiding this comment.
일단 말씀하신 방향으로 진행하는게 좋을거 같아서 수정했습니다.
| ) | ||
|
|
||
| // showBYOHAgentGuideCmd represents command for showing BYOH agent installation guide. | ||
| var showByohAgentGuideCmd = &cobra.Command{ |
There was a problem hiding this comment.
agent를 설치하고자 하는 (BYOH에 의해 K8S 노드가 될) 대상 호스트의 Role은 출력된 명령어 내용을 복사한 후 사용자가 다시 수정을 해야하는데 출력된 내용을 그대로 복사하고 붙여넣기로 실행하는 형태가 되면 좋겠습니다. 노드 Role 형태 역시 저희가 정해서 사용할 것이기 때문에 중간에 수동으로 수정 작업이 필요하다면 실수할 가능성도 있다고 생각합니다.
tks cluster show-[node-, 생략?]agent-cmd --type <controlplane|tks|worker> 형태로 쓰는게 어떨까 싶은데, BYOH만 해당되는 것이라 적합한 서브 명령어 이름은 더 생각을 해봐야 합니다.
- show-node-cmd
- show-byoh-node-cmd
- show-byoh-node-agent-cmd
- ...
There was a problem hiding this comment.
그렇게 type을 명시적으로 지정하게 하려면, 일단 사용자가 각 type의 용도에 대해 정확히 파악을 해야겠군요. 그럼 guide 출력할 때 해당 부분 설명을 추가하는게 좋겠네요. 이러이러한 role이 있으니 상황에 맞게 잘 입력하라고.. (topology 예시도 포함한다던지..) 명령어 이름은, '-cmd' 로 끝나는 이름은 왠지 각 agent 에서 수행 가능한 action command를 의미하는 느낌이라 좀 고민이 필요하겠습니다.
There was a problem hiding this comment.
의견주신 부분 수정하였습니다.
- type 인자 추가해서 자동으로 출력되도록 하고,
- help message 에 node type 설명 추가
- 커맨드명에 3번과 유사하게 'node' 추가하였습니다.
재검토 부탁드립니다.
|
This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
cmd/show_byoh_agent_guide.go
Outdated
| sudo ./byoh-hostagent-linux-amd64 --kubeconfig mgmt-cluster.conf --namespace $YOUR_CLUSTER_UUID $LABEL_OPT --v 20 2>&1 | tee byoh-agent.log | ||
|
|
||
| # Install binary bundle | ||
| $ curl -L -o k8s-v1.22.3-bundle.tar.gz https://www.dropbox.com/s/4v2ug32pl4p8rq1/k8s-v1.22.3.tar.xz?dl=0 |
There was a problem hiding this comment.
xz 로 압축한 것이라 k8s-v1.22.3-bundle.tar.xz 로 저장하는 것이 좋겠습니다.
There was a problem hiding this comment.
네 그렇네요. 근데 이 dropbox는 주관님 개인 계정인가요?
|
This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
|
파일들은 전부 동일하게 tks-file-repo 의 'stable' release asset 으로부터 다운로드 받도록 수정하였습니다. 머지합니다. |
https://github.com/openinfradev/tks-issues/issues/267