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

Create 구현 #11

Merged
merged 7 commits into from
Dec 26, 2023
Merged

Create 구현 #11

merged 7 commits into from
Dec 26, 2023

Conversation

n-y-kim
Copy link
Member

@n-y-kim n-y-kim commented Dec 26, 2023

PR 요약

  • Create 기능 구현 거의 완료.. ㅜ
  • Create을 하기 위해 NetworkInterfaceList 정보 GET 추가
  • Create을 하기 위해 SubnetList 정보 GET 추가
  • Create을 하기 위해 AccessControlGroupList 정보 GET 추가

변경 사항 명시

ServerInterface의 Create

CreateRequestString 추가

  • 기존의 request.go/response.go 에 있던 함수를 덜어내고 request_response_function.go 를 새로 생성함.
  • 파라미터 값들을 url string으로 엮기 위한 역할.

Subnet 구조체, 인터페이스 추가

  • pkg/subnet.go
  • types/server/subnet.go

NetworkInterface 구조체 수정, 인터페이스 추가

  • pkg/networkinterface.go
  • types/server/network.go

AccessControlGroupList 구조체 추가, 인터페이스 추가

  • pkg/accesscontrolgroup.go
  • types/server/accesscontrolgroup.go

기타 특이사항

  • Create 파라미터의 NetworkInterfaceList 구현이 불가피함. NetworkInterfaceList GET 부분 추가 구현해서 별도 PR 올릴 예정.
  • NetworkInterfaceList 관련 필드 정의가 잘못됨. 다시 정의해서 NetworkInterfaceList PR에 포함할 예정
  • Create는 변수끼리 Conditional 조건이 엮어있어서 추가 구현해야 할 부분들이 좀 더 있음

@n-y-kim n-y-kim merged commit 3de2d1a into cloud-club:main Dec 26, 2023
3 checks passed
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.

[TODO] NCP VM 생성 입출력 구현
1 participant