Refactor #88
closed
Refactor #765: Convert Pulp to use MongoEngine
Convert the 'workers' collection to a mongoengine model
100%
Description
Deliverables:
- Convert the model according to the conversion guide
- Verify that the user facing listing in /status/ works the same as it did before the switch to mongoengine
Updated by bmbouter over 9 years ago
- Subject changed from Convert another database model to use mongoengine to Convert the 'worker' collection to a mongoengine model
Updated by bmbouter over 9 years ago
- Tags Groomed added
- Tags deleted (
Sprint Candidate)
Updated by bmbouter over 9 years ago
Maybe these conversion notes would be helpful.
https://fedorahosted.org/pulp/wiki/ConvertingModelsToMongoengine
Updated by cduryee over 9 years ago
bmbouter wrote:
Maybe these conversion notes would be helpful.
https://fedorahosted.org/pulp/wiki/ConvertingModelsToMongoengine
It is helpful, thanks!
Updated by cduryee over 9 years ago
- Status changed from ASSIGNED to NEW
- Assignee deleted (
cduryee)
Unassigning issue from myself for the moment. I will pick it back up once I'm done with a higher sev issue.
Updated by bmbouter over 9 years ago
- Subject changed from Convert the 'worker' collection to a mongoengine model to Convert the 'workers' collection to a mongoengine model
Updated by cduryee over 9 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to cduryee
Updated by cduryee over 9 years ago
- Status changed from ASSIGNED to POST
oops, forgot to set to POST.
Added by cduryee over 9 years ago
Added by cduryee over 9 years ago
Revision 384ace02 | View on GitHub
convert "workers" collection to mongoengine
This patch converts the workers collection to mongoengine. It also moves some
functionality into tasks.py that previously lived in the worker manager.
One thing to note, it appears that the existing workers collection did not have
a _ns field. Thus, I did not add this into the new collection.
This commit includes a fix to CriteriaQuerySet's update() method which was
incorrectly firing the post_save() hook on TaskStatus objects, even when the
Document was not a task status.
A spec file requirement for python-blinker was also added to support
mongoengine signals. Additionally, a small unrelated flake8 fixup to
test_users.py was added.
fixes #88
Updated by mhrivnak over 9 years ago
- Sprint/Milestone set to 14
- % Done changed from 0 to 90
Updated by cduryee over 9 years ago
- Status changed from POST to MODIFIED
- % Done changed from 90 to 100
- Platform Release set to master
Updated by cduryee over 9 years ago
Applied in changeset pulp|384ace02340dd4d30cfafe30bd598be763071695.
Updated by rbarlow over 9 years ago
- Platform Release changed from master to 2.7.0
Updated by bmbouter over 9 years ago
- Sprint Candidate set to Yes
- Tags deleted (
Sprint Candidate)
Updated by dkliban@redhat.com over 9 years ago
- Status changed from MODIFIED to 5
Updated by rbarlow almost 9 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
.
convert "workers" collection to mongoengine
This patch converts the
workerscollection to mongoengine. It also moves some functionality into tasks.py that previously lived in the worker manager.One thing to note, it appears that the existing workers collection did not have a
_nsfield. Thus, I did not add this into the new collection.This commit includes a fix to CriteriaQuerySet's
update()method which was incorrectly firing thepost_save()hook on TaskStatus objects, even when the Document was not a task status.A spec file requirement for
python-blinkerwas also added to support mongoengine signals. Additionally, a small unrelated flake8 fixup to test_users.py was added.fixes #88