Project

General

Profile

Bug #34184

Generic content units don't have the endpoint to filter out CV or CV-Version specific content

Added by Samir Jha 5 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Samir Jha
Category:
Content Views
Target version:
Katello 4.4.0
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/9868, https://github.com/Katello/katello/pull/9870, https://github.com/Katello/katello/pull/9881
Fixed in Releases:
Katello 4.4.0
Found in Releases:
Red Hat JIRA:

Description

Looking at the code, seems like the repository_content_controller L83 filter_by_content_view_version logic needs to be expanded to handle generic content. The API currently just throws an error with generic_content_units is not a function.

Note the functions packages, ansible_collections, docker_manifests etc in katello/app/models/katello/content_view_version.rb that help filter out the content units in specific repositories.

Associated revisions

Revision efb05fd6 (diff)
Added by Justin Sherrill 5 months ago

Fixes #34184 - Add content_view_version filter to generic content units (#9881)

  • Fixes #34184 - Add content_view_version filter to generic content units
  • Refs #34184 - add a test for generic filtering by cvv

Co-authored-by: Samir Jha <>

History

#1 Updated by The Foreman Bot 5 months ago

  • Assignee set to Samir Jha
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9870 added

#2 Updated by Jonathon Turel 5 months ago

  • Triaged changed from No to Yes
  • Target version changed from Katello 4.3.1 to Katello 4.4.0

#3 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/katello/pull/9868 added

#4 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/katello/pull/9881 added

#5 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases Katello 4.4.0 added

#6 Updated by Justin Sherrill 5 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/Katello/katello/pull/9899 added

#8 Updated by Ian Ballou 3 months ago

  • Category set to Content Views

#9 Updated by Ian Ballou 3 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/9899)

Also available in: Atom PDF