-
Notifications
You must be signed in to change notification settings - Fork 65
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
Add commands to generate cost of each GCP project #1947
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
1e0c54f
to
239c6e6
Compare
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.
I did a first pass and I've a few structural questions around this change in terms of where config should live, and more minor suggestions
for more information, see https://pre-commit.ci
This is confusing and not actually helpful for billing
This is done via JSON Schema now
I want to open an issue for tracking the 'single source of truth' about our contracts and which ones are cloud cost pass through, but I see it's already a line item in #2431 |
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.
Two minor comments but basically looks good, this is looking good and I've already been testing out the functionality.
deployer/billing.py
Outdated
""" | ||
match = re.match(r"(\d\d\d\d)-(\d\d)", month_str) | ||
if not match: | ||
raise typer.BadParameter(f"{month_str} should be formatted as YYYY-MM") |
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.
give an example should be formatted as YYYY-MM e.g. 2022-01
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.
Done
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.
lgtm
yay! |
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/4568225181 |
yay thank you for your review and encouragement in getting this done, @pnasrat! |
Fixes #1921
Things left to do: