Project

Profile

Help

Refactor #2158

closed

Remove pulp_celerybeat use of scheduled calls from database

Added by mhrivnak over 7 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 9
Quarter:

Description

pulp_celerybeat (2.Y) currently loads scheduled calls from mongodb. The 3.0 MVP will not load any scheduled tasks from the database. The code that handles the loading of schedule from DB should be removed.


Related issues

Related to Pulp - Issue #1641: unit.server.db.model.test_dispatch.TestScheduledCallSave.test_new is non-deterministicCLOSED - WONTFIXActions
Related to Pulp - Task #1978: Change the name of the scheduler worker from "scheduler" to "pulp_celerybeat"CLOSED - CURRENTRELEASE

Actions
Related to Pulp - Task #2176: Remove auto-disable feature of scheduled tasksCLOSED - CURRENTRELEASEdkliban@redhat.com

Actions
Blocked by Pulp - Refactor #2087: create django models for task systemCLOSED - CURRENTRELEASEpcreech

Actions
Actions #1

Updated by mhrivnak over 7 years ago

  • Blocked by Refactor #2087: create django models for task system added
Actions #2

Updated by mhrivnak over 7 years ago

  • Related to Issue #1641: unit.server.db.model.test_dispatch.TestScheduledCallSave.test_new is non-deterministic added
Actions #3

Updated by bmbouter over 7 years ago

  • Description updated (diff)
  • Groomed changed from No to Yes
Actions #4

Updated by bmbouter over 7 years ago

  • Description updated (diff)
Actions #5

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone set to 25
Actions #6

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone changed from 25 to 26
Actions #7

Updated by bmbouter over 7 years ago

This PR would be merged after the work I am doing on my branch. I've left the scheduler.py and worker_watcher.py untouched[0]. Those should be moved to pulp.tasking here[1].

Also this story does not mention that the pulp_celerybeat needs to record the workers correctly in addition to handling scheduled calls. If that is supposed to be a separate story please file one.

[0]: https://github.com/bmbouter/pulp/tree/2154-Belgian-Dubbel/server/pulp/server/async
[1]: https://github.com/bmbouter/pulp/tree/2154-Belgian-Dubbel/tasking/pulp/tasking

Actions #8

Updated by bmbouter over 7 years ago

  • Related to Task #1978: Change the name of the scheduler worker from "scheduler" to "pulp_celerybeat" added
Actions #9

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions #10

Updated by mhrivnak over 7 years ago

  • Related to Task #2176: Remove auto-disable feature of scheduled tasks added
Actions #11

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone changed from 26 to 27
Actions #12

Updated by mhrivnak over 7 years ago

Per PulpCon planning discussion, the MVP does not include the ability for a user to schedule tasks. We should revise this "refactor" issue accordingly.

Actions #13

Updated by dkliban@redhat.com over 7 years ago

  • Subject changed from Convert pulp_celerybeat to use scheduled calls from django to Remove pulp_celerybeat use of scheduled calls from database
  • Description updated (diff)

Updated the description and title to reflect the fact that 3.0 MVP will not execute any tasks using schedule data in the database.

Actions #14

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from ASSIGNED to POST

Added by dkliban@redhat.com over 7 years ago

Revision 365f9fd0 | View on GitHub

Removes database backed celerybeat schedule

This change also converts worker watcher to use Django. This patch renames the scheduler to pulp_celerybeat.

closes: #2158 https://pulp.plan.io/issues/2158 closes: #2156 https://pulp.plan.io/issues/2156 closes: #1978 https://pulp.plan.io/issues/1978

blah

Added by dkliban@redhat.com over 7 years ago

Revision 365f9fd0 | View on GitHub

Removes database backed celerybeat schedule

This change also converts worker watcher to use Django. This patch renames the scheduler to pulp_celerybeat.

closes: #2158 https://pulp.plan.io/issues/2158 closes: #2156 https://pulp.plan.io/issues/2156 closes: #1978 https://pulp.plan.io/issues/1978

blah

Actions #15

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #16

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 9
Actions #17

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (27)
Actions #18

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #19

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #20

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF