Update go.mod - Githubissues

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

Closed jihoon-seo closed 3 years ago

jihoon-seo commented 3 years ago

go mod tidy 명령으로 go.mod 파일을 정리하고, cb-spider & cb-store 의 버전은 수동으로 명시했습니다.
- cb-spider & cb-store 외 다른 dependency 들을 업그레이드 하지는 않았습니다.
beego pkg 의 import path, import version 을 업데이트 했습니다.
- 관련 issue: https://github.com/cloud-barista/cb-tumblebug/issues/368
- 관련 PR: https://github.com/cloud-barista/cb-tumblebug/pull/397
go run *.go 명령으로 정상 실행되는 것을 확인했습니다.

sykim-etri commented 3 years ago

/lgtm

jihoon-seo commented 3 years ago

@sykim-etri CB-Ladybug repo 에는 prow-commands.yml 과 같은 파일이 적용되어 있지 않아서 'command comment 를 이용한 자동 label 달기' 기능이 현재는 작동하지 않고 있습니다.

추후 auto-merge.yml 와 같은 파일도 적용된다면 유용하게 활용하실 수 있겠습니다. :)

sykim-etri commented 3 years ago

@jihoon-seo 혹시 해당 yml을 적용하려면 어떻게 해야 하나요? 손박사님께서 해주실 것 같긴 한데..^^

jihoon-seo commented 3 years ago

@sykim-etri 말씀하신 대로, @seokho-son 께서 CB-Ladybug repo 에 PR 을 올리실 예정입니다.. ^^

[유의 사항] auto-merge.yml 파일을 보시면 현재는 'auto-merge command 사용 가능한 사용자' 가 하드코딩 되어 있습니다. CB-Ladybug 에 적용 시 적절한 사용자 리스트로 대체되어야 하겠습니다.

(추후 하드코딩 부분이 해결될 수도 있습니다.. ^^)

seokho-son commented 3 years ago

@sykim-etri FW 저장소에 대해서 하나씩 추가해나가려고 합니다. CB-Ladybug 먼저 PR을 올려보았습니다. (#26) 해당 PR이 머지되면, Prow 스타일의 커멘드 기능이 활성화될 것입니다..^^

참고로 auto-merge 는 모든 저장소에 바로 반영하지는 않을 예정이며, (TB에서도 아직 원하는대로 동작하지 않습니다.) 각자 필요에 따라 활용하시면 될 것 같습니다. (코드 참고: https://github.com/cloud-barista/cb-tumblebug/blob/master/.github/workflows/auto-merge.yml)

sykim-etri commented 3 years ago

@seokho-son 네. 알겠습니다. 이제 CB-Ladybug의 github이 더 예쁘게 보이겠네요.ㅎㅎ