Skip to content

Feature: Allow specifying user (and group) for sync group and instance notify scripts #296

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Feature: Allow specifying user (and group) for sync group and instance notify scripts #296

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions templates/keepalived.conf.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,16 +96,16 @@ vrrp_sync_group {{ name }} {
{% endfor %}
}
{% if sync_group.notify_script is defined %}
notify "{{ sync_group.notify_script }}"
notify "{{ sync_group.notify_script }}"{{ ' ' if sync_group.notify_script_user_group is defined else ''}}{{sync_group.notify_script_user_group | default('') }}
{% endif %}
{% if sync_group.notify_master is defined %}
notify_master "{{ sync_group.notify_master }}"
notify_master "{{ sync_group.notify_master }}"{{ ' ' if sync_group.notify_master_user_group is defined else ''}}{{sync_group.notify_master_user_group | default('') }}
{% endif %}
{% if sync_group.notify_backup is defined %}
notify_backup "{{ sync_group.notify_backup }}"
notify_backup "{{ sync_group.notify_backup }}"{{ ' ' if sync_group.notify_backup_user_group is defined else ''}}{{sync_group.notify_backup_user_group | default('') }}
{% endif %}
{% if sync_group.notify_fault is defined %}
notify_fault "{{ sync_group.notify_fault }}"
notify_fault "{{ sync_group.notify_fault }}"{{ ' ' if sync_group.notify_fault_user_group is defined else ''}}{{sync_group.notify_fault_user_group | default('') }}
{% endif %}
{% if sync_group.global_tracking is defined and sync_group.global_tracking | bool %}
global_tracking # All VRRP share the same tracking config.
Expand Down Expand Up @@ -248,22 +248,22 @@ vrrp_instance {{ name }} {
{% endif %}

{% if instance.notify_script is defined %}
notify "{{ instance.notify_script }}"
notify "{{ instance.notify_script }}"{{ ' ' if instance.notify_script_user_group is defined else ''}}{{instance.notify_script_user_group | default('') }}
{% endif %}
{% if instance.notify_master is defined %}
notify_master "{{ instance.notify_master }}"
notify_master "{{ instance.notify_master }}"{{ ' ' if instance.notify_master_user_group is defined else ''}}{{instance.notify_master_user_group | default('') }}
{% endif %}
{% if instance.notify_master_rx_lower_pri is defined %}
notify_master_rx_lower_pri "{{ instance.notify_master_rx_lower_pri }}"
notify_master_rx_lower_pri "{{ instance.notify_master_rx_lower_pri }}"{{ ' ' if instance.notify_master_rx_lower_pri_user_group is defined else ''}}{{instance.notify_master_rx_lower_pri_user_group | default('') }}
{% endif %}
{% if instance.notify_backup is defined %}
notify_backup "{{ instance.notify_backup }}"
notify_backup "{{ instance.notify_backup }}"{{ ' ' if instance.notify_backup_user_group is defined else ''}}{{instance.notify_backup_user_group | default('') }}
{% endif %}
{% if instance.notify_fault is defined %}
notify_fault "{{ instance.notify_fault }}"
notify_fault "{{ instance.notify_fault }}"{{ ' ' if instance.notify_fault_user_group is defined else ''}}{{instance.notify_fault_user_group | default('') }}
{% endif %}
{% if instance.notify_stop is defined %}
notify_stop "{{ instance.notify_stop }}"
notify_stop "{{ instance.notify_stop }}"{{ ' ' if instance.notify_stop_user_group is defined else ''}}{{instance.notify_stop_user_group | default('') }}
{% endif %}
{% if instance.smtp_alert is defined and instance.smtp_alert | bool %}
smtp_alert # Send email notification during state transition
Expand Down
6 changes: 6 additions & 0 deletions tests/keepalived-install-complex-example.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,14 +43,20 @@
# For convenience, if you set a variable src_*, the file mentioned in the 'src'
# will be uploaded from the deployment host to the keepalived server at the
# notify_script location.
# To specify a specific user (and group) to run each script as, set a variable *_user_group'
# as seen below.
#
# notify_script:
# notify_script_user_group:
# src_notify_script:
# notify_master:
# notify_master_user_group:
# src_notify_master:
# notify_backup:
# notify_backup_user_group:
# src_notify_backup:
# notify_fault:
# notify_fault_user_group:
# src_notify_fault:

# Uncomment this to have keepalived status checking. They will run on all the hosts
Expand Down