Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #5645

closed

Story #5517: [EPIC] Automation Hub Release Blockers

Need to return deprecated status in collection versions endpoint

Added by newswangerd over 4 years ago. Updated over 3 years ago.

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

Description

On the /pulp/api/v3/content/ansible/collection_versions/ endpoint add the following:

  • Add the deprecation status of the collection that each version is part of.
  • Add a ?deprecated=true|false param to the to filter out versions that are part of a deprecated collection. If ?deprecated=true versions should only be returned that are part of a deprecated collection. If ?deprecated=false versions that are not part of a deprecated collection should be returned. If the param is not set, all versions should be returned.
Actions
Copy link
 #1

Updated by daviddavis over 4 years ago

  • Groomed changed from No to Yes
  • Sprint set to Sprint 61
Actions
Copy link
 #2

Updated by bmbouter over 4 years ago

Part of the challenge is that at the model layer "depcrecation" is on Collection not CollectionVersion. We could have the viewset and serializer fulfill this, but I want to confirm you don't want this moved in the db also.

Also can't we use the filtering using _ notation to provide the filtering as /foo/bar/versions/?collection_deprecated=true|false ?

Actions
Copy link
 #3

Updated by newswangerd over 4 years ago

Part of the challenge is that at the model layer "depcrecation" is on Collection not CollectionVersion. We could have the viewset and serializer fulfill this, but I want to confirm you don't want this moved in the db also.

We don't need the deprecation status added to the database for collection versions.

Also can't we use the filtering using _ notation to provide the filtering as /foo/bar/versions/?collection_deprecated=true|false ?

&collection__deprecated=true returned this:

{
    "errors": [
        "Invalid Filter: 'collection__deprecated'"
    ]
}

Not sure if I'm doing something wrong or it's just not supported.

Added by daviddavis over 4 years ago

Revision 5ee426ef | View on GitHub

Expose collection deprecated and deprecated filter

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

Actions
Copy link
 #4

Updated by daviddavis over 4 years ago

  • Status changed from NEW to POST
  • Assignee set to daviddavis
Actions
Copy link
 #5

Updated by daviddavis over 4 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #6

Updated by bmbouter over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF