Project

Profile

Help

Issue #4052

closed

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

Added by jsherril@redhat.com over 5 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello
Sprint:
Sprint 47
Quarter:

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

Actions #1

Updated by jsherril@redhat.com over 5 years ago

  • Tags Katello-P1 added
Actions #2

Updated by CodeHeeler over 5 years ago

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

Updated by dkliban@redhat.com over 5 years ago

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

Updated by dkliban@redhat.com over 5 years 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 content[0].

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

Actions #5

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 43 to Sprint 44
Actions #6

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 44 to Sprint 45
Actions #7

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 45 to Sprint 46
Actions #8

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 46 to Sprint 47
Actions #9

Updated by dkliban@redhat.com over 5 years 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

Actions #10

Updated by bherring about 5 years ago

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

Updated by bherring about 5 years ago

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

Updated by daviddavis about 5 years ago

  • Tags Pulp 3 added
Actions #13

Updated by daviddavis about 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #14

Updated by bmbouter about 5 years ago

  • Tags deleted (Pulp 3)
Actions #15

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #16

Updated by ggainey almost 4 years ago

  • Tags Katello added
  • Tags deleted (Katello-P1)

Also available in: Atom PDF