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

Make exectutor report deploys to the scheduler #8189

Closed
1 of 3 tasks
wouterdb opened this issue Oct 9, 2024 · 1 comment
Closed
1 of 3 tasks

Make exectutor report deploys to the scheduler #8189

wouterdb opened this issue Oct 9, 2024 · 1 comment
Assignees

Comments

@wouterdb
Copy link
Contributor

wouterdb commented Oct 9, 2024

On deploy, the executor reports start and end to the server, we want to move this to the scheduler.

It should be feasible by:

  1. updating executor to not call deploy_start and deploy_done to the server, but do it in the surrounding Deploy task
  2. inline the deploy_start and deploy_done methods into the scheduler
  3. strip them to cover only the case at hand (when replacing in-db state with in-memory state, be mindful of recovery behavior)

May interact with #8187 and should not be done in parallel

@wouterdb wouterdb changed the title Make exectutor report to the scheduler Make exectutor report deploys to the scheduler Oct 9, 2024
@wouterdb
Copy link
Contributor Author

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

Successfully merging a pull request may close this issue.

1 participant