Document Temporal Cloud Schedule metrics - #5134
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📖 Docs PR preview links
|
| Each metric has its own set of applicable labels. See the Metrics Reference for complete details. | ||
|
|
||
| ### Metric Family | ||
| ### Metric family |
There was a problem hiding this comment.
family is lowercase?
There was a problem hiding this comment.
yeah that's a change to match the docs style guide
| #### temporal\_cloud\_v1\_schedule\_action\_success\_count | ||
|
|
||
| Successfully executed scheduled workflows per second. | ||
| Successful scheduled executions per second. |
There was a problem hiding this comment.
The source is from https://github.com/temporalio/temporal/blob/767162845ef95cb5e4f7157fb980f9c5ce4cbf12/common/metrics/metric_defs.go#L1530, in my opinion this sort of implies successful scheduled actions but its more successful start workflow executions. might be minor and not matter.
There was a problem hiding this comment.
| Successful scheduled executions per second. | |
| Successful scheduled start Workflow executions per second. |
@kepe-temporal what do you think of this?
There was a problem hiding this comment.
Sorry, after thinking about it more, I would like to retract my comment and have it be more general, I think Successful scheduled executions per second. is better.
Summary
Validation
vale --config .vale-ci.ini docs/cloud/metrics/openmetrics/api-reference.mdx docs/cloud/metrics/openmetrics/metrics-reference.mdxpasses with zero findingsgit diff --checkpasses┆Attachments: EDU-6981 Document Temporal Cloud Schedule metrics