Project

Profile

Help

Refactor #88

Refactor #765: Convert Pulp to use MongoEngine

Convert the 'workers' collection to a mongoengine model

Added by mhrivnak almost 7 years ago. Updated 5 months ago.

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

100%

Estimated time:
Platform Release:
2.7.0
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
March 2015
Quarter:

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

Associated revisions

Revision 384ace02 View on GitHub
Added by cduryee over 6 years ago

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

Revision 384ace02 View on GitHub
Added by cduryee over 6 years ago

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

Revision f23f5ed2 View on GitHub
Added by ragabala about 3 years ago

Uploading RPM with large metadata and testing no exceptions raised

This commit verifies whether an rpm file constants.RPM_LARGE_METADATA can be uploaded into an rpm repo in pulp without any DocumentTooLarge error raised.

refer #723_ Closes #88

History

#1 Updated by rbarlow almost 7 years ago

  • Tracker changed from Task to Refactor

#2 Updated by bmbouter almost 7 years ago

  • Tags Sprint Candidate added

#3 Updated by bmbouter over 6 years ago

  • Subject changed from Convert another database model to use mongoengine to Convert the 'worker' collection to a mongoengine model

#4 Updated by bmbouter over 6 years ago

  • Description updated (diff)

#5 Updated by bmbouter over 6 years ago

  • Tags Groomed added
  • Tags deleted (Sprint Candidate)

#6 Updated by bmbouter over 6 years ago

  • Tags Sprint Candidate added

#7 Updated by cduryee over 6 years ago

  • Assignee set to cduryee

#8 Updated by cduryee over 6 years ago

  • Status changed from NEW to ASSIGNED

#9 Updated by bmbouter over 6 years ago

Maybe these conversion notes would be helpful.

https://fedorahosted.org/pulp/wiki/ConvertingModelsToMongoengine

#10 Updated by cduryee over 6 years ago

bmbouter wrote:

Maybe these conversion notes would be helpful.

https://fedorahosted.org/pulp/wiki/ConvertingModelsToMongoengine

It is helpful, thanks!

#11 Updated by bmbouter over 6 years ago

  • Description updated (diff)

#12 Updated by cduryee over 6 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.

#13 Updated by bmbouter over 6 years ago

  • Parent task set to #765

#14 Updated by bmbouter over 6 years ago

  • Subject changed from Convert the 'worker' collection to a mongoengine model to Convert the 'workers' collection to a mongoengine model

#15 Updated by cduryee over 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to cduryee

#16 Updated by cduryee over 6 years ago

  • Status changed from ASSIGNED to POST

oops, forgot to set to POST.

https://github.com/pulp/pulp/pull/1743

#17 Updated by mhrivnak over 6 years ago

  • Sprint/Milestone set to 14
  • % Done changed from 0 to 90

#18 Updated by cduryee over 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 90 to 100
  • Platform Release set to master

#20 Updated by rbarlow over 6 years ago

  • Platform Release changed from master to 2.7.0

#21 Updated by bmbouter over 6 years ago

  • Groomed set to Yes
  • Tags deleted (Groomed)

#22 Updated by bmbouter over 6 years ago

  • Sprint Candidate set to Yes
  • Tags deleted (Sprint Candidate)

#23 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5

#24 Updated by rbarlow almost 6 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#25 Updated by bmbouter over 3 years ago

  • Sprint set to March 2015

#26 Updated by bmbouter over 3 years ago

  • Sprint/Milestone deleted (14)

#27 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF