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

ALIBABA Dirver Tag handler feature #1251

Merged
merged 7 commits into from
Jul 25, 2024

Conversation

SungWoongz
Copy link
Contributor

작업내용

  • 지원하는 자원별 TagList 추가, 조회 개발
  • ALIBABA tag 지원 자원 VM, VPC, SUBNET, SG, KEYPAIR, NLB, DISK, MYIMAGE, CLUSTER
  • 인터페이스 개발
    • type DescribeVpcsResponse struct
    • type DescribeVSwitchesResponse struct
  • Func 개발
    • Tag 추가 : AddTag(resType irs.RSType, resIID irs.IID, tag irs.KeyValue) (irs.KeyValue, error)
    • Tag 목록 : ListTag(resType irs.RSType, resIID irs.IID) ([]irs.KeyValue, error)
    • Tag 정보 : GetTag(resType irs.RSType, resIID irs.IID, key string) (irs.KeyValue, error)
    • Tag 삭제 : RemoveTag(resType irs.RSType, resIID irs.IID, key string) (bool, error)
    • Tag 찾기 : FindTag(resType irs.RSType, keyword string) ([]*irs.TagInfo, error)
  • ALIBABA Test_Resource 테스트 수행

특이사항 : alibaba cloud 태그 검색 시 LIKE 기능 지원하지 않음

Copy link

Could you please check and revise Korean texts?
Note - All output of print and log statements should be in English. 😉

cloud-control-manager/cloud-driver/drivers/alibaba/main/Test_Resources.go

100:+					cblogger.Infof(" Tag 목록 조회 실패 : ", err)
102:+					cblogger.Info("Tag 목록 조회 결과")
111:+					cblogger.Info(" Tag 조회 실패 : ", err)
113:+					cblogger.Info("GetTag 조회 결과")
122:+					cblogger.Info(" Tag 조회 실패 : ", err)
124:+					cblogger.Info("FindTag 조회 결과")
134:+					cblogger.Info(" Tag 조회 실패 : ", err)
136:+					cblogger.Info("AddTag 조회 결과")
144:+					cblogger.Info(" Tag 조회 실패 : ", err)
146:+					cblogger.Info("RemoveTag 조회 결과")
198:+					cblogger.Infof(" Tag 목록 조회 실패 : ", err)
200:+					cblogger.Info("Tag 목록 조회 결과")
209:+					cblogger.Info(" Tag 조회 실패 : ", err)
211:+					cblogger.Info("GetTag 조회 결과")
220:+					cblogger.Info(" Tag 조회 실패 : ", err)
222:+					cblogger.Info("FindTag 조회 결과")
232:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
234:+				// 		cblogger.Info("AddTag 조회 결과")
242:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
244:+				// 		cblogger.Info("RemoveTag 조회 결과")
295:+					cblogger.Infof(" MyImage 목록 조회 실패 : ", err)
297:+					cblogger.Info("MyImage 목록 조회 결과")
306:+					cblogger.Info(" MyImage 조회 실패 : ", err)
308:+					cblogger.Info("GetMyImage 조회 결과")
317:+					cblogger.Info(" MyImage 조회 실패 : ", err)
319:+					cblogger.Info("CreateMyImage 조회 결과")
329:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
331:+				// 		cblogger.Info("AddTag 조회 결과")
339:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
341:+				// 		cblogger.Info("RemoveTag 조회 결과")
406:+					cblogger.Infof(" MyImage 목록 조회 실패 : ", err)
408:+					cblogger.Info("MyImage 목록 조회 결과")
416:+					cblogger.Info(" MyImage 조회 실패 : ", err)
418:+					cblogger.Info("GetMyImage 조회 결과")
427:+					cblogger.Info(" MyImage 조회 실패 : ", err)
429:+					cblogger.Info("CreateMyImage 조회 결과")
439:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
441:+				// 		cblogger.Info("AddTag 조회 결과")
449:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
451:+				// 		cblogger.Info("RemoveTag 조회 결과")

cloud-control-manager/cloud-driver/drivers/alibaba/resources/CommonHandler.go

572:+		cblogger.Info("쿼리", queryParams)
656:+	// cblogger.Info("쿼리", queryParams)
705:+	// cblogger.Info("쿼리", queryParams)

Copy link

Could you please check and revise Korean texts?
Note - All output of print and log statements should be in English. 😉

cloud-control-manager/cloud-driver/drivers/alibaba/main/Test_Resources.go

