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

Add retry error handling for Experiment and AnalysisRun controller #227

Open
jessesuen opened this issue Oct 26, 2019 · 1 comment
Open

Comments

@jessesuen
Copy link
Member

jessesuen commented Oct 26, 2019

Both Experiment and AnalysisRun controller do not follow the pattern of returning errors from the context back to the controller, so that it can be retried. Instead, it prefers to mark errors into the status, so that they are surfaced as errors to the user. In some situations, retry is desired. Second, it does not leverage the benefits of the ratelimiting workqueue. This issue is to add that back and return errors back to the work queue when appropriate.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2022

This issue is stale because it has been open 60 days with no activity.

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

3 participants