openstack
/
puppet-tripleo
Code Issues Proposed changes

Add resource-stickiness=INFINITY to VIPs 

Right now we do not add any resource stickiness to the VIPs. This has
one consequence when we configure IHA: Namely, when a fenced compute
node comes back (i.e. it recovers), pacemaker is free to move the VIPs
around the controllers (see http://clusterlabs.org/pacemaker/doc/en-US/Pacemaker/1.1/html/Clusters_from_Scratch/_prevent_resources_from_moving_after_recovery.html)
to optimize the resource placement. We can observe the VIP moving with
the following message:
Apr 12 06:37:04 [979790] controller-1 pengine: notice: LogAction: * Move ip-10.0.0.110 ( controller-1 -> controller-0 )

This movement of the VIP is highly undesirable because in Instance HA the fence_compute agent needs to talk to keystone via the VIP, and if the VIP is on the move we might get the following errors:
Apr 12 06:37:23 [979787] controller-1 stonith-ng: warning: log_action: fence_compute[259311] stderr: [ Starting new HTTP connection (1): 10.0.0.110 ]
Apr 12 06:37:23 [979787] controller-1 stonith-ng: warning: log_action: fence_compute[259311] stderr: [ keystoneauth1.exceptions.connection.ConnectFailure: Unable to establish connection to http://10.0.0.110:5000 ]
Apr 12 06:37:28 [979787] controller-1 stonith-ng: warning: log_action: fence_compute[261144] stderr: [ REQ: curl -g -i -X GET http://10.0.0.110:5000 -H "Accept: application/json" -H "User-Agent: python-keystoneclient" ]
Apr 12 06:37:28 [979787] controller-1 stonith-ng: warning: log_action: fence_compute[261144] stderr: [ Starting new HTTP connection (1): 10.0.0.110 ]
Apr 12 06:37:28 [979787] controller-1 stonith-ng: warning: log_action: fence_compute[261144] stderr: [ keystoneauth1.exceptions.connection.ConnectFailure: Unable to establish connection to http://10.0.0.110:5000 ]

By setting the resource-stickiness of the VIPs to INFINITY we control how strongly they prefer to stay running where they are.

Change-Id: I6862452d2250ac4c2c3e04840983510a3cd13536
Closes-Bug: #1763586
(cherry picked from commit 2131880c71)
This commit is contained in:
Michele Baldessari 2018-04-13 08:48:18 +02:00 committed by Chris Jones
parent 1f69cc5f76
commit cb114de452
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
manifests/pacemaker/haproxy_with_vip.pp
View File

@ -80,6 +80,7 @@ define tripleo::pacemaker::haproxy_with_vip(
cidr_netmask => $netmask,
nic => $nic,
ipv6_addrlabel => $ipv6_addrlabel,
meta_params => 'resource-stickiness=INFINITY',
location_rule => $location_rule,
tries => $pcs_tries,
}