100:+					cblogger.Infof(" Tag 목록 조회 실패 : ", err)
102:+					cblogger.Info("Tag 목록 조회 결과")
111:+					cblogger.Info(" Tag 조회 실패 : ", err)
113:+					cblogger.Info("GetTag 조회 결과")
122:+					cblogger.Info(" Tag 조회 실패 : ", err)
124:+					cblogger.Info("FindTag 조회 결과")
134:+					cblogger.Info(" Tag 조회 실패 : ", err)
136:+					cblogger.Info("AddTag 조회 결과")
144:+					cblogger.Info(" Tag 조회 실패 : ", err)
146:+					cblogger.Info("RemoveTag 조회 결과")
198:+					cblogger.Infof(" Tag 목록 조회 실패 : ", err)
200:+					cblogger.Info("Tag 목록 조회 결과")
209:+					cblogger.Info(" Tag 조회 실패 : ", err)
211:+					cblogger.Info("GetTag 조회 결과")
220:+					cblogger.Info(" Tag 조회 실패 : ", err)
222:+					cblogger.Info("FindTag 조회 결과")
232:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
234:+				// 		cblogger.Info("AddTag 조회 결과")
242:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
244:+				// 		cblogger.Info("RemoveTag 조회 결과")
295:+					cblogger.Infof(" MyImage 목록 조회 실패 : ", err)
297:+					cblogger.Info("MyImage 목록 조회 결과")
306:+					cblogger.Info(" MyImage 조회 실패 : ", err)
308:+					cblogger.Info("GetMyImage 조회 결과")
317:+					cblogger.Info(" MyImage 조회 실패 : ", err)
319:+					cblogger.Info("CreateMyImage 조회 결과")
329:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
331:+				// 		cblogger.Info("AddTag 조회 결과")
339:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
341:+				// 		cblogger.Info("RemoveTag 조회 결과")
406:+					cblogger.Infof(" MyImage 목록 조회 실패 : ", err)
408:+					cblogger.Info("MyImage 목록 조회 결과")
416:+					cblogger.Info(" MyImage 조회 실패 : ", err)
418:+					cblogger.Info("GetMyImage 조회 결과")
427:+					cblogger.Info(" MyImage 조회 실패 : ", err)
429:+					cblogger.Info("CreateMyImage 조회 결과")
439:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
441:+				// 		cblogger.Info("AddTag 조회 결과")
449:+				// 		cblogger.Info(" Tag 조회 실패 : ", err)
451:+				// 		cblogger.Info("RemoveTag 조회 결과")

@powerkimhub powerkimhub self-requested a review July 24, 2024 13:01
@powerkimhub
Copy link
Member

@SungWoongz


| 특이사항 : alibaba cloud 태그 검색 시 LIKE 기능 지원하지 않음

  • 관련하여, FindTag()에서 keyword 포함 검색(LINKE와 같은)의 경우
  • 자체 개발로 LIKE 기능을 추가하신 걸로 이해하면 되는지요?

@SungWoongz
Copy link
Contributor Author

SungWoongz commented Jul 25, 2024

@powerkimhub

  • equal만 하기 때문에 like는 고려하지 않았습니다.
  • 모든 태그를 내부에서 like 처리하기에는 부하가 심할 것 같습니다.
  • like 기능을 반드시 해야되는 것인지 결정 해주시길 바랍니다.

@powerkimhub
Copy link
Member

@SungWoongz


  • equal만 하기 때문에 like는 고려하지 않았습니다.
  • 모든 태그를 내부에서 like 처리하기에는 부하가 심할 것 같습니다.
  • like 기능을 반드시 해야되는 것인지 결정 해주시길 바랍니다.

  • Spider 규격 및 타 CSP와의 기능적 일관성을 위하여
    • 일단, 추진 부탁 드립니다.
  • 가급적, Go routine, fetch한 정보 재활용 등
    • 성능 이슈를 잠재울 수 있는 방법을 고려해 주시기 바랍니다.

@SungWoongz
Copy link
Contributor Author

@powerkimhub

  • 네 알겠습니다.
  • 이번 PR 이후에 추가로 보완하겠습니다.

@powerkimhub
Copy link
Member

@SungWoongz

  • Like 기능 관련하여,
  • #1253 으로 오픈해두었습니다.

@powerkimhub powerkimhub merged commit f6d9d5f into cloud-barista:master Jul 25, 2024
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.

3 participants