Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

broken reference to pulp-glue in main branch #654

Closed
maciej-markowski opened this issue Mar 14, 2023 · 2 comments · Fixed by #657
Closed

broken reference to pulp-glue in main branch #654

maciej-markowski opened this issue Mar 14, 2023 · 2 comments · Fixed by #657
Assignees
Labels
Documentation Improvements or additions to documentation
Milestone

Comments

@maciej-markowski
Copy link

Summary

Reference to pulp-glue was recently bumped which breaks the installation.

Steps to reproduce

(venv) [vagrant@nmdws ~]$ git clone --depth=1 https://github.com/pulp/pulp-cli.git
Cloning into 'pulp-cli'...
remote: Enumerating objects: 288, done.
remote: Counting objects: 100% (288/288), done.
remote: Compressing objects: 100% (229/229), done.
remote: Total 288 (delta 74), reused 165 (delta 42), pack-reused 0
Receiving objects: 100% (288/288), 247.33 KiB | 2.78 MiB/s, done.
Resolving deltas: 100% (74/74), done.
(venv) [vagrant@nmdws ~]$ cd pulp-cli
(venv) [vagrant@nmdws pulp-cli]$ pip install .
Processing /home/vagrant/pulp-cli
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Collecting click<9.0.0,>=8.0.0
  Using cached click-8.1.3-py3-none-any.whl (96 kB)
ERROR: Could not find a version that satisfies the requirement pulp-glue==0.19.0.dev (from pulp-cli==0.19.0.dev0) (from versions: 0.0.0, 0.17.0, 0.17.1, 0.18.0)
ERROR: No matching distribution found for pulp-glue==0.19.0.dev (from pulp-cli==0.19.0.dev0)

Expected behavior

Installation is completed correctly.

Stacktrace/Error log

ERROR: Could not find a version that satisfies the requirement pulp-glue==0.19.0.dev (from pulp-cli==0.19.0.dev0) (from versions: 0.0.0, 0.17.0, 0.17.1, 0.18.0)
ERROR: No matching distribution found for pulp-glue==0.19.0.dev (from pulp-cli==0.19.0.dev0)

Pulp and pulp-cli version info

Additonal context

pulp-cli installation from main branch is used in pulp_python pipeline and this bumped reference causes pipeline to fail.
https://github.com/maciej-markowski/pulp_python/actions/runs/4417009990/jobs/7744404558#step:6:257

@maciej-markowski maciej-markowski added bug Something isn't working (template-set) Triage-Needed Needs to be reviewed at next pulp-cli mtg labels Mar 14, 2023
@mdellweg
Copy link
Member

This is a source installation. It needs to install the cli and glue in the same call to the same version.
pip install ./pulp-cli ./pulp-cli/pulp-glue should work.

pip install -e ./pulp-cli -e ./pulp-cli/pulp-glue for editable installations.

@mdellweg mdellweg added invalid This request doesn't seem right Documentation Improvements or additions to documentation and removed bug Something isn't working (template-set) Triage-Needed Needs to be reviewed at next pulp-cli mtg invalid This request doesn't seem right labels Mar 15, 2023
@mdellweg mdellweg self-assigned this Mar 16, 2023
@mdellweg
Copy link
Member

I figured, this information is misleading in the docs. Will fix it.

@mdellweg mdellweg added this to the 0.18.0 milestone Mar 16, 2023
mdellweg added a commit to mdellweg/pulp-cli that referenced this issue Mar 16, 2023
@mdellweg mdellweg linked a pull request Mar 16, 2023 that will close this issue
3 tasks
mdellweg added a commit to mdellweg/pulp-cli that referenced this issue Mar 16, 2023
mdellweg added a commit that referenced this issue Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants