개발/Terraform on NaverCloud20 멘토님 피드백 반영 (3) & 회고 마지막 멘토님 피드백 반영 및 오픈소스 활동에 대한 회고를 해보려고 한다. [원철 멘토님 피드백] 1) 폴더 및 파일명 cloudmssql → mssql 로 변경 2) sensitive 속성 함께 추가하기 & 패스워드 validation 조건에 backtick이 빠짐 참고 : https://github.com/dusdjhyeon/terraform-provider-ncloud/commit/ba1a9bd694b1e6fe5b2381239c87e5bf809f4583 "user_password": { Type: schema.TypeString, Required: true, Sensitive: true, ValidateDiagFunc: validation.ToDiagFunc(validation.All( valid.. 2023. 12. 7. 멘토님 피드백 반영 (2) 멘토님들께서 계속해서 피드백을 주셨다. [원철 멘토님 피드백] 1) cloud_mssql.go에서 region_code attribute 삭제하기 2) IntDivisibleBy(1433) 대신 IntBetween(1433, 1433) 이나 IntInSlice([]int{ 1433}) 또는 validation 함수를 추가하는 형태로 변경 "port": { Type: schema.TypeInt, Optional: true, ValidateDiagFunc: validation.ToDiagFunc(validation.Any( //validation. validation.IntBetween(10000, 20000), validation.IntBetween(1433, 1433), // 변경 부분 )), Descr.. 2023. 12. 7. 멘토님 피드백 반영 (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 에서 Val.. 2023. 12. 7. MSSQL 가이드 문서 pr 반영하기 가이드 문서를 좀 더 보완해주고 pr을 올렸다. comment도 같이 남겼다. 가이드 문서까지 작성하니, 개발을 실감했다. 나중에 반영되어서 terraform registry에 보여진다면 뿌듯할 것 같다 !! 2023. 9. 23. 이전 1 2 3 4 5 다음
