New keepalived config is bugged

[Luis Filipe Domingues]

The patch from https://review.openstack.org/#/c/634209/ does not work properly well.

The option with the source of the script was called "src_ping_script" and it should be called src_check_script.

Plus, if we just use "check_script" and "src_check_script", the file from the source will be copy, to the target, and take the name set on "check_script". This could be an issue if we have parameters under "check_script".
For example, if we have something like:
check_script: "/etc/keepalived/pingable_check.sh {{ keepalived_ping_count }} {{ keepalived_ping_address }}"

We could end up with a file name:
"/etc/keepalived/pingable_check.sh 1 203.0.113.89"

We should for that have an optional field for setting the destination without taking the parameters, like dest_check_script.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/637839

[Luis Filipe Domingues (domigues-luis)]

Fix for the script name has been merged on upstream project:
https://github.com/evrardjp/ansible-keepalived

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/637839
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=cf4a3d96c925c0d1de0be80ced8ac88f2799a1eb
Submitter: Zuul
Branch: master

commit cf4a3d96c925c0d1de0be80ced8ac88f2799a1eb
Author: Domingues Luis Filipe <email address hidden>
Date: Tue Feb 19 14:10:35 2019 +0100

    Fix keepalive pingable_check_script

    The source field needs to be called src_check_script
    instead of src_ping_check. Plus, check_script takes
    parameters, so we need to define the target name under
    dest_check_script field.

    Change-Id: I7b02a5d58816d3dc7b21a0b5f3bedab530f2b906
    Closes-bug: #1816561

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/openstack-ansible 19.0.0.0b1 development milestone.