Project

Profile

Help

Issue #5533

closed

Pulp incorrectly namespaces Detail model endpoints if there is no master viewset

Added by daviddavis about 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 60
Quarter:

Description

Context: https://www.redhat.com/archives/pulp-dev/2019-October/msg00021.html

Suppose you have a viewset for a Detail model that doesn't inherit from the Master viewset (ContentViewset in this case):

class DistributionTreeViewSet(NamedModelViewSet,
                              mixins.RetrieveModelMixin,
                              mixins.ListModelMixin,
                              mixins.DestroyModelMixin):
    """
    Distribution Tree Viewset.
    """

    endpoint_name = 'distribution_trees'
    queryset = DistributionTree.objects.all()
    serializer_class = DistributionTreeSerializer

Pulp will automatically try to define the endpoint using master_endpoint (which doesn't exist), app_label, and endpoint_name. In this case, this ends up being "/pulp/api/v3/distribution_trees/rpm/distribution_trees/" which is probably not what the plugin writer wants.

Actions #1

Updated by daviddavis about 5 years ago

  • Subject changed from Pulp automatically namespaces Detail model endpoints to Pulp incorrectly namespaces Detail model endpoints if there is no master viewset
Actions #2

Updated by daviddavis about 5 years ago

  • Status changed from NEW to POST
  • Assignee set to daviddavis
Actions #3

Updated by daviddavis about 5 years ago

  • Sprint set to Sprint 60

Adding to sprint since the PoC is being merged.

Added by daviddavis about 5 years ago

Revision 0aa99adb | View on GitHub

Fix Detail viewsets erroneous namespacing

If there is no Master viewset, don't automatically try to namespace the viewset endpoint as it erroneously namespaces the endpoint with the detail viewset's endpoint_name.

fixes #5533 https://pulp.plan.io/issues/5533

Actions #4

Updated by daviddavis about 5 years ago

  • Status changed from POST to MODIFIED
Actions #5

Updated by bmbouter almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #6

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF