cloud-barista / cb-ladybug

Cloud-Barista Multi-Cloud Application Runtime Framework : Support Multi-Cloud Kubernetes Service
Apache License 2.0
11 stars 11 forks source link

Add grpc api #48

Closed jmleefree closed 3 years ago

jmleefree commented 3 years ago
sykim-etri commented 3 years ago

@jmleefree 카페모카 버전 릴리즈 후 리뷰 및 머지할 예정이니 일정에 참고하시기 바랍니다.

jihoon-seo commented 3 years ago

go.mod 에 conflict 가 발생하여, resolve 했습니다. (8e205c6)

그런데, build 에러가 나네요

https://github.com/cloud-barista/cb-ladybug/pull/48/checks?check_run_id=2970529882#step:3:71

Step 5/19 : RUN GOOS=linux CGO_ENABLED=0 go build -ldflags '-w -extldflags "-static"' -tags cb-ladybug -o cb-ladybug -v src/main.go
 ---> Running in 310d4ecaf6c8
go: github.com/cloud-barista/cb-log@v0.4.0: missing go.sum entry; to add it:
    go mod download github.com/cloud-barista/cb-log
The command '/bin/sh -c GOOS=linux CGO_ENABLED=0 go build -ldflags '-w -extldflags "-static"' -tags cb-ladybug -o cb-ladybug -v src/main.go' returned a non-zero code: 1

Error: Process completed with exit code 1.

제가 conflict 를 resolve 할 때 go.sum 은 수정하지 않아서 발생하는 에러입니다.

제가 좀 더 탐구해 보겠습니다.

jihoon-seo commented 3 years ago

/lgtm 필요하다면 squash 를 하는 방안도 있겠습니다.

sykim-etri commented 3 years ago

@itnpeople @vlatte docs/test의 *.sh들이 GRPC 테스트 지원을 위해 CB_CALL_METHOD라는 환경 변수를 기준으로 일부 변경되었습니다. REST API 테스트 코드는 그대로 들어가 있습니다만 테스트가 되진 않았을 것으로 판단됩니다. 리뷰하시고 의견 부탁드립니다.

vlatte commented 3 years ago

@sykim-etri test script 중 REST API 부분 테스트 완료하였습니다.

sykim-etri commented 3 years ago

@vlatte 테스트까지 해주셨군요. 코드상으로만 특이한 점이 없는지만 봐주셨어도 되는데.. 감사합니다.