Skip to content

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

Open
kuzukami wants to merge 1 commit into
PagerDuty:masterfrom
kuzukami:parametrise-record-separator-for-multiline-in-zabbix
Open

new optional argument as record separator for multi-line support in zabbix#35
kuzukami wants to merge 1 commit into
PagerDuty:masterfrom
kuzukami:parametrise-record-separator-for-multiline-in-zabbix

Conversation

@kuzukami

@kuzukami kuzukami commented Oct 1, 2019

Copy link
Copy Markdown

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

kuzukami commented Oct 1, 2019

Copy link
Copy Markdown
Author

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