Project

Profile

Help

Issue #7620

closed

pulpcore registry api doesn't like Accept headers when syncing from pulp

Added by jsherril@redhat.com about 4 years ago. Updated about 4 years ago.

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

Description

When i have a pulp3 installation with pulp_container 2.0.1 hosting a container repository and i setup a 2nd pulp3 server and try to sync from it, i get an error:

{"errors":[{"code":"UNSUPPORTED","message":"Could not satisfy the request Accept header.","detail":{}}]

It seems that pulp3 doesn't like the Accept header that the other pulp3 server is requesting:

here's the accept header from the syncing pulpcore server:

"application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.list.v2+json,application/vnd.oci.image.index.v1+json,application/vnd.oci.image.manifest.v1+json"

if i delete that header on the request (in our shim layer), it goes through fine.

Actions #1

Updated by jsherril@redhat.com about 4 years ago

and to clarify, this is happening when the blobs are being fetched. It appears the manifests are fetched with no issue

Actions #2

Updated by ipanova@redhat.com about 4 years ago

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

Updated by ipanova@redhat.com about 4 years ago

(07:50:54 PM) ipanova: quick google fu showed that we need to write a custom renderer https://stackoverflow.com/questions/30903951/django-doesnt-parse-a-custom-http-accept-header
(07:51:20 PM) ipanova: https://github.com/pulp/pulp_container/blob/47781dbab4b852542303bee5980c41030aa03d9b/pulp_container/app/registry_api.py#L111
(07:51:29 PM) ipanova: we have it written for the manifest but not for blobs
Actions #4

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 82 to Sprint 83
Actions #5

Updated by ipanova@redhat.com about 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
Actions #6

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 83 to Sprint 84
Actions #7

Updated by pulpbot about 4 years ago

  • Status changed from ASSIGNED to POST
Actions #8

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 84 to Sprint 85

Added by ipanova@redhat.com about 4 years ago

Revision 970e2cfd | View on GitHub

Fixed Renderer to handle properly Manifest and Blob responses.

closes #7620 https://pulp.plan.io/issues/7620

Added by ipanova@redhat.com about 4 years ago

Revision 970e2cfd | View on GitHub

Fixed Renderer to handle properly Manifest and Blob responses.

closes #7620 https://pulp.plan.io/issues/7620

Actions #9

Updated by ipanova@redhat.com about 4 years ago

  • Status changed from POST to MODIFIED
Actions #10

Updated by ipanova@redhat.com about 4 years ago

  • Sprint/Milestone set to 2.2.0
Actions #11

Updated by pulpbot about 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF