New keepalived config is bugged
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack-Ansible |
Fix Released
|
Undecided
|
Luis Filipe Domingues |
Bug Description
The patch from https:/
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/keepalive
We could end up with a file name:
"/etc/keepalive
We should for that have an optional field for setting the destination without taking the parameters, like dest_check_script.
Changed in openstack-ansible: | |
status: | New → In Progress |
assignee: | nobody → Luis Filipe Domingues (domigues-luis) |
Changed in openstack-ansible: | |
assignee: | Luis Filipe Domingues (domigues-luis) → Jesse Pretorius (jesse-pretorius) |
Changed in openstack-ansible: | |
assignee: | Jesse Pretorius (jesse-pretorius) → Luis Filipe Domingues (domigues-luis) |
Fix proposed to branch: master /review. openstack. org/637839
Review: https:/