-
Notifications
You must be signed in to change notification settings - Fork 587
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
Read cost observability table [VS-475] #7923
Conversation
Codecov Report
@@ Coverage Diff @@
## ah_var_store #7923 +/- ##
================================================
Coverage ? 86.286%
Complexity ? 35188
================================================
Files ? 2170
Lines ? 164888
Branches ? 17786
================================================
Hits ? 142276
Misses ? 16289
Partials ? 6323 |
> cost_observability.json | ||
>>> | ||
runtime { | ||
docker: "gcr.io/google.com/cloudsdktool/cloud-sdk:390.0.0" |
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.
nit: any reason not to do latest
?
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.
We should have a team agreement on this - I would think we'd want to stick with known versions for repeatability?
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.
We had an issue in Batch when we used 'latest' and that upgraded to a version with a bug that broke our CI in a weird way
> cost_observability.json | ||
>>> | ||
runtime { | ||
docker: "gcr.io/google.com/cloudsdktool/cloud-sdk:390.0.0" |
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.
We should have a team agreement on this - I would think we'd want to stick with known versions for repeatability?
Reads the
cost_observability
table for the specifiedproject_id
/dataset_name
/call_set_identifier
and sums up the associated event bytes: