Project

General

Profile

Bug #32362

katello pulp url setting in katello.yaml is no longer set by the installer and the default uses localhost rather than FQDN

Added by Ian Ballou about 1 year ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

As it turns out, Katello wasn't ready for this to be removed: https://github.com/theforeman/puppet-katello/commit/aa27779c4bf46442d5a5543c54e593798ce580af#diff-510884a7414666a0d988133c5f869510210d883aa1580d1248e74f1d16ca7d91L34

The default is no good any more and people see localhost for their repos' "Published At" url. Default: https://github.com/Katello/katello/blob/master/lib/katello/engine.rb#L40

This issue was brought up in a community thread: https://community.theforeman.org/t/katello-4-0-all-repos-are-localhost/23201

We need to change the default to use the machine's FQDN.

Associated revisions

Revision 1619e1c5 (diff)
Added by Ryan Verdile 12 months ago

Fixes #32362 - Change default pulp url to primary pulp smart proxy url (#9377)

Also moved full_path method from repo.rb to repository.rb.
Added test case to repository_test.rb
Removed pulp2 testcases

History

#1 Updated by Ian Ballou about 1 year ago

To fix this, we could default to use the primary pulp smart proxy's url.

#2 Updated by Samir Jha about 1 year ago

  • Triaged changed from No to Yes

#3 Updated by The Foreman Bot 12 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9377 added

#4 Updated by James Jeffers 12 months ago

  • Target version changed from Katello 4.0.1 to Katello 4.0.2

#5 Updated by The Foreman Bot 12 months ago

  • Fixed in Releases Katello 4.2.0 added

#6 Updated by Ryan Verdile 12 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by Justin Sherrill 11 months ago

  • Bugzilla link set to 1972381

#8 Updated by The Foreman Bot 10 months ago

  • Assignee set to James Jeffers

Also available in: Atom PDF