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

Apply changes for API prefix addition of Spider and TB #98

Conversation

jihoon-seo
Copy link
Member

/spider 대응:
conf/setup.env
export SPIDER_URL=http://localhost:1024/spider

(Spider URL 지정이 환경 변수로 되어 있으니
docker, docker-compose 에서 컨테이너 띄울 때에도
환경변수를 바꾸면 돼서 편하고 좋네요)


/tumblebug 대응:
src/apiserver/apiserver.go
g := e.Group("/tumblebug/ns", common.NsValidation())


[TODO]

  • test (Shooter) 스크립트에 /spider, /tumblebug 변경 반영 안 한 파일들이 많음
  • 컨테이너 이미지 새로 만들고, Docker Hub 에 올리기
  • CB-Deployer 의 docker-compose.yaml 업데이트
  • API-GW conf 업데이트
  • 등등

Related:

@seokho-son
Copy link
Member

@jihoon-seo 좋은 방법인 것 같습니다.
향후에는 SPIDER_URL (endpoint) 에 대한 설정도,
API를 통해 수정가능하도록 만들면 다양한 상황에 따라 능동적으로 대처할 수 있을 것으로 보입니다.

혹시 해당 커밋은 기본적인 테스트 (spider를 통한 VM 및 MCIS 생성 등)이 완료된 것인지요?

@jihoon-seo
Copy link
Member Author

@seokho-son
Spider를 통한

  • VM 및 MCIS 생성 은 안 해봤고, 대신에
  • AWS publicIP 생성, 삭제 는 되는 것을 확인하였습니다.

(이 commit 의 주요 변경사항인

  • /tumblebug prefix 를 붙여야만 TB 가 인식하는 것
  • TB 가 Spider 에 API call 할 때 /spider prefix 붙여서 call 하는 것

을 확인하였습니다.)

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

Successfully merging this pull request may close these issues.

None yet

2 participants