Project

Profile

Help

Issue #8980

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

Added by dkliban@redhat.com 4 months ago. Updated 4 months 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 - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 0b02f957 View on GitHub
Added by dkliban@redhat.com 4 months ago

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

History

#1 Updated by ipanova@redhat.com 4 months ago

  • Description updated (diff)

#2 Updated by ipanova@redhat.com 4 months ago

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

#3 Updated by dkliban@redhat.com 4 months ago

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

#4 Updated by bmbouter 4 months ago

  • Sprint/Milestone set to 3.14.0

#5 Updated by pulpbot 4 months ago

  • Status changed from ASSIGNED to POST

#6 Updated by dkliban@redhat.com 4 months ago

  • Triaged changed from No to Yes

#7 Updated by dkliban@redhat.com 4 months ago

  • Status changed from POST to MODIFIED

#8 Updated by pulpbot 4 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF