Project

General

Profile

Actions

Bug #35459

closed

Non-enabled repository types make it into the apipie help-text

Added by Ian Ballou over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2043126

Description of problem:
Non-enabled repository types make it into the apipie help-text, such as "deb" or "python".

Version-Release number of selected component (if applicable):
Katello 4.2+

How reproducible:
Always.

Steps to Reproduce:
1. hammer repository create --help

Actual results:
--content-type ENUM Type of repo
Possible value(s): 'ansible_collection', 'deb', 'docker', 'file', 'ostree',
'python', 'yum'

Expected results:
Don't include types not supported by the smart proxy.

Additional info:
The issue is due to how the apipie cache is generated statically. Since it's not generated with a running Foreman, we cannot use the enabled repository types.

For a medium-term fix, we could remove the repo types completely from the apipie definition and add in an endpoint to check the enabled content types.

For a longer-term fix, we could switch to having the apipie cache generating dynamically in Foreman. That will require more consideration.

Actions #1

Updated by Ian Ballou over 1 year ago

  • Subject changed from Non-enabled repository types make it into the apipie help-text to Non-enabled repository types make it into the apipie help-text

API is GET /katello/api/repositories/repository_types

Actions #2

Updated by Ian Ballou over 1 year ago

  • Target version set to Katello 4.7.0
  • Triaged changed from No to Yes
Actions #3

Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ian Ballou
  • Pull request https://github.com/Katello/katello/pull/10259 added
Actions #4

Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/864 added
Actions #5

Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.7.0 added
Actions #6

Updated by Anonymous over 1 year ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF