Skip to content

Add cost governance page to best practices - #5145

Merged
Duncanma merged 6 commits into
mainfrom
docs/cost-governance-best-practices
Aug 19, 2026
Merged

Add cost governance page to best practices#5145
Duncanma merged 6 commits into
mainfrom
docs/cost-governance-best-practices

Conversation

@Duncanma

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new best-practices guide, "Cost governance on Temporal Cloud," covering cost attribution via Namespace tagging, budget forecast/burn-rate tracking, and anomaly detection on usage — built on the Billing API, Billing Center, Usage Dashboards, and OpenMetrics.
  • Content is adapted from a draft written by a Temporal cost/pricing expert, restructured for house style (headings, admonitions, tables, internal linking) without changing the substance of the guidance.
  • Registers the page in sidebars.js and links it from the best-practices index.

Opened as a draft for docs-team review of tone/placement before merging.

Test plan

  • vale --config .vale-ci.ini docs/best-practices/cost-governance.mdx docs/best-practices/index.mdx — 0 issues
  • yarn build — succeeds
  • yarn check-links — no new failures introduced by this page (remaining findings are pre-existing checker artifacts affecting every best-practices page: trailing-slash fragment-redirect noise, and live-site/GitHub-source 404s expected for any unmerged page)
  • yarn check:orphans — clean
  • Verified rendering locally (tables, JSON code block, admonition all render correctly)

Publishes a new best-practices guide covering cost attribution via
Namespace tagging, budget forecast/burn-rate tracking, and anomaly
detection on usage, built on the Billing API and OpenMetrics.
Copilot AI balanced review requested due to automatic review settings August 19, 2026 17:43
@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview Aug 19, 2026 10:21pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds cost-governance guidance to Temporal Cloud best practices.

Changes:

  • Documents cost attribution, forecasting, and anomaly detection.
  • Adds the guide to navigation and the best-practices index.
  • Clarifies which practices are Cloud-specific.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
sidebars.js Adds the guide to navigation.
docs/best-practices/index.mdx Links and categorizes the guide.
docs/best-practices/cost-governance.mdx Provides the cost-governance framework.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/best-practices/cost-governance.mdx Outdated
Comment thread docs/best-practices/cost-governance.mdx Outdated
Comment thread docs/best-practices/cost-governance.mdx Outdated
@github-actions

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
looks ok

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Duncanma
❌ tlotemporal
You have signed the CLA already but the status is still pending? Let us recheck it.

looks ok

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Duncanma
Duncanma marked this pull request as ready for review August 19, 2026 20:20
@Duncanma
Duncanma requested a review from a team as a code owner August 19, 2026 20:20
| [Actions in Event History](/cloud/actions-usage#actions-in-workflows) | Per Workflow Execution | Estimating Actions for a specific execution | Account Owners, Global Admins, Namespace Admins, Developers, and Read-Only users |
| [Action metric](/cloud/metrics/openmetrics/metrics-reference#temporal_cloud_v1_billable_action_count) | 1-minute, by Workflow and Action type | Near-real-time usage behavior, alerting, debugging | Service account with the Metrics Read-Only role |

### Actions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to have this section when we already have a much more detailed explanation linked /cloud/actions

I do feel like best practice documents should assume a basic knowledge of what an action is an doesn't need these conceptual sections

@Duncanma Duncanma Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to leave this one as-is for now — thinking this is a common point of confusion so Tlo likely included the repetition here on purpose, even with /cloud/actions one click away. Open to revisiting if it still feels redundant on another pass.

Comment thread docs/best-practices/cost-governance.mdx Outdated
Comment thread docs/best-practices/cost-governance.mdx Outdated
Drops the "where did this bill come from" framing in favor of a more
neutral statement of the guide's goal, and renames the three use-case
headings to state the recommendation directly, matching the verb-based
heading style used elsewhere in best-practices.
@Duncanma
Duncanma merged commit 845270e into main Aug 19, 2026
14 of 15 checks passed
@Duncanma
Duncanma deleted the docs/cost-governance-best-practices branch August 19, 2026 23:17
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.

5 participants