Project

General

Profile

Bug #24115

katello allows setting invalid repo settings that pulp does not accept

Added by Chris Duryee 11 months ago. Updated 9 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 9 months ago

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

History

#1 Updated by Chris Duryee 11 months ago

  • Bugzilla link set to 1570068

#2 Updated by The Foreman Bot 10 months ago

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

#3 Updated by The Foreman Bot 10 months ago

  • Status changed from New to Ready For Testing

#4 Updated by John Mitsch 10 months ago

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

#5 Updated by The Foreman Bot 9 months ago

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

#6 Updated by Chris Duryee 9 months ago

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

#7 Updated by Christine Fouant 9 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF