Project

Profile

Help

Send by e-mail

Issue #7002

Streamed responses don't appear in the access_log of the pulpcore-content app

Added by dkliban@redhat.com over 1 year ago. Updated about 1 year ago.

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

Description

  1. Create an RPM Remote with policy=on_demand
  2. Sync a repository using that remote.
  3. Create a publication from the new repository version.
  4. Create a distribution associated with the publication.
  5. Request a non-metadata file from the base_url of the distribution
  6. Notice that the request is not logged
  7. Request the same file again.
  8. Notice that the request is logged this time.

I expect the request to appear in the logs every time.

Associated revisions

Revision 9208b80a View on GitHub
Added by gerrod about 1 year ago

Streamed responses are properly returned to handler to be logged

When a request for content that was on_demand occured for the first time, no logs where present due to the response not properly being return to the content handler, despite the response actually occuring.

fixes #7002 https://pulp.plan.io/issues/7002

Revision f0c6967b View on GitHub
Added by gerrod about 1 year ago

Streamed responses are properly returned to handler to be logged

When a request for content that was on_demand occured for the first time, no logs where present due to the response not properly being return to the content handler, despite the response actually occuring.

fixes #7002 https://pulp.plan.io/issues/7002

(cherry picked from commit 9208b80a984433c4e058c1fb5d2875aac8a7e702)

History

#1 Updated by fao89 over 1 year ago

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

#2 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 75 to Sprint 76

#3 Updated by rchan about 1 year ago

  • Sprint changed from Sprint 76 to Sprint 77

#4 Updated by gerrod about 1 year ago

  • Assignee set to gerrod

#5 Updated by pulpbot about 1 year ago

  • Status changed from NEW to POST

#6 Updated by gerrod about 1 year ago

  • Status changed from POST to MODIFIED

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

  • Sprint/Milestone set to 3.6.0

#9 Updated by pulpbot about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Send by e-mail

Also available in: Atom PDF