Skip to content

fix: add partition specific OD pricing - #3336

Merged
bwagner5 merged 2 commits into
aws:mainfrom
tzneal:bake-in-partition-specific-od-pricing
Feb 6, 2023
Merged

bwagner5 merged 2 commits into
aws:mainfrom
tzneal:bake-in-partition-specific-od-pricing

Conversation

@tzneal

@tzneal tzneal commented Feb 1, 2023 •

Copy link
Copy Markdown
Contributor

Fixes #2706

Description
Add static OD pricing for the various partitions to enable more accurate instance launching and consolidation in other partitions. Additionally this fixes the OD pricing lookup for the China partition where prices are in CNY as opposed to USD.

How was this change tested?

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

Release Note


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@tzneal
tzneal requested a review from a team as a code owner February 1, 2023 14:21
@tzneal
tzneal requested a review from njtran February 1, 2023 14:21
@netlify

netlify Bot commented Feb 1, 2023 •

Copy link
Copy Markdown

✅ Deploy Preview for karpenter-docs-prod canceled.

Name Link
🔨 Latest commit 74893fa
🔍 Latest deploy log https://app.netlify.com/sites/karpenter-docs-prod/deploys/63e130031fe4b500099f26fc

@tzneal
tzneal force-pushed the bake-in-partition-specific-od-pricing branch 2 times, most recently from 626dc6d to 6888fc1 Compare February 1, 2023 14:40
Add static OD pricing for the various partitions to enable more
accurate instance launching and consolidation in other partitions.
Additionally this fixes the OD pricing lookup for the China partition
where prices are in CNY as opposed to USD.
@tzneal
tzneal force-pushed the bake-in-partition-specific-od-pricing branch from c8e14e9 to 3e51e8b Compare February 1, 2023 14:58

@bwagner5 bwagner5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

Karpenter is unable to get current on-demand pricing in GovCloud (us-gov-west-1)

2 participants