Martin Peřina has posted comments on this change.

Change subject: core: adding retry for host start for PPC
......................................................................


Patch Set 2: Code-Review+1

(3 comments)

Minor spelling issues in commit message, otherwise OK

http://gerrit.ovirt.org/#/c/34590/2//COMMIT_MSG
Commit Message:

Line 6: 
Line 7: core: adding retry for host start for PPC
Line 8: 
Line 9: Sometimes a restart operation started by engine is not completed because
Line 10: the host PM failed to put the host on 'on' state
... to put the host to 'on' state ...
Line 11: This occurred on PPC hosts only until now.
Line 12: Some fencing scripts as the one used here (IPMI) have an option to se
Line 13: retries for the 'on' operation by sending : retry_on=<number>
Line 14: 


Line 8: 
Line 9: Sometimes a restart operation started by engine is not completed because
Line 10: the host PM failed to put the host on 'on' state
Line 11: This occurred on PPC hosts only until now.
Line 12: Some fencing scripts as the one used here (IPMI) have an option to se
... have an option to set ...
Line 13: retries for the 'on' operation by sending : retry_on=<number>
Line 14: 
Line 15: The patch adds "retry_on=2" for teh relevant PPC fencing scripts such
Line 16: that this will be sent by engine as a default when a PPC host has one 
of


Line 11: This occurred on PPC hosts only until now.
Line 12: Some fencing scripts as the one used here (IPMI) have an option to se
Line 13: retries for the 'on' operation by sending : retry_on=<number>
Line 14: 
Line 15: The patch adds "retry_on=2" for teh relevant PPC fencing scripts such
... for the relevant ...
Line 16: that this will be sent by engine as a default when a PPC host has one 
of
Line 17: those agents used
Line 18: 
Line 19: Change-Id: I2c7a195526eb3f6454116806131294de2d3fc067


-- 
To view, visit http://gerrit.ovirt.org/34590
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2c7a195526eb3f6454116806131294de2d3fc067
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to