Bug 1321130

Summary: [restraint] rhts_sync_set: command not found
Product: [Retired] Restraint Reporter: PaulB <pbunyan>
Component: generalAssignee: Bill Peck <bpeck>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.1.24CC: asavkov, bpeck, dcallagh, jburke, mjia, pbunyan, rjoost
Target Milestone: 0.1.25   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-08-26 04:48:03 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:
Attachments:
Description Flags
add missing underscore versions of sync/block commands none

Description PaulB 2016-03-24 18:04:16 UTC
Description of problem:
 multihost testing fails due to: 
  rhts_sync_set: command not found

Version-Release number of selected component (if applicable):
 restraint: 0.1.24-1.el6 

Actual results:
https://beaker.engineering.redhat.com/recipes/2592674
https://beaker.engineering.redhat.com/recipes/2592674/tasks/39411080/results/193915166/logs/resultoutputfile.log
---<-snip->--- 
rhts_sync_set: command not found
---<-snip->---

Expected results:
restraint recognises task rhts* commands

Additional info:
-spoke with bpeck about this
-these are legacy rhts commands
-restraint is aware of rhts-* commands
-not rhts_* commands

bpeck said he will make restraint smarter :)

Comment 1 Bill Peck 2016-03-28 14:37:09 UTC
Created attachment 1140889 [details]
add missing underscore versions of sync/block commands

Change-Id: I21e051b9b1195d88eab8f2c9b8b9ad1170c2a98e

Comment 2 Dan Callaghan 2016-04-21 00:59:54 UTC
https://gerrit.beaker-project.org/4761

Comment 3 PaulB 2016-04-25 12:40:07 UTC
Bill,
Running remote EET with latest restraint:
 restraint-0.1.24-1*

I hit an issue with kernel/RHEL6/drivers/3rd-party,
as the runtest.sh has reference to rhts-flush.

Do you account for rhts-flush?

Best,
-pbunyan

Comment 4 Dan Callaghan 2016-04-27 03:39:39 UTC
(In reply to PaulB from comment #3)
> Do you account for rhts-flush?

I filed a separate bug for that: bug 1330816. It looks like an easy one, I think restraint can just supply a no-op implementation because restraint calls to the LC synchronously so there should be nothing to actually flush, right Bill?