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

Check appropriate 3rd party pkg #64

Open
jihoon-seo opened this issue Feb 10, 2021 · 3 comments
Open

Check appropriate 3rd party pkg #64

jihoon-seo opened this issue Feb 10, 2021 · 3 comments
Assignees

Comments

@jihoon-seo
Copy link
Member

jihoon-seo commented Feb 10, 2021

  • 관련 issue: Check appropriate 3rd party pkg cb-tumblebug#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
Copy link
Member Author

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

image

@dev-secloudit
Copy link
Member

@jihoon-seo

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

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

@dev-secloudit
Copy link
Member

dev-secloudit commented Apr 21, 2021

@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
influxdata/kapacitor#1986
-> 공식 kapacitor client 활용 시 duplicated 된 패키지를 활용하기 때문에 go Module 기반으로 구동 시 에러 발생

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

@dev-secloudit dev-secloudit pinned this issue May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants