cloud-barista / cb-operator

The Operator for Cloud-Barista system
Apache License 2.0
6 stars 6 forks source link

Update README #115

Closed jihoon-seo closed 3 years ago

jihoon-seo commented 3 years ago
jihoon-seo commented 3 years ago

@powerkimhub

jihoon-seo commented 3 years ago

@powerkimhub 꼼꼼한 확인 감사합니다.

Table format이 깨지고 있습니다. 확인이 필요해보입니다.


Kubernetes의 경우 Port를 Compose 버전과 동일하게 할 수는 없는지요?



[안]

  1. NodePort 서비스가 이용하는 포트의 범위를 SP, TB 등에서 사용하고 있는 포트를 포함하도록 지정 (가능한지는 모르겠습니다)
  2. cb-operator run 명령 시퀀스에 kubectl port-forward 명령을 통해 CB 구성요소 Pod들을 expose 하는 단계를 추가

이번 PR 말고, 다음번에는 kube~ mode와 compose mode 가이드가 많은 부분 중복되고 있는 데, 중복되는 부분을 하나로 공유 할수 있는 방법으로 개선하면 좋을 듯합니다.

맞습니다. 중복되는 부분을 스니펫으로 만들고 include (embed) 할 수 있으면 좋은데 (스니펫 예시) 찾아 보니 GitHub README 에는 이러한 기능을 사용할 수 없는 것 같습니다.

embed 가 불가능하다면, 다음으로는, 중복 부분을 별도의 파일로 분리하고 그 파일로의 링크를 거는 방법이 있는데요, 이것은 유저의 context switching 을 유발합니다.

위의 사항을 고려하여, 다음 중에서 결정하여 적용하면 될 것 같습니다.

  1. 현행 유지
    • 단점: 중복 부분 많음, 수정이 필요한 경우 양쪽 수정 필요 등...
  2. 중복 부분을 별도의 파일로 분리하고 그 파일로의 링크를 걸기
    • 단점: 유저의 context switching 을 유발
  3. 다른 방법을 발굴
powerkimhub commented 3 years ago

[안]

NodePort 서비스가 이용하는 포트의 범위를 SP, TB 등에서 사용하고 있는 포트를 포함하도록 지정 (가능한지는 모르겠습니다) cb-operator run 명령 시퀀스에 kubectl port-forward 명령을 통해 CB 구성요소 Pod들을 expose 하는 단계를 추가

jihoon-seo commented 3 years ago

제가 merge 하겠습니다.. ^^