Project

General

Profile

Bug #9157

Hard coded port for docker pull via capsule

Added by Partha Aji over 5 years ago. Updated 9 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Docker
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

The present katello foreman docker integration code assumes a hard coded port of 5000 for docker pull

https://github.com/parthaa/katello/commit/4fd7ec33c6d05e85edcf35316ffabbf5cff60c7f#diff-ec75feb699ba64886883dfdab8cca89eR30
Container_Extensions has this patch of code that makes this assumption

+ if self.capsule && Repository.where(:pulp_id => repository_name).count > 0
+ "#{URI.hostname}:5000/#{repo_url}"
+ else ..

History

#1 Updated by Eric Helms over 5 years ago

  • Target version set to 66
  • Legacy Backlogs Release (now unused) set to 23
  • Triaged changed from No to Yes

#2 Updated by Eric Helms over 5 years ago

  • Target version changed from 66 to 67

#3 Updated by Eric Helms about 5 years ago

  • Target version changed from 67 to 68

#4 Updated by Eric Helms about 5 years ago

  • Legacy Backlogs Release (now unused) changed from 23 to 31

#5 Updated by Eric Helms about 5 years ago

  • Target version deleted (68)

#6 Updated by Eric Helms almost 5 years ago

  • Legacy Backlogs Release (now unused) changed from 31 to 70

#7 Updated by Eric Helms over 4 years ago

  • Legacy Backlogs Release (now unused) changed from 70 to 86

#8 Updated by Eric Helms about 4 years ago

  • Legacy Backlogs Release (now unused) changed from 86 to 143

#9 Updated by Chris Duryee almost 4 years ago

Is it ok to assume 5000? It looks like it is baked in as 5000 in the installer puppet setup.

#10 Updated by Justin Sherrill almost 4 years ago

  • Legacy Backlogs Release (now unused) changed from 143 to 114

#11 Updated by John Mitsch 9 months ago

  • Status changed from New to Closed

Thanks for reporting this issue. This issue was created over 4 years ago and hasn't seen an update in 1 year. We are closing this in an effort to keep a realistic backlog. Please open up a new issue that includes a link to this issue if you feel this still needs to be addressed. We can then triage the new issue and reassess.

#12 Updated by Justin Sherrill 9 months ago

  • Target version deleted (Katello Backlog)

#13 Updated by Justin Sherrill 9 months ago

  • Status changed from Closed to Rejected

Also available in: Atom PDF