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

Inform the User when temporal quarantines a job #13452

Closed
2 tasks
supertopher opened this issue Jun 3, 2022 · 2 comments
Closed
2 tasks

Inform the User when temporal quarantines a job #13452

supertopher opened this issue Jun 3, 2022 · 2 comments

Comments

@supertopher
Copy link
Contributor

Wut

Temporal has a quarantine state. This state is to reduce overloading the system with a large pile of failed requests. It's a good idea, we want this to happen.

Temporal will attempt to run a job 10 times. If temporal does not receive an ACK for the start of the job for all 10 retries that job goes into quarantine.

From the user's perspective this can look like a single sync attempt was fired and that attempt in still in progress.

To unstick the job, an engineer must run tctl

How to do this

¯_(ツ)_/¯

TLDR you figure it out

We could have a time based check from airbyte to check with temporal on current job status and inform the user for non-running state

We could check state on every page reload and avoid the whole timing thing

Some way smarter way you just though of while reading my silly suggestions

You will know you are done when

  • Users are aware their jobs are not running in the UI when temporal knows they are not running due to quarantine state
  • Our database reflects that jobs temporal knows are in a quarantine state are in fact quarantined
@supertopher
Copy link
Contributor Author

@benmoriceau @lmossman please edit this or comment if my understanding of the world here is incorrect or you see a better way

@davinchia
Copy link
Contributor

We no longer quarantine jobs as of #13779.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants