Project

Profile

Help

Issue #8984

The alias "pbindings" installs versions equal to 0.0.0

Added by lmjachky 3 months ago. Updated about 2 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
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 102
Quarter:

Description

The alias pbindings incorrectly installs the python bindings. On a fresh vagrant VM (Fedora 33), all bindings are installed with versions equal to 0.0.0.

The root cause of this problem lies here: https://github.com/pulp/pulp-openapi-generator/blob/52754d3d8fb2cbc099748a6d5b5b20ce7ae7076a/generate.sh#L23. In pbindings, we pass one of the following values to the script generate.sh (https://github.com/pulp/pulp_installer/blob/f44e5987d07521d83efb61a9b7e80089af22dfea/roles/pulp_devel/templates/alias.bashrc.j2#L190):

  1. pulpcore
  2. pulp_rpm
  3. pulp_python
  4. pulp_container
  5. ...

But, it looks like the prefix pulp is missing in the pulp/api/v3/status endpoint for all components:

(pulp) [vagrant@pulp3-source-fedora33 ~]$ http :24817/pulp/api/v3/status/ | jq --arg plugin 'pulp_container' -r '.versions[]'
{
  "component": "core",
  "version": "3.14.0.dev"
}
{
  "component": "container",
  "version": "2.7.0.dev"
}
{
  "component": "file",
  "version": "1.9.0.dev"
}
{
  "component": "rpm",
  "version": "3.14.0.dev"
}

Solution: Remove the prefix pulp or pulp_ in jq --arg plugin $1 (https://github.com/pulp/pulp-openapi-generator/blob/52754d3d8fb2cbc099748a6d5b5b20ce7ae7076a/generate.sh#L23).

Associated revisions

Revision a8549128 View on GitHub
Added by Lubos Mjachky about 2 months ago

Remove the pulp/pulp_ prefix from the component name

closes #8984

History

#1 Updated by dkliban@redhat.com 3 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 99

#2 Updated by rchan 3 months ago

  • Sprint changed from Sprint 99 to Sprint 100

#3 Updated by rchan 2 months ago

  • Sprint changed from Sprint 100 to Sprint 101

#4 Updated by ipanova@redhat.com about 2 months ago

  • Sprint changed from Sprint 101 to Sprint 102

#5 Updated by lmjachky about 2 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to lmjachky

#6 Updated by Anonymous about 2 months ago

  • Status changed from ASSIGNED to MODIFIED

Please register to edit this issue

Also available in: Atom PDF