멘토님 피드백 반영 (1)

최근에 MSSQL PR이 merge 되고, 이슈가 closed 되었다. 

 

마지막으로 pr을 올린 뒤로, 멘토님들이 피드백을 해주셔서 해당 피드백들을 바탕으로 코드를 수정한 기록들을 올려보려고 한다. 

 

[의선 멘토님 피드백]

 

1) 스키마에 정의된 mirror_subnet_no 속성을 멀티존 여부 isMultiZone에 따라서 요청 파라미터에 포함시키기 (cloud_mssql.go)

 

참고 API 문서 : https://api.ncloud-docs.com/docs/database-vmssql-createcloudmssqlinstance

 

수정된 코드

수정된 코드

결과

 

2) 입력 받을수 있는 옵션을 스키마에서 검증할 수 있음

• 현재 스펙상 HDD, SSD 두 개의 옵션을 받을 수 있는데 스키마에서 검증해주기

멘토님 피드백

 

코드 수정하기

 

3) ValidateFunc 에서 ValidateDiagFunc 으로 변환해주는 helper를 sdk 버전으로 사용하기

 

service_name, user_name, user_password, backup_time, port 부분을 수정해주었다. 

service_name 예시

 

4) mssql default port 번호 추가하기

포트 번호 추가

 

[영민 멘토님 피드백]

영민 멘토님 피드백

적용하기

 

 

의선 멘토님의 피드백 반영 중 스키마에 정의된 mirror_subnet_no 속성을 멀티존 여부 isMultiZone에 따라서 요청 파라미터에 포함시키는 것을 mirror 서버를 직접 생성하게 해서 해당 서버와 비교하게 검증을 했다. 

 

따라서 test를 진행했을 때, 콘솔 상에서 mirror 서버가 잘 생성되었는지 확인했다. 

mirror 서버 생성 확인

subnet 생성 확인

 

추가적으로 가이드 문서 attirbute에 mirror_subnet_no 추가하였다. 

mirror_subnet_no 추가

 

마지막으로 pr 반영하고 comment를 작성했다. 

pr 반영 및 comment 작성