Task #5693
Whitelist the specific objects in pulpcore.plugin.__init__
100%
Description
The plugin API is a little unclear given that there are many objects actually importable from pulpcore.app.plugin, but we only intend for Plugin writers to use a specific set of them.
These should be named in an all method or pulpcore.plugin.__init__ somehow.
Associated revisions
Revision 2888a025
View on GitHub
Use BaseModel
instead of Model
pulpcore changed the name of Model
to BaseModel
. This PR updates
pulp_ansible to do the same.
Required PR: https://github.com/pulp/pulpcore/pull/421
Revision 5ff9778e
View on GitHub
Remove unused pulpcore.plugin objects
This was done by an audit against master for pulp_rpm, pulp_container, pulp_file, pulp_deb, pulp-certguard, and pulp_cookbook.
pulp_container will require a small update replacing its CharInFilter usage.
Revision 5ccc056e
View on GitHub
Use BaseModel
instead of Model
pulpcore changed the name of Model
to BaseModel
. This PR updates
pulp-2to3-migration to do the same.
Required PR: https://github.com/pulp/pulpcore/pull/421
Revision 5ccc056e
View on GitHub
Use BaseModel
instead of Model
pulpcore changed the name of Model
to BaseModel
. This PR updates
pulp-2to3-migration to do the same.
Required PR: https://github.com/pulp/pulpcore/pull/421
Revision 5ccc056e
View on GitHub
Use BaseModel
instead of Model
pulpcore changed the name of Model
to BaseModel
. This PR updates
pulp-2to3-migration to do the same.
Required PR: https://github.com/pulp/pulpcore/pull/421
Revision 8c36d2f9
View on GitHub
Remove unused pulpcore.plugin objects
This was done by an audit against master for pulp_rpm, pulp_container, pulp_file, pulp_deb, pulp-certguard, and pulp_cookbook.
pulp_container will require a small update replacing its CharInFilter usage.
https://pulp.plan.io/issues/5693 closes #5693
(cherry picked from commit 5ff9778ecdb18d8622f40b68a6a4ed0e21b6b53a)
Revision 153729a5
View on GitHub
Use BaseModel
instead of Model
pulpcore changed the name of Model
to BaseModel
. This PR updated
pulp_rpm to do the same.
Required PR: https://github.com/pulp/pulpcore/pull/421
https://pulp.plan.io/issues/5693 closes #5693
(cherry picked from commit 6692804c4fcfc09a04d966c2b32a66033cb9daeb)
History
#1
Updated by bmbouter about 1 year ago
- Status changed from NEW to ASSIGNED
- Assignee set to bmbouter
#2
Updated by bmbouter about 1 year ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/pulpcore/pull/421
#3
Updated by bmbouter about 1 year ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulpcore|5ff9778ecdb18d8622f40b68a6a4ed0e21b6b53a.
#4
Updated by bmbouter about 1 year ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Please register to edit this issue
Use
BaseModel
instead ofModel
pulpcore changed the name of
Model
toBaseModel
. This PR updated pulp_rpm to do the same.Required PR: https://github.com/pulp/pulpcore/pull/421
https://pulp.plan.io/issues/5693 closes #5693