Bug 1293010

Summary:	beaker-transfer should not try to move logs for deleted jobs
Product:	[Retired] Beaker	Reporter:	Dan Callaghan <dcallagh>
Component:	general	Assignee:	matt jia <mjia>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	21	CC:	dcallagh, dowang, mjia, rjoost
Target Milestone:	22.1	Keywords:	Patch
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-02-01 04:30:47 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Dan Callaghan 2015-12-19 02:37:10 UTC

Once a job has been deleted, there is no point beaker-transfer trying to move the logs anymore because they will just be deleted on the next run of beaker-log-delete. It should just ignore recipes from deleted jobs.

Comment 2 matt jia 2015-12-21 04:40:24 UTC

On Gerrit:

   http://gerrit.beaker-project.org/#/c/4552/

Comment 4 Dan Callaghan 2016-01-19 07:01:57 UTC

The following patch will help with verifying the beaker-transfer-related bugs because it lets us see exactly which recipes beaker-transfer is moving and when:

http://gerrit.beaker-project.org/#/c/4601/2

Suggested steps to verify this bug:
1. Schedule a job with /distribution/reservesys
2. Wait for it to start running and produce some logs
3. While it's still running, Cancel and then immediately Delete the job (the idea here is to delete it *before* beaker-transfer notices that it's finished and tries to transfer it)

Expected results:
beaker-transfer should ignore deleted recipe, it should not try to transfer the logs.

Comment 6 Dan Callaghan 2016-02-01 04:30:47 UTC

Beaker 22.1 has been released.