Project

General

Profile

Bug #24115

katello allows setting invalid repo settings that pulp does not accept

Added by Chris Duryee about 1 year ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

If you create a yum repo and add "upstream_password" to the repo via hammer (hammer -u admin -p changeme repository update --id 1 --upstream-password "foo"), katello will tell pulp to create a task to update the importer and return immediately back to the user.

However, Pulp will give an error, and not update the importer:

"""
PulpDataException: Configuration errors:
The configuration parameter <basic_auth_password> requires the <basic_auth_username> parameter to also be set.
"""

Katello should return this error back to the user instead of assuming the task was successful. Once this happens, subsequent updates to the importer will fail until the original issue is corrected, since the incorrect value is now in katello_repositories.

Associated revisions

Revision 81db0bd5 (diff)
Added by Christine Fouant 12 months ago

fixes #24115, #24534 - validate upstream auth on repository

History

#1 Updated by Chris Duryee about 1 year ago

  • Bugzilla link set to 1570068

#2 Updated by The Foreman Bot about 1 year ago

  • Assignee set to Christine Fouant
  • Pull request https://github.com/Katello/katello/pull/7584 added

#3 Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing

#4 Updated by John Mitsch about 1 year ago

  • Triaged changed from No to Yes
  • Target version set to Katello 3.9.0

#5 Updated by The Foreman Bot 12 months ago

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

#6 Updated by Chris Duryee 12 months ago

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

#7 Updated by Christine Fouant 12 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF