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.

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

first pass at updating Billing Project guidance #151

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

KatherineCox
Copy link
Collaborator

@KatherineCox KatherineCox commented Apr 25, 2022

Thought this would be fairly straightforward, just recommend they use a single Billing Project to start. But two snags:

  • It now makes more sense to swap sections 5 and 6, since Google Projects won't exist on GCP until Workspaces have been created, so there won't be anything to track spending for until they create Workspaces.
  • This means we need to update the overview figure.

So, here's a start at updating the prose, but I need to address those two issues before this is ready to go Done!

Those might be better addressed in a separate PR? Nah, that'll just make merging harder.

@github-actions
Copy link

github-actions bot commented Apr 25, 2022

No spelling errors! 🎉
Comment updated at 2022-06-08 with changes from 89bfbf6

@github-actions
Copy link

github-actions bot commented Apr 25, 2022

⚠️ There are broken URLs that need to be addressed. Read this guide for more info.
Download the errors here.
Comment updated at 2022-06-08 with changes from 89bfbf6

@github-actions
Copy link

github-actions bot commented Apr 25, 2022

Re-rendered previews from the latest commit:

Updated at 2022-06-08 with changes from 89bfbf6

@KatherineCox
Copy link
Collaborator Author

@avahoffman @cutsort

Before I started working on Billing videos, my original goal was just to update the Getting Started text to reflect the change to 1 Workspace: 1 Project.

Then I realized I needed to update the overview figure, which seemed like a good opportunity to also make a Billing short...which keeps growing into more videos, haha.

So, in the interest of making some incremental forward progress, here's PR that just updates the prose and overview figure but does not include any of the new videos. I'll add videos in other PR(s), once they're ready.

Major changes

  • now recommend they only set up one Terra Billing Project to start (keep it simple)
  • swap order of steps 5 and 6 (Set Alerts/View spend <-> Create Workspaces)
  • updated overview figure to reflect the reordering
  • (also updated the _child_google_billing_view_spend in Update _child_google_billing_view_spend.Rmd AnVIL_Template#118)

So, primarily take a look at PI steps 4, 5, and 6 (i.e. chapters 2.5-2.7)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant