diff --git a/roles/openvpn/templates/announcer.j2 b/roles/openvpn/templates/announcer.j2 index 0b4a10a63..8353068b7 100755 --- a/roles/openvpn/templates/announcer.j2 +++ b/roles/openvpn/templates/announcer.j2 @@ -4,15 +4,14 @@ HANDLE= UUID= if [ -f /etc/iiab/openvpn_handle ]; then - # Option #0: Source directly from /etc/iiab/local_vars.yml in future? - # Option #1 + # Option #1: Source directly from /etc/iiab/local_vars.yml in future? + # Option #2 HANDLE=`cat /etc/iiab/openvpn_handle` - # Option #2: UUID is later used, if openvpn_handle is not set # Sourcing a variable from ~4 different places is a recipe for total confusion # (or worse!) Far better to make variable openvpn_handle and file # /etc/iiab/openvpn_handle "obligatory" (EMPTY STRING "" IS TOLERATED, in which -# case OpenVPN server should use /etc/iiab/uuid in lieu of the handle). +# case OpenVPN server tries to use /etc/iiab/uuid below, in lieu of handle...) # CLARIF: "systemctl restart openvpn@xscenet" still runs even if the above is # defied. e.g. if an implementer deletes /etc/iiab/openvpn_handle by accident.