Bug 1280307

Summary: [RFE] consider working with reboot option of fence-agents
Product: [oVirt] ovirt-engine Reporter: Oved Ourfali <oourfali>
Component: BLL.InfraAssignee: Oved Ourfali <oourfali>
Status: CLOSED DEFERRED QA Contact: Pavel Stehlik <pstehlik>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.6.0CC: amureini, bugs, emesika, mperina
Target Milestone: ---Keywords: FutureFeature
Target Release: ---Flags: oourfali: ovirt-future?
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-06-07 18:39:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Oved Ourfali 2015-11-11 12:52:11 UTC 
Description of problem:
Fence agent scripts check whether the host went down when doing reboot, so we can remove the logic we do in the engine, and replace it with calling the reboot option of the fence agents.

More details in
https://github.com/ClusterLabs/fence-agents/blob/master/fence/agents/lib/fencing.py.py

important lines starts at 820 where code switch for cycle / non-cycle. Non-cycle is what interested you, this code is covered at 832-851.
Comment 1 Oved Ourfali 2015-11-17 09:34:53 UTC 
When planning the additional refactoring points on fencing, we should consider this as well.
Comment 2 Oved Ourfali 2016-01-27 06:36:37 UTC 
Martin - I don't see us doing that anytime soon.
Putting ovirt-future.
If it turns out to be something simple or with high benefit then we'll re-target it accordingly.
Comment 3 Martin Perina 2016-03-21 10:54:13 UTC 
*** Bug 1280308 has been marked as a duplicate of this bug. ***