pulp.server.db.model should be moved to pulp.server.db.models
This should happen for two reasons:
1. All of our models live there, it should be a plural module name.
2. pulp.server.db.model is currently a package acting as a module. By importing any of the modules in this package, we are forced to import everything that is in model/__init__.py. This creates a precarious situation for circular imports.
The second issue could also be fixed by moving the modules out of the model package and turning model into a real module. Either way, this should happen very soon because we will be updating all the plugins to use the models directly, and we want to make this change before it will break the plugins.
Updated by firstname.lastname@example.org over 8 years ago
There is at least one (extremely minor) circular import issue caused by this. The fix in that case is to add an extra import that ensures a feet first import. When this is solved, please also remove the extra import in webservices/views/serializers/test_serializers.py. See the note there for more details.
Updated by bmbouter over 4 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.