Project

Profile

Help

Task #1220

Refactor #765: Convert Pulp to use MongoEngine

Document that migrations should use PyMongo directly and never use model code

Added by bmbouter about 5 years ago. Updated over 1 year ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Documentation, Pulp 2
Sprint:
Quarter:

Description

Using any kind of model (mongoengine or not) in a migration is problematic. Models change over time, but migrations should always remain the same. This task is to document that migrations should use PyMongo directly and never use a model of any kind.

In addition to the maintainability problem, mongoengine models don't even work as expected because the save() handler won't work as it does in other parts of the code. Normally save() moves files into place and in the puppet case calculates checksums, but at pulp-manage-db time none of those things work because the PluginManager isn't instantiated prior to the models running.

History

#1 Updated by bmbouter about 5 years ago

  • Tracker changed from Issue to Task
  • Subject changed from mongoengine model pre_save signals don't work in pulp-manage-db to Document that migrations should use PyMongo directly and never use model code
  • Description updated (diff)
  • Groomed set to No
  • Sprint Candidate set to Yes
  • Tags Documentation added

Rewritten to get more to the point that migrations should never use models and we should document this.

#2 Updated by bmbouter about 5 years ago

  • Description updated (diff)

#3 Updated by mhrivnak over 4 years ago

  • Sprint Candidate changed from Yes to No

#4 Updated by bmbouter over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

#5 Updated by bmbouter over 1 year ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#6 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF