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

[+Price] Implement Azure's PriceInfoHandler #990

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

ish-hcc
Copy link
Member

@ish-hcc ish-hcc commented Dec 19, 2023

  • Implement all of 2 functions of PriceInfoHandler
    • ListProductFamily()
    • GetPriceInfo()

@ish-hcc
Copy link
Member Author

ish-hcc commented Dec 19, 2023

CSP 마다 정보가 다르다 보니 파싱하기가 만만치 않네요.
추후에 조금 더 다듬을 필요는 있어보입니다. 일단 작업한 부분까지 완성하여 풀 리퀘스트 작성드립니다.

@powerkimhub
Copy link
Member

CSP 마다 정보가 다르다 보니 파싱하기가 만만치 않네요. 추후에 조금 더 다듬을 필요는 있어보입니다. 일단 작업한 부분까지 완성하여 풀 리퀘스트 작성드립니다.

  • 넵, 맞습니다.
  • 일차 큰 틀에서 개발하고, 사용해보면서 점차 개선해가야 할 걸로 생각됩니다.

@powerkimhub
Copy link
Member

  • Call Log를 추가해주셨네요.

  • 추가해주시면 좋지만, 시간이 부족하시면 안하셔도 되겠습니다.

  • 관련하여, HisCall(History of Call) 관련해서 기존에 메일로 공유 드렸던 내용을 재안내 드립니다.

    현재 자원 개발시 마다 적용 중인 Call Log 작업에 대해서 
    아래와 같은 사유로 잠정 보류하고자 합니다.
    
    현재 Call Log 기능 및 운영 체계가 이미 적용 되어 있으므로 
    향후, 필요시 쉽게 적용할 수 있으리라 생각합니다. 
    * 기존, cb-log는 현행대로 유지 부탁 드립니다.
    
    [사유]
    * 신규 자원 확장 개발에 공수 집중
    * 모든 자원에 대한 Call Log 정보가 필요하지는 않다는 점
    * 비동기 호출 등으로 인한 호출 소요 시간 정보 부적합 성 등 내재
    * 향후 필요 시점에 필요 자원에 추가 및 필요시 기존 호출 교정
    * 이후 자원 개발 시부터 적용
    

@powerkimhub powerkimhub merged commit 3b03d06 into cloud-barista:master Dec 19, 2023
1 check passed
@powerkimhub powerkimhub linked an issue Dec 22, 2023 that may be closed by this pull request
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.

[+Price] Azure Cloud Driver Implementation
3 participants