madkiss/keystone — pull request #1 by leseb (2012-06-21 22:12:24+00:00)
Title: Indentation issue

The if ! check_binary test seems badly indented. It's weird because the indentation is similar as the one propose by the manual http://www.linux-ha.org/doc/dev-guides/_testing_for_binaries_literal_have_binary_literal_and_literal_check_binary_literal.html. Actually this test is never performed and the function keystone_monitor returns return $OCF_SUCCESS. Thus, Pacemaker believes that the resource is running. At least, keystone daemon is running but nothing is reachable from the keystone client binary. After this change, the ra works like a charm :)