Refactor #2156
closed
Convert pulp_celerybeat to use a django lock model
100%
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
Updated by mhrivnak over 7 years ago
- Blocked by Refactor #2087: create django models for task system added
Updated by bmbouter over 7 years ago
- Description updated (diff)
- Groomed changed from No to Yes
Updated by dkliban@redhat.com over 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to dkliban@redhat.com
Added by fdobrovo over 7 years ago
Updated by fdobrovo over 7 years ago
- Status changed from ASSIGNED to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|6e81d8d723144c586933192754b8d99fe8af3414.
Updated by fdobrovo over 7 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...
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
Updated by dkliban@redhat.com over 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|365f9fd0a0b09969e65ef6f112a7a973e2bb63cc.
Updated by bmbouter over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Convert pulp_resource_manager to use a django lock model
closes #2156 https://pulp.plan.io/issues/2156