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

PriceInfo AWS filter implementation #156

Merged
merged 11 commits into from
Feb 8, 2024

Conversation

SungWoongz
Copy link
Collaborator

AWS 드라이버 특이사항

ListProductFamily

  • 별도 컴퓨팅 인프라 환경 제공하는 SDK 존재하지 않음 → 하드코딩

    var result []string
    
    	result = append(result, "AmazonEC2")
    	return result, nil
    

GetPriceInfo

  • AdminWeb상에서 Pricing Information 관련 필터 적용시 소스상 필터 후처리로 인해 AWS Price Info에는 모든 결과 값들이 나올 수 있음
    • product에는 필터 적용된 값만 정상적으로 출력
  • InstanceType + OS 를 포함한 것이 한 개의 product
    • 각 product는 고유의 sku를 가지고 있다.
    • product는 여러개의 Terms를 가지고 있다.
    • terms는 OnDemand, Reserved 두가지 type이 있으며
      각 type별로 정책에 따른 가격이 여러개 있을 수 있다.

AWS Driver Details

ListProductFamily

  • SDK not present in separate computing infrastructure environment → hardcoding

    var result []string
    
    	result = append(result, "AmazonEC2")
    	return result, nil
    

GetPriceInfo

  • When applying the Pruning Information-related filter on the Admin Web, AWS Price Info may have all the result values due to the post-processing of the filter on the source
    • Only filter applied values are output normally to the product
  • One product with InstanceType + OS
    • Each product has its own sku.
    • The product has several Term's.
    • There are two types of terms: OnDemand and Reserved
      There may be several prices according to the policy for each type.

@SungWoongz SungWoongz merged commit 8814d24 into test_priceinfo_mhlee_240124 Feb 8, 2024
@SungWoongz
Copy link
Collaborator Author

Region -> us-west-1 설정해야함
다른 region 확인 필요

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