Project

Profile

Help

Task #2907

Remove plugin API objects that are not used by core

Added by bmbouter over 2 years ago. Updated 6 months ago.

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

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 23

Description

The plugin API has several placeholder objects that are not used by the plugin. It's very possible these objects won't be in the GA release even based on the MVP. Specifically these objects:

Cataloger
Profiler

We also need to remove almost all or all of the serializers and viewsets which are currently mass-imported into pulpcore.plugin.

Only bring in viewsets and serializers that the plugin writer is intended to subclass. That would involve changing these two imports:

viewsets.py
serializers.py

Those two modules should only importer the following objects by name (not with *):

pulpcore.plugin.serializers.ContentSerializer
pulpcore.plugin.serializers.ImporterSerializer
pulpcore.plugin.serializers.PublisherSerializer
pulpcore.plugin.viewsets.ContentViewSet
pulpcore.plugin.viewsets.ImporterViewSet
pulpcore.plugin.viewsets.PublisherViewSet

History

#1 Updated by dkliban@redhat.com over 2 years ago

dkliban will groom

#2 Updated by amacdona@redhat.com over 2 years ago

We will still need to import master ViewSets and Serializers that the plugins inherit from.

At a minimum, we need to keep:

  • pulpcore.plugin.serializers.ContentSerializer
  • pulpcore.plugin.serializers.ImporterSerializer
  • pulpcore.plugin.serializers.PublisherSerializer
  • pulpcore.plugin.viewsets.ContentViewSet
  • pulpcore.plugin.viewsets.ImporterViewSet
  • pulpcore.plugin.viewsets.PublisherViewSet

#3 Updated by bmbouter over 2 years ago

  • Description updated (diff)

@asmacdo, I agree. I also think those are the base minimum set so that plugin writers can inherit from them.

#4 Updated by dkliban@redhat.com over 2 years ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

#5 Updated by mhrivnak about 2 years ago

  • Priority changed from Normal to High

#6 Updated by mhrivnak about 2 years ago

  • Sprint/Milestone set to 42

#7 Updated by bizhang about 2 years ago

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

#8 Updated by bizhang about 2 years ago

  • Status changed from ASSIGNED to POST

#9 Updated by werwty about 2 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#10 Updated by bmbouter almost 2 years ago

  • Tags deleted (Pulp 3 Plugin Writer Alpha)

Cleaning up Redmine tags

#11 Updated by bmbouter over 1 year ago

  • Sprint set to Sprint 23

#12 Updated by bmbouter over 1 year ago

  • Sprint/Milestone deleted (42)

#13 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#14 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF