Project

Profile

Help

Issue #6697

closed

pulp_installer doesn't add a newline char in requirements.in

Added by dkliban@redhat.com over 4 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
3.3.1
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 73
Quarter:

Description

A user on pulp-list reported the following during upgrade to 3.3.1:

After remove entire pulp installation directory. Now I am getting an error

fatal: [pulp3-2]: FAILED! => {"changed": false, "cmd": ["/opt/utils/venv/pulp/3.7.3/bin/pip-compile"], "delta": "0:00:00.778728", "end": "2020-05-08 17:26:32.680701", "failed_when_result": true, "msg": "non-zero return code", "rc": 2, "start": "2020-05-08 17:26:31.901973", "stderr": "Could not find a version that matches pulp-rpm-clientpulp-file-client (from -r requirements.in (line 6))\nNo versions found\nWas https://artifactory.inf.bloomberg.com/artifactory/api/pypi/bloomberg-pypi-ose/simple reachable?", "stderr_lines": ["Could not find a version that matches pulp-rpm-clientpulp-file-client (from -r requirements.in (line 6))", "No versions found", "Was https://artifactory.inf.bloomberg.com/artifactory/api/pypi/bloomberg-pypi-ose/simple reachable?"], "stdout": "", "stdout_lines": []}

It looks like this file is wrong
# more /opt/utils/venv/pulp/3.7.3/requirements.in
pulpcore==3.3.1
pulp-file==0.3.0
pulp-rpm==3.3.1
# Any plugins listed below were already installed but not specified in
# pulp_install_plugins
pulp-rpm-clientpulp-file-client

Actions #1

Updated by dkliban@redhat.com over 4 years ago

  • Version set to 3.3.1
  • Tags Pulp 3 installer added
Actions #2

Updated by fao89 over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 72
Actions #3

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions #4

Updated by dkliban@redhat.com over 4 years ago

We most likely need to remove the '-' from this line here[0].

[0] https://github.com/pulp/pulp_installer/blob/master/roles/pulp/templates/requirements.in.j2#L11

Actions #5

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (dkliban@redhat.com)
Actions #6

Updated by bmbouter over 4 years ago

  • Category set to Installer - Moved to GitHub issues
  • Tags deleted (Pulp 3 installer)
Actions #7

Updated by fao89 over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to fao89
Actions #8

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 72 to Sprint 73

Added by Fabricio Aguiar over 4 years ago

Revision f2e28cbd | View on GitHub

Enforce new lines on requirements.in

Before:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-clientpulp-file-clientpulp-npmpulp-npm-clientpulp-rpm-clientpulp-smash

After:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-client
pulp-file-client
pulp-npm
pulp-npm-client
pulp-rpm-client
pulp-smash

https://pulp.plan.io/issues/6697 closes #6697

Added by Fabricio Aguiar over 4 years ago

Revision f2e28cbd | View on GitHub

Enforce new lines on requirements.in

Before:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-clientpulp-file-clientpulp-npmpulp-npm-clientpulp-rpm-clientpulp-smash

After:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-client
pulp-file-client
pulp-npm
pulp-npm-client
pulp-rpm-client
pulp-smash

https://pulp.plan.io/issues/6697 closes #6697

Actions #9

Updated by fao89 over 4 years ago

  • Status changed from ASSIGNED to POST
Actions #11

Updated by Anonymous over 4 years ago

  • Status changed from POST to MODIFIED

Added by Fabricio Aguiar over 4 years ago

Revision f2bef07d | View on GitHub

Enforce new lines on requirements.in

Before:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-clientpulp-file-clientpulp-npmpulp-npm-clientpulp-rpm-clientpulp-smash

After:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-client
pulp-file-client
pulp-npm
pulp-npm-client
pulp-rpm-client
pulp-smash

https://pulp.plan.io/issues/6697 closes #6697

Actions #12

Updated by Anonymous over 4 years ago

Added by Fabricio Aguiar over 4 years ago

Revision 2571c5c7 | View on GitHub

Enforce new lines on requirements.in

Before:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-clientpulp-file-clientpulp-npmpulp-npm-clientpulp-rpm-clientpulp-smash

After:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-client
pulp-file-client
pulp-npm
pulp-npm-client
pulp-rpm-client
pulp-smash

https://pulp.plan.io/issues/6697 closes #6697

Added by Fabricio Aguiar over 4 years ago

Revision 2571c5c7 | View on GitHub

Enforce new lines on requirements.in

Before:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-clientpulp-file-clientpulp-npmpulp-npm-clientpulp-rpm-clientpulp-smash

After:

[vagrant@pulp3-source-fedora31 ~]$ cat /usr/local/lib/pulp/requirements.in
pulpcore==3.3.1
pulp-ansible
pulp-rpm
pulp-file
pulp-ansible-client
pulp-file-client
pulp-npm
pulp-npm-client
pulp-rpm-client
pulp-smash

https://pulp.plan.io/issues/6697 closes #6697

Actions #13

Updated by Anonymous over 4 years ago

Actions #14

Updated by fao89 over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF