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.

Sign up for GitHub

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

Jump to bottom

[WX-1792] Emit Cost Per Hour #7550

Open
wants to merge 47 commits into
base: develop
Choose a base branch
from
Open

[WX-1792] Emit Cost Per Hour #7550

wants to merge 47 commits into from
+594 −140

Conversation

THWiseman
Copy link
Contributor

@THWiseman THWiseman commented Sep 18, 2024 •
edited by jgainerdewar
Loading

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

  • I updated CHANGELOG.md in this PR
  • I assert that this change shouldn't be included in CHANGELOG.md because it doesn't impact community users

Terra Release Notes

  • I added a suggested release notes entry in this Jira ticket
  • I assert that this change doesn't need Jira release notes because it doesn't impact Terra users

jgainerdewar and others added 30 commits September 4, 2024 15:53
@jgainerdewar
Improve doc
5d5f06f
@jgainerdewar
Merge branch 'develop' into jgd_WX-1784_realCostResponse
8346395
@jgainerdewar
Terrible but functional
0024b74
@jgainerdewar
Remove support for includeTaskBreakdown, includeSubworkflowBreakdown
d0b3164
@jgainerdewar
Nicer json walking
a29e9b7
@jgainerdewar
Better status handling
2ca8c3f
@jgainerdewar
Report cost computation errors
9fccf2b
@jgainerdewar
More better error handling
f5b5b9b
@jgainerdewar
Merge branch 'develop' into jgd_WX-1784_realCostResponse
c02b264
@jgainerdewar
Remove dead code
520ff4a
@jgainerdewar
Merge branch 'jgd_WX-1784_realCostResponse' of github.com:broadinstit…
df6dcc1 
…ute/cromwell into jgd_WX-1784_realCostResponse
@jgainerdewar
Interpret BigDecimal as a number in metadata
8baf043
@jgainerdewar
MetadataBuilderActor unit tests
39f6d3d
@jgainerdewar
Standardize on BigD
e5a51ae
@jgainerdewar
Oops there's more
ceb077c
@THWiseman
Merge branch 'develop' into jgd_WX-1784_realCostResponse
da531e9
@jgainerdewar
More tests
cbacb80
@jgainerdewar
Merge branch 'jgd_WX-1784_realCostResponse' of github.com:broadinstit…
39ec889 
…ute/cromwell into jgd_WX-1784_realCostResponse
@jgainerdewar
Use a real currency
984e50e
@jgainerdewar
Merge branch 'develop' into jgd_WX-1784_realCostResponse
7de9ca1
@THWiseman
the gist
c044228
@THWiseman
Merge branch 'develop' into WX-1792-emit-cost-per-hour
331aeb0
@THWiseman
bonus debug logging
ac5f731
@THWiseman
RAM usage
26e841d
@jgainerdewar
Bug fixes
df7abf7
@jgainerdewar
Cost catalog massaging
21c0376
@jgainerdewar
Standardize resource type
7c18124
@jgainerdewar
Drop cost catalog entries we won't use
a8ed112
@jgainerdewar
Fall back to predefined sku for N1 custom
b1f7237
@jgainerdewar
Rearrange
d59c321
@jgainerdewar jgainerdewar marked this pull request as ready for review September 30, 2024 21:01
@jgainerdewar jgainerdewar requested a review from a team as a code owner September 30, 2024 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@THWiseman @jgainerdewar