Project

Profile

Help

Issue #4052

repo version content api seems to be defined improperly for the api bindings

Added by jsherril@redhat.com about 1 year ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello-P1
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 47

Description

The following api:

GET /pulp/api/v3/repositories/62/versions/1/content/

works just fine for me when curling, but when using the bindings (ruby example):

content = api.repositories_versions_content(repo._latest_version_href)
print content
print content.class

prints:

{}
PulpClient::Content

It seems that the bindings are expecting this api to return a single content object, but instead it returns a results set/list of content objects

History

#1 Updated by jsherril@redhat.com about 1 year ago

  • Tags Katello-P1 added

#2 Updated by CodeHeeler about 1 year ago

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

#3 Updated by dkliban@redhat.com about 1 year ago

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

#4 Updated by dkliban@redhat.com about 1 year ago

I've been able to reproduce the problem. The REST API returns paginated results, however, the OpenAPI schema does not reflect this.

This issue will go away when we convert to single table content0.

[0] https://pulp.plan.io/issues/3812

#5 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 43 to Sprint 44

#6 Updated by rchan 12 months ago

  • Sprint changed from Sprint 44 to Sprint 45

#7 Updated by rchan 11 months ago

  • Sprint changed from Sprint 45 to Sprint 46

#8 Updated by rchan 10 months ago

  • Sprint changed from Sprint 46 to Sprint 47

#9 Updated by dkliban@redhat.com 9 months ago

  • Status changed from ASSIGNED to MODIFIED

This issue was fixed when we changed the repo version content API.

https://github.com/pulp/pulp/commit/2e8545ed00ce2e71dd612ff5a4ea936e951029d9

#10 Updated by bherring 9 months ago

  • Related to Test #4362: Ansible installer for Pulp3 needs to add a systemd unit file for the pulp_content_app service added

#11 Updated by bherring 9 months ago

  • Related to deleted (Test #4362: Ansible installer for Pulp3 needs to add a systemd unit file for the pulp_content_app service)

#12 Updated by daviddavis 8 months ago

  • Tags Pulp 3 added

#13 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#14 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF