Issue #6943: New Pulp Project fixutres site is incorrectly setup for python plugin - Pulp

Actions

Send by e-mail Copy link

Issue #6943

closed

Infrastructure - Task #6638: [Epic] Host fixtures docker container at https://fixtures.pulpproject.org

New Pulp Project fixutres site is incorrectly setup for python plugin

Added by gerrod almost 4 years ago. Updated almost 4 years ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

daviddavis

Category:

Sprint/Milestone:

Start date:

Due date:

Estimated time:

Severity:

2. Medium

Version:

Platform Release:

OS:

Triaged:

Yes

Groomed:

Sprint Candidate:

Tags:

Sprint:

Sprint 75

Quarter:

Description

The links for the python packages in the /python-pypi/simple/ directory are broken. They link to localhost:8000/ instead of the hostname of the site: fixtures.pulpproject.org/ . Python tests rely on the new fixture site and thus break due to this error.

<?xml version='1.0' encoding='utf8'?>
<html>
<head>
    <title>Links for shelf-reader</title>
    <meta name="api-version" value="2" />
</head>
<body>


  <a href="http://localhost:8000/fixtures/python-pypi/packages/shelf_reader-0.1-py2-none-any.whl" rel="internal">shelf_reader-0.1-py2-none-any.whl</a><br />


  <a href="http://localhost:8000/fixtures/python-pypi/packages/shelf-reader-0.1.tar.gz" rel="internal">shelf-reader-0.1.tar.gz</a><br />

</body>
</html>

Actions

Copy link

Updated by daviddavis almost 4 years ago

Parent issue set to #6638

Actions

Copy link

Updated by gerrod almost 4 years ago

Discussion from IRC:

<daviddavis> gerrod: I saw this. we'll probably have to have the container accept a parameter at runtime for host.
<gerrod> daviddavis, i'm confused, why are the links set to localhost instead of the where the packages are on the site?
<daviddavis> gerrod: the links get generated when the docker container is built. see here https://git.io/Jfy9w and here https://git.io/Jfy9r
<daviddavis> gerrod: fixtures.pulpproject.org simply takes the pre-built container which is hardcoded to use localhost:8000 and runs it
<daviddavis> gerrod: I think the solution is to have a variable that can be specified at container runtime
<daviddavis> I'm not a container expert and so I'm unsure how it'll work. we have a CI meeting tomorrow though and I will raise the issue.

Actions

Copy link