Project

General

Profile

Actions

Bug #24115

closed

katello allows setting invalid repo settings that pulp does not accept

Added by Chris Duryee almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
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.

Actions #1

Updated by Chris Duryee almost 6 years ago

  • Bugzilla link set to 1570068
Actions #2

Updated by The Foreman Bot almost 6 years ago

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

Updated by The Foreman Bot almost 6 years ago

  • Status changed from New to Ready For Testing
Actions #4

Updated by John Mitsch almost 6 years ago

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

Updated by The Foreman Bot over 5 years ago

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

Updated by Chris Duryee over 5 years ago

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

Updated by Christine Fouant over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF