Do not mark demotion ready before database checks
This commit is contained in:
@@ -627,7 +627,9 @@ def demote_prepare(config, runtime_state, request_parameters):
|
|||||||
|
|
||||||
return {
|
return {
|
||||||
"ocf_rc": 0,
|
"ocf_rc": 0,
|
||||||
"ready": True,
|
"ready": False,
|
||||||
|
"identity_validated": True,
|
||||||
|
"reason": "database_checks_pending",
|
||||||
"mutation": False,
|
"mutation": False,
|
||||||
"node": node_name,
|
"node": node_name,
|
||||||
"target_node": target_node,
|
"target_node": target_node,
|
||||||
@@ -638,5 +640,6 @@ def demote_prepare(config, runtime_state, request_parameters):
|
|||||||
"local_target": True,
|
"local_target": True,
|
||||||
"successor_selected": True,
|
"successor_selected": True,
|
||||||
"postgresql_mutated": False,
|
"postgresql_mutated": False,
|
||||||
|
"database_checks": False,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -43,7 +43,10 @@ result = plugin.demote_prepare(
|
|||||||
)
|
)
|
||||||
|
|
||||||
assert result["ocf_rc"] == 0
|
assert result["ocf_rc"] == 0
|
||||||
assert result["ready"] is True
|
assert result["ready"] is False
|
||||||
|
assert result["identity_validated"] is True
|
||||||
|
assert result["reason"] == "database_checks_pending"
|
||||||
|
assert result["checks"]["database_checks"] is False
|
||||||
assert result["mutation"] is False
|
assert result["mutation"] is False
|
||||||
assert result["successor_node"] == "david-prodesk"
|
assert result["successor_node"] == "david-prodesk"
|
||||||
print("valid_prepare accepted")
|
print("valid_prepare accepted")
|
||||||
|
|||||||
Reference in New Issue
Block a user