Check appropriate 3rd party pkg

jihoon-seo commented 3 years ago

관련 issue: https://github.com/cloud-barista/cb-tumblebug/issues/368
현재 CB-Dragonfly 에서는 shaodan/kapacitor-client 패키지가 많이 활용되고 있습니다.
그런데 이 패키지는 공식 repo 가 아니라 forked repo 라서, 향후 업데이트가 잘 될 것이라고 기대하기 어려운 점 등이 있습니다.
CB-Dragonfly 에서 이를 공식 repo 로 대체하는 것이 가능할까요?
- 예: import "github.com/shaodan/kapacitor-client" => import "github.com/influxdata/kapacitor/client/v1" 로 replace 하고, (필요 시 go.mod 를 업데이트 하고,) CB-Dragonfly 가 정상적으로 build & run 되는지 테스트

한 번 확인해 주시면 감사하겠습니다.. ^^

jihoon-seo commented 3 years ago

FOSSA 를 이용한 라이센스 분석 결과를 보면, shaodan/kapacitor-client 패키지에 라이센스가 명시되어 있지 않다는 이슈가 있습니다.

dev-secloudit commented 3 years ago

@jihoon-seo

현재 kapacitor 공식 repo가 아닌 별도의 repo를 통해서 현재 추가적으로 개발된 기능을 일부 사용하고 있습니다. 가장 먼저 해당 기능들이 기존 공식 repo에서도 충분히 호환해서 지원하는 지 확인할 필요가 있을 것 같습니다.

해당 부분에 대해서는 공식 repo에 대해서 적용 테스트 후 결과를 공유드리도록 하겠습니다.

dev-secloudit commented 3 years ago

@jihoon-seo

해당 내용을 확인해본 결과, kapacitor 공식 client 패키지를 활용하는 경우, 예전의 influxql 패키지를 import 하고 있어서 go.mod를 활용하는데 문제가 있습니다. 따라서 go Modules에 호환 가능한 kapacitor client를 사용하고 있습니다.

CB-Dragonfly 패키지 변경 시 go.mod 에러 (go mod tidy 명령어 실행)

go: finding module for package github.com/influxdata/influxdb/influxql
github.com/cloud-barista/cb-dragonfly/pkg/core/alert imports
        github.com/influxdata/kapacitor/client/v1 imports
        github.com/influxdata/influxdb/influxql: module github.com/influxdata/influxdb@latest found (v1.8.5), but does not contain package github.com/influxdata/influxdb/influxql

참고자료

[1] Kapacitor Client - No package github.com/influxdata/influxdb/influxql https://github.com/influxdata/kapacitor/issues/1986 -> 공식 kapacitor client 활용 시 duplicated 된 패키지를 활용하기 때문에 go Module 기반으로 구동 시 에러 발생

[2] Convert build to go mod https://github.com/influxdata/kapacitor/issues/2378 -> kapacitor 오픈소스의 경우 아직 go.mod 기반의 빌드를 지원하지 않음 (위와 같은 kapacitor client import 에러 발생)

cloud-barista / cb-dragonfly

Check appropriate 3rd party pkg #64