Project

Profile

Help

Backport #9547

closed

Backport "Failed task did not clean up properly resource reservations" to 3.7.z

Added by ttereshc over 2 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Triaged:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 110
Quarter:

Description

In Katello, if migration to pulp 3 fails due to some issue, e.g. ran out of disk space, resources for failed tasks are not cleaned up.


Related issues

Copied from Pulp - Issue #7907: Failed task did not clean up properly resource reservationsCLOSED - CURRENTRELEASEbmbouterActions
Actions #1

Updated by ttereshc over 2 years ago

  • Copied from Issue #7907: Failed task did not clean up properly resource reservations added
Actions #2

Updated by ttereshc over 2 years ago

  • Triaged changed from Yes to No
Actions #3

Updated by rchan over 2 years ago

  • Sprint changed from Sprint 108 to Sprint 109
Actions #4

Updated by rchan over 2 years ago

  • Sprint changed from Sprint 109 to Sprint 110
Actions #5

Updated by bmbouter over 2 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
Actions #6

Updated by pulpbot over 2 years ago

  • Status changed from ASSIGNED to POST

Added by bmbouter over 2 years ago

Revision 4310d43e | View on GitHub

Adds additional lock cleanup to worker cleanup

As another layer of security to guard against lock cleanup not occurring due to Redis not delivering the _release_resource task, ensure all locks are also cleaned up even for tasks that are in their final states.

backports #7907

fixes #9547

(cherry picked from commit 516df3147b56660fbc9b22c215e309da1ff8080e)

Actions #7

Updated by bmbouter over 2 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #8

Updated by pulpbot over 2 years ago

  • Sprint/Milestone set to 3.7.9
Actions #9

Updated by pulpbot over 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF