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 "disabled" property to deployments #14529

Open
11 tasks done
dylanbhughes opened this issue Jul 9, 2024 · 1 comment · May be fixed by #15129
Open
11 tasks done

Add "disabled" property to deployments #14529

dylanbhughes opened this issue Jul 9, 2024 · 1 comment · May be fixed by #15129
Assignees
Labels
3.x api Related the Prefect REST API enhancement An improvement of an existing feature feature A new feature

Comments

@dylanbhughes
Copy link
Contributor

dylanbhughes commented Jul 9, 2024

First check

  • I added a descriptive title to this issue.
  • I used the GitHub search to find a similar request and didn't find it.
  • I searched the Prefect documentation for this feature.

Prefect Version

3.x

Describe the current behavior

Currently deployments have paused but this impacts only schedules and their associated flow runs.,

Describe the proposed behavior

Add disabled property to deployment which disables all flow run creation mechanisms for that deployment.

  • disabled property on Deployment
  • New API routes for /disable and /enable
  • Calling create_flow_run for a disabled deployment returns an error
  • Automation triggers that would create flow runs for the deployment instead do nothing
  • When /disable is called, we delete the scheduled runs for the deployment
  • The scheduling mechanism does not schedule new runs for disabled deployments
  • Editing schedules for a disabled deployment returns an error Decided against implementing
  • Can't pause/unpause a disabled deployment

Example Use

No response

Additional context

For the Prefect team: https://www.notion.so/prefect/Enabling-Disabling-Deployments-9c482ed8fedb434cba8043a44e106069

@MaleicAcid
Copy link

really need this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x api Related the Prefect REST API enhancement An improvement of an existing feature feature A new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants