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

[🐛 BUG]: Local activities run on workflow worker pool #1940

Closed
1 task done
Zylius opened this issue Jun 10, 2024 · 2 comments · Fixed by temporalio/roadrunner-temporal#527
Closed
1 task done
Assignees
Labels
B-bug Bug: bug, exception P-temporal Plugin: Temporal
Milestone

Comments

@Zylius
Copy link

Zylius commented Jun 10, 2024

No duplicates 🥲.

  • I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?

RoadRunner runs all local activities on workflow worker pool, instead of activity pool. This causes the configuration for temporal.activities.allocate_timeout to be ignored, and some local activities do eventually fail/timeout waiting for a free worker.

Version (rr --version)

2023.3.12

How to reproduce the issue?

Just launch a workflow with a local activity and watch only the workflow worker execution count rise.

Relevant log output

Workers of [temporal]:
+---------+-----------+---------+---------+---------+--------------------+
|   PID   |  STATUS   |  EXECS  | MEMORY  |  CPU%   |      CREATED       |
+---------+-----------+---------+---------+---------+--------------------+
|      43 | ready     |       0 | 122 MB  |    0.07 | 32 minutes ago     |
|      70 | ready     | 129,747 | 280 MB  |   38.44 | 32 minutes ago     |
+---------+-----------+---------+---------+---------+--------------------+
~/src $ 
@Zylius Zylius added B-bug Bug: bug, exception F-need-verification labels Jun 10, 2024
@rustatian rustatian added P-temporal Plugin: Temporal and removed F-need-verification labels Jun 10, 2024
@rustatian rustatian moved this to 🔖 Ready in Jira 😄 Jun 10, 2024
@rustatian rustatian added this to the v2024.1.4 milestone Jun 10, 2024
@rustatian
Copy link
Member

Hey @Zylius 👋
Thanks for the report. Will be fixed this Thursday in v2024.1.4.

@rustatian
Copy link
Member

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-bug Bug: bug, exception P-temporal Plugin: Temporal
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants