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

Feat support k8scluster container remote command #1950

Conversation

sykim-etri
Copy link
Member

본 PR은 K8sCluster 내 컨테이너에 원격 명령을 전달할 수 있는 기능 API를 추가합니다.
사용 방식은 VM 인프라 대상 원격 명령과 유사합니다.

o 추가 API

  • /ns/{nsId}/cmd/k8sCluster/{k8sClusterId}

@seokho-son
Copy link
Member

@sykim-etri 기능 추가 감사합니다!

추가된 외부 pkg에 의해서, 상당히 많은 의존성이 추가되는 것 같은데,
혹시 이를 최소화할 수 있는 방법이 있을지 우선 문의드립니다.

  • k8s.io/api v0.31.3
  • k8s.io/client-go v0.31.3

@sykim-etri
Copy link
Member Author

@seokho-son
Kubernetes REST API를 직접 호출하는 신규 라이브러리를 개발하는 방안도 있겠습니다만 효과적이지는 않을 것으로 생각됩니다.

@seokho-son
Copy link
Member

@sykim-etri 의존성을 조금이라도 더 줄일 수 있는 방법을 찾아보고는 있는데, 쉽지 않네요.. :)

(참고) 최근 K8s 커뮤니티에서도 관련 논의가 좀 일어나고 있습니다.

[Umbrella] Make client-go lighter and easier to consume #127888

Copy link
Member

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

@sykim-etri API 코멘트만 조금 보완하면 좋을 것 같습니다.

src/api/rest/server/resource/k8scluster.go Show resolved Hide resolved
Copy link
Member

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

LGTM!

@seokho-son seokho-son merged commit 3517795 into cloud-barista:main Jan 6, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants