[CORE-53] Refactor storage cost estimate to use Rawls #1486
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rawls now supports getting workspace bucket location, so Orchestration doesn't have to directly request this information from Google, which was causing incorrect behavior around requester pays. This change modifies the storage cost estimate API so that it uses Rawls to get bucket location (for rate calculations). It also adds an optional
userProject
query parameter so that a reader may use the API to get a storage cost estimate on a requester pays workspace if they provide a project to bill.For UI users, this will fix the error in getting estimated cost on requester pays workspaces for writers and owners. It won't look any different for readers, because the UI hides estimated cost from readers in the first place.
Have you read CONTRIBUTING.md lately? If not, do that first.
I, the developer opening this PR, do solemnly pinky swear that:
In all cases: