Skip to content
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.

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

new optional argument as record separator for multi-line support in zabbix #35

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

Conversation

kuzukami
Copy link

@kuzukami kuzukami commented Oct 1, 2019

An optional fourth argument is introduced as the record separator prefix 'eor_sep_prefix' for the parser of alert text 'body_str'.
The new record separator is defined as 'eor_sep_prefix' + "newline".
We can use any comment text of multi-line for a pagerduty alert even if the alert text contains ":" because the record separator can be set to a rare word, such as "@#@%@newline".
The default value of optional fourth argument 'eor_sep_prefix' is an empty string to preserve the backward compatibility of script.
The both cases of backward compatible 3args and new record separator 4args were tested.

… in zabbix-integration

An optional fourth argument is introduced as the record separator prefix 'eor_sep_prefix' for the parser of alert text 'body_str'.
The new record separator is defined as 'eor_sep_prefix' + "<newline>".
We can use any comment text of multi-line for a pagerduty alert even if the alert text contains ":" because the record separator can be set to a rare word, such as "@#@%@<newline>".
The default value of optional fourth argument 'eor_sep_prefix' is an empty string to preserve the backward compatibility of script.
@kuzukami
Copy link
Author

kuzukami commented Oct 1, 2019

Sample Setup for Media types is below.
image

The settings 'Default message' for Operations, Recovery operations and Update oprations are the same.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant