Project

Profile

Help

Issue #7002

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

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

Status:
CLOSED - CURRENTRELEASE
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 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 11 months 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 11 months 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 about 1 year ago

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

#2 Updated by rchan 12 months ago

  • Sprint changed from Sprint 75 to Sprint 76

#3 Updated by rchan 12 months ago

  • Sprint changed from Sprint 76 to Sprint 77

#4 Updated by gerrod 11 months ago

  • Assignee set to gerrod

#5 Updated by pulpbot 11 months ago

  • Status changed from NEW to POST

#6 Updated by gerrod 11 months ago

  • Status changed from POST to MODIFIED

#8 Updated by dkliban@redhat.com 10 months ago

  • Sprint/Milestone set to 3.6.0

#9 Updated by pulpbot 10 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF