Skip to content

Feature: Scheduler pipeline management: Pause/Resume#30

Merged
jaseemjaskp merged 3 commits into
mainfrom
feature/Pause-Scheduled-Pipeline
Mar 1, 2024
Merged

Feature: Scheduler pipeline management: Pause/Resume#30
jaseemjaskp merged 3 commits into
mainfrom
feature/Pause-Scheduled-Pipeline

Conversation

@athul-rs

Copy link
Copy Markdown
Contributor

What

... Pipeline feature to Paused and Resume.

Why

... Earlier the button functionality was not implemented.

How

... Changing pipeline status as well as celery beat status for the pipeline.

Relevant Docs

Related Issues or PRs

Dependencies Versions / ENV Variables

Notes on Testing

...

Screenshots

... Not applicable here, since UI part was already implemented.

Checklist

I have read and understood the Contribution Guidelines.

Comment thread backend/pipeline/models.py
Comment thread backend/pipeline/views.py Outdated

@muhammad-ali-e muhammad-ali-e left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jaseemjaskp
jaseemjaskp merged commit 7aec6c5 into main Mar 1, 2024
@jaseemjaskp
jaseemjaskp deleted the feature/Pause-Scheduled-Pipeline branch March 1, 2024 05:37
task = PeriodicTask.objects.get(name=task_name)
task.enabled = False
task.save()
update_pipeline(task_name, Pipeline.PipelineStatus.PAUSED, False)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: @athul-rs using named kwargs is much more explicit when passing around bools - in this case we'll have to inspect the function to know the behaviour

pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
* Feature: Scheduler pipeline management: Pause/Resume

* Update serializer

---------

Co-authored-by: Jaseem Jas <89440144+jaseemjaskp@users.noreply.github.com>
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 this pull request may close these issues.

4 participants