Refactor #765: Convert Pulp to use MongoEngine
Document that migrations should use PyMongo directly and never use model code
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.
#1 Updated by bmbouter about 6 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.
#5 Updated by bmbouter over 2 years 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.
Please register to edit this issue