-
Notifications
You must be signed in to change notification settings - Fork 586
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
Add CPU Credits Data for Burstable Instance Types #596
Comments
Isn't this already covered by scrape.py:add_t2_credits()? |
yes, this should be there already @jcgoette or were you maybe expecting something else? |
It's been awhile, but I think I meant adding data such as |
I see. I've always just calculated those as
Credits per hour = base performance*60 and max credits = base perf*60*24. I
guess they could be added though.
The max credits is already on the website (not sure about the json file).
The credits per hour is not, but it's just max credits/24.
With this info, how do you wish to proceed?
Rishi
…On Wed, May 18, 2022, 10:43 PM Justin Goette ***@***.***> wrote:
It's been awhile, but I think I meant adding data such as CPU credits
earned per hour and Maximum earned credits that can be accrued. Unless
that is somewhere I missed?
—
Reply to this email directly, view it on GitHub
<#596 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZFF2IINYMYXL7PNCVIBZTVKWTELANCNFSM5I7LT3ZA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
We can probably add a derived column for this. We have many filters which are just multiplication (daily, weekly charges). |
Data from https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/burstable-credits-baseline-concepts.html#earning-CPU-credits.
Tangentially related to #147.
I'm happy to take a whack at this as time permits.
The text was updated successfully, but these errors were encountered: