Hold Targeter leaves lockfiles after failure

Bug #977358 reported by Thomas Berezansky
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned

Bug Description

I have attempted to make the hold targeter less likely to leave a lockfile sitting around after failures in the branch below. It appears to work for us when errors do crop up.

Mainly it consists of a number of try/otherwise blocks wrapped around the existing code sets.

I dunno if this should be dropped into 2.2 or not.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/hold_targeter

Tags: pullrequest
Changed in evergreen:
milestone: none → 2.3.0-alpha2
Changed in evergreen:
milestone: 2.3.0-alpha2 → 2.3.0-beta1
Changed in evergreen:
milestone: 2.3.0-beta1 → 2.3.0-beta2
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.3.0-beta2 → 2.3.0-rc1
Changed in evergreen:
milestone: 2.3.0-rc1 → 2.3.0
Changed in evergreen:
milestone: 2.3.0 → 2.3.1
Revision history for this message
Ben Shum (bshum) wrote :

Reassigning to 2.4 as this might be more new featurish now that 2.3.0 has come and gone.

Changed in evergreen:
milestone: 2.3.1 → 2.4.0-alpha
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Low
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-alpha1 → 2.4.0-beta
Changed in evergreen:
assignee: nobody → Jason Stephenson (jstephenson)
Revision history for this message
Ben Shum (bshum) wrote :

We added this to our system and it seems fine so far.

Ben Shum (bshum)
Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Jason Stephenson (jstephenson) → nobody
Ben Shum (bshum)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.