최근에 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 부분을 수정해주었다.
4) mssql default port 번호 추가하기
[영민 멘토님 피드백]
의선 멘토님의 피드백 반영 중 스키마에 정의된 mirror_subnet_no 속성을 멀티존 여부 isMultiZone에 따라서 요청 파라미터에 포함시키는 것을 mirror 서버를 직접 생성하게 해서 해당 서버와 비교하게 검증을 했다.
따라서 test를 진행했을 때, 콘솔 상에서 mirror 서버가 잘 생성되었는지 확인했다.
추가적으로 가이드 문서 attirbute에 mirror_subnet_no 추가하였다.
마지막으로 pr 반영하고 comment를 작성했다.
'개발 > Terraform on NaverCloud' 카테고리의 다른 글
멘토님 피드백 반영 (3) & 회고 (1) | 2023.12.07 |
---|---|
멘토님 피드백 반영 (2) (1) | 2023.12.07 |
MSSQL 가이드 문서 pr 반영하기 (0) | 2023.09.23 |
MSSQL 가이드 문서 작성하기 (0) | 2023.09.21 |
MSSQL 개발 코드 보완하기: data source (0) | 2023.09.21 |