Skip to content

feat(worker): emit Temporal SDK metrics #990

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

Merged
merged 2 commits into from
Mar 26, 2025

Conversation

jvallesm
Copy link
Collaborator

Because

  • We want to observe Temporal metrics and scale workers according to them.

This commit

@jvallesm jvallesm self-assigned this Mar 25, 2025
Copy link

linear bot commented Mar 25, 2025

@jvallesm jvallesm marked this pull request as ready for review March 25, 2025 17:46
@jvallesm jvallesm requested a review from donch1989 as a code owner March 25, 2025 17:46
@jvallesm
Copy link
Collaborator Author

QA ✅

CleanShot 2025-03-25 at 18 46 09

@jvallesm jvallesm force-pushed the jvalles/ins-7396-emit-temporal-metrics-from-workers branch from 33266f6 to ba4e4f2 Compare March 25, 2025 17:49
@jvallesm jvallesm merged commit 1be2b44 into main Mar 26, 2025
10 of 11 checks passed
@jvallesm jvallesm deleted the jvalles/ins-7396-emit-temporal-metrics-from-workers branch March 26, 2025 08:43
jvallesm added a commit to instill-ai/instill-core that referenced this pull request Mar 27, 2025
Because

- `pipeline-backend` introduced config changes in
instill-ai/pipeline-backend/pull/990

This commit

- Updates the pipeline Helm charts
- Scrapes the Temporal metrics on docker-compose and exposes a Grafana
dashboard, following [this guide

](https://docs.temporal.io/cloud/metrics/prometheus-grafana#grafana-dashboards-setup)
jvallesm pushed a commit that referenced this pull request Mar 28, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.52.0-beta](v0.51.0-beta...v0.52.0-beta)
(2025-03-28)


### Features

* **memory:** remove recipe from in-process memory
([#984](#984))
([ca57ca8](ca57ca8))
* **streaming:** use Redis pub/sub to stream pipeline triggers
([#981](#981))
([80b7f17](80b7f17))
* **trigger:** use MinIO to store workflow memory
([#986](#986))
([fc36c6c](fc36c6c))
* **worker:** emit Temporal SDK metrics
([#990](#990))
([1be2b44](1be2b44))
* **worker:** extract worker back to its own process
([#987](#987))
([8925b0f](8925b0f))


### Bug Fixes

* **ci:** optimize disk usage in coverage worklfow
([#982](#982))
([f8f2707](f8f2707))
* **component,document:** fix the document cannot be converted to an
image
([#994](#994))
([22e879f](22e879f))
* **worker:** use pipeline server host in pipeline client
([#989](#989))
([06e2a9d](06e2a9d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants