-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[usage] Store workspace, project, type & class #11460
Conversation
6e1af8b
to
54852da
Compare
CreditsUsed: 470, | ||
InstanceID: instanceId, | ||
AttributionID: teamAttributionID, | ||
UserID: ownerID, |
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.
What would be the ownerID here for a prebuild and do we expect there to be one?
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.
SELECT wsi.id, ws.type, ws.ownerId from d_b_workspace as ws, d_b_workspace_instance as wsi where wsi.workspaceId = ws.id AND ws.type = 'prebuild' LIMIT 100;
https://console.cloud.google.com/bigquery?sq=441577903635:724ea3fda1724aab80bae2f7babc5758
Prebuilds have owners. Sadly they are the user who setup the prebuild but oh well.
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.
the user who setup the prebuild
I guess for the billing session this information will be irrelevant, but fine to keep here in case for future purposes.
LGTM |
Description
Related Issue(s)
Fixes #
How to test
Unit tests
Release Notes
Documentation
Werft options: