Project

Profile

Help

Issue #6643

closed

Issue #6623: pulpcore and plugin pre-flight check seems to not be enforcing and then installer fails at collectstatic

pre-flight check does not understand the plugin "upgrade" boolean

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

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

Description

Consider the following scenario:

  1. Pulp 3.2.1 & pulp-file 0.2 were installed.
  2. The user runs the latest pulp_installer (3.3), to upgrade pulpcore (3.3), and pulp-file (0.3).
  3. The user specifies to install pulp-file in pulp_install_plugins, and includes "upgrade:true ". This instructs the installer to upgrade pulp-file to the latest version, which is currently 0.3.

This causes the following to happen

  1. /usr/local/lib/pulp/requirements.txt exists and contains pulp-file==0.2
  2. pip-compile sees "pulp-file" in requirements.in that pulp_installer generates, but has no further instructions to upgrade pulp-file.
  3. pip-compile produces the following error, and the installer fails:
There are incompatible versions in the resolved dependencies:
  pulpcore==3.3.0 (from -r requirements.in (line 1))
  pulpcore<3.3,>=3.0 (from pulp-file==0.2.0->-r requirements.in (line 2))
  1. However, if the pre-flight check was ignored, the install would have succeeded, because pulp-file would have been upgraded actually.

Users can work around this currently by specifying "version" for the plugin.

We can fix this by running a pre-flight command like: pip-compile -P pulp-file or if multiple plugins are specified to be upgraded: pip-compile -P pulp-file -P pulp-container This is in addition to the requirements.in existing like it currently does.

I confirmed that this will fix this with a venv, starting with these packages installed: pip install pulp-container==1.2.0 pulp-file==0.2.0 pip install pulp-container==1.2.0 pulp-file==0.2.0 pulpcore==3.2.1

Actions #1

Updated by mdepaulo@redhat.com over 4 years ago

  • Description updated (diff)
Actions #2

Updated by mdepaulo@redhat.com over 4 years ago

  • Description updated (diff)
Actions #3

Updated by fao89 over 4 years ago

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

Updated by mdepaulo@redhat.com over 4 years ago

  • Status changed from ASSIGNED to POST

Added by Mike DePaulo over 4 years ago

Revision 7c5365d9 | View on GitHub

Problem: pre-flight check does not understand the plugin "upgrade" boolean

Solution: Call pip-compile -P pulp-file -P pip-compile for example, where each named plugin has upgrade: true. The requirements.in will continue to list all plugins, and their specified versions (if any.)

fixes: #6643 pre-flight check does not understand the plugin "upgrade" boolean https://pulp.plan.io/issues/6643

re: https://pulp.plan.io/issues/6623 pulpcore and plugin pre-flight check seems to not be enforcing and then installer fails at collectstatic

Added by Mike DePaulo over 4 years ago

Revision 7c5365d9 | View on GitHub

Problem: pre-flight check does not understand the plugin "upgrade" boolean

Solution: Call pip-compile -P pulp-file -P pip-compile for example, where each named plugin has upgrade: true. The requirements.in will continue to list all plugins, and their specified versions (if any.)

fixes: #6643 pre-flight check does not understand the plugin "upgrade" boolean https://pulp.plan.io/issues/6643

re: https://pulp.plan.io/issues/6623 pulpcore and plugin pre-flight check seems to not be enforcing and then installer fails at collectstatic

Actions #5

Updated by Anonymous over 4 years ago

  • Status changed from POST to MODIFIED
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 dkliban@redhat.com over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF