Project

Profile

Help

Refactor #2156

closed

Convert pulp_celerybeat to use a django lock model

Added by mhrivnak almost 8 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 currently uses a mongodb collection with a uniqueness constraint to obtain a lock. Multiple processes attempt to insert a document, and the one who succeeds is the active process.

Given the lock model that has already been created in django, convert pulp_celerybeat to use it instead of the mongo model.

It's important to test that these locks are acquired and released as expected. Hand testing this by manually invoking two pulp_celerybeats and testing failover would be good.


Related issues

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

Actions
Actions #1

Updated by mhrivnak almost 8 years ago

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

Updated by mhrivnak almost 8 years ago

  • Project changed from RPM Support to Pulp
Actions #3

Updated by mhrivnak almost 8 years ago

  • Tags Pulp 3 added
Actions #4

Updated by bmbouter almost 8 years ago

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

Updated by mhrivnak almost 8 years ago

Actions #6

Updated by mhrivnak almost 8 years ago

  • Sprint/Milestone set to 25
Actions #7

Updated by mhrivnak almost 8 years ago

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

Updated by dkliban@redhat.com almost 8 years ago

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

Updated by mhrivnak almost 8 years ago

  • Sprint/Milestone changed from 26 to 27

Added by fdobrovo almost 8 years ago

Revision 6e81d8d7 | View on GitHub

Convert pulp_resource_manager to use a django lock model

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

Actions #10

Updated by fdobrovo almost 8 years ago

  • Status changed from ASSIGNED to MODIFIED
  • % Done changed from 0 to 100
Actions #11

Updated by fdobrovo almost 8 years ago

  • Status changed from MODIFIED to ASSIGNED

I accidently have written a wrong number to commit https://github.com/pulp/pulp/commit/6e81d8d723144c586933192754b8d99fe8af3414 #2156 instead of #2157. So the commit belongs to #2157!! Sorry...

Actions #12

Updated by dkliban@redhat.com almost 8 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 #13

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from POST to MODIFIED
Actions #14

Updated by bmbouter over 6 years ago

  • Sprint set to Sprint 9
Actions #15

Updated by bmbouter over 6 years ago

  • Sprint/Milestone deleted (27)
Actions #16

Updated by daviddavis about 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #17

Updated by bmbouter about 5 years ago

  • Tags deleted (Pulp 3)
Actions #18

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF