cloud-barista / cm-honeybee

Apache License 2.0
3 stars 2 forks source link

Need to add an API that return/response on-premise model #35

Open yunkon-kim opened 1 week ago

yunkon-kim commented 1 week ago

@ish-hcc (cc. @powerkimhub)

안녕하세요. Honeybee API 추가를 부탁 드립니다.

지금까지의 여건 상, Honeybee에서는 소스 컴퓨팅 인프라에 대한 형상 정보를 그대로 제공(Raw 데이터)하는 API가 제공되었습니다.

On-premise 모델 (a.k.a. 인프라 소스 모델) 초안을 공유드리오니, 형상 정보를 정제(Refine) 및 제공하는 API 추가를 부탁드립니다. : 기존 제공되던 형상 정보를 On-premise 모델에 담아서 리턴해주시는 정도의 API이면 될 것 같습니다. : Honeybee에서 공유해주신 형상 정보 및 기존 자료들을 바탕으로, : 마이그레이션에 필요한 것으로 보이는 정보들만으로 정제 및 모델화한 버전입니다. (Depth도 가급적 줄여준 상태입니다.)

: 관련 모델 pkg

import onprem "github.com/cloud-barista/cm-model/infra/onprem"

: On-premise model과 관련한 이슈, 개선의견 등이 있으시면 편하게 말씀해주시기 바랍니다 ^^

(요약) : on-premise 모델을 별도 저장소 cm-model에서 제공해 봤습니다. : 실험적이고, 모델만 import 하고 싶은데, 관련 없는 pkg들이 영향을 주는 것을 피하고자 별도의 저장소를 개설하였습니다. : 자세한 사항은 https://github.com/cloud-barista/cm-model 를 참고하시기 바랍니다.

yunkon-kim commented 1 week ago

단위와 관련하여 말씀드릴 사항이 있습니다.

전체적으로 봤을 때, 단위를 GiB, GHz로 통일하면 좋을 것 같았습니다. 이를 주석에 남겨 놓았으니 검토 부탁 드리겠습니다. https://github.com/cloud-barista/cm-model/blob/main/infra/onprem/infra.go

yunkon-kim commented 1 week ago

@ish-hcc 마이너한 수정 사항이 있었습니다 🙏 참고 부탁드립니다.

ish-hcc commented 1 week ago

@yunkon-kim 확인했습니다 수고하셨습니다! 확인해서 작업해보도록 하겠습니다

ish-hcc commented 17 hours ago

@yunkon-kim 박사님 안녕하세요? 작업해주신 cm-model을 활용해서 honeybee에서 정제된 데이터를 반환해 주는 API를 구현하여 테스트 완료 하였습니다. 한가지 제안 드릴 사항이 있는데, Gateway의 경우 Array 형태로 인터페이스 마다 존재 할 수 있도록 하고, IP의 경우에도 Array 형태로, 여러 IP를 담을 수 있도록 개선해 주시면 어떨까 합니다. 현재 Honeybee Agent에서는 인터페이스 마다 다중 Gateway, 다중 IP를 수집하고 있습니다.

실제 인프라 구성에서도 참고가 되지 않을까 하여 제안드립니다. 아니면, 인프라 추천, 마이그레이션에서만 현 상태를 유지하고, 필요에 의해서 다중 Gateway, 다중 IP를 봐야 할때는 정제된 데이터가 아닌 수집된 Raw데이터를 참고한다고 하면, 현 상태를 유지해도 될 것 같습니다.

yunkon-kim commented 16 hours ago

@ish-hcc 의견 공유 감사합니다 :-)

의견 주신 모델 관련 사항 논의를 위해 https://github.com/cloud-barista/cm-model/issues/5 를 열었습니다.

https://github.com/cloud-barista/cm-model/issues/5#issuecomment-2359799263 이 코멘트를 중심으로 살펴봐 주시면 될 것 같습니다 :-)