Project

Profile

Help

Issue #8980

closed

content app finishes servicing on_demand content requests before artifacts are fully saved

Added by dkliban@redhat.com almost 3 years ago. Updated almost 3 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:
Sprint:
Sprint 99
Quarter:


Related issues

Has duplicate Container Support - Issue #8911: Fix pull test that fails interminentlyCLOSED - DUPLICATEActions
Actions #1

Updated by ipanova@redhat.com almost 3 years ago

  • Description updated (diff)
Actions #2

Updated by ipanova@redhat.com almost 3 years ago

  • Has duplicate Issue #8911: Fix pull test that fails interminently added
Actions #3

Updated by dkliban@redhat.com almost 3 years ago

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

Updated by bmbouter almost 3 years ago

  • Sprint/Milestone set to 3.14.0
Actions #5

Updated by pulpbot almost 3 years ago

  • Status changed from ASSIGNED to POST
Actions #6

Updated by dkliban@redhat.com almost 3 years ago

  • Triaged changed from No to Yes

Added by dkliban@redhat.com almost 3 years ago

Revision 0b02f957 | View on GitHub

Shields coroutine that is saving Artifact and Content

After the client receives the last chunk of the response, it closes the connection with the content app. This causes all scheduled coroutines to be cancelled. If the the thread that is performing the Artifact/Content save has not finished running, the Content is never saved.

This patch ensures that the Artifact and the Content are fully saved to the database after being streamed to the user.

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

Actions #7

Updated by dkliban@redhat.com almost 3 years ago

  • Status changed from POST to MODIFIED
Actions #8

Updated by pulpbot almost 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF