-
Notifications
You must be signed in to change notification settings - Fork 360
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
[WX-1792] Emit Cost Per Hour #7550
Conversation
…ute/cromwell into jgd_WX-1784_realCostResponse
…ute/cromwell into jgd_WX-1784_realCostResponse
…e/cromwell into WX-1792-emit-cost-per-hour
/* | ||
The cost catalog is annoyingly inconsistent and unstructured in this area. | ||
- For N1 machines, only predefined SKUs are included, and they have "Predefined" in their description strings. | ||
We will eventually fall back to using these SKUs for custom machines, but accurately represent them as Predefined here. | ||
- For non-N1 machines, both custom and predefined SKUs are included, custom ones include "Custom" in their description | ||
strings and predefined SKUs are only identifiable by the absence of "Custom." | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is so annoying. Definitely appreciate the comment!
This remains ready for review, but I haven't been able to do the scale testing I planned because there's a change needed to get the auth to GCP working in Terra deployments. Planning to take care of that and do the scale testing before merging this. |
Description
Creates
vmCostPerHour
metadata records for PAPIv2beta workflow tasks. Currently this cost includes only RAM and CPU, not GPU, disks, etc.Release Notes Confirmation
CHANGELOG.md
CHANGELOG.md
in this PRCHANGELOG.md
because it doesn't impact community usersTerra Release Notes