Skip to content

Update notify_operator.py - #2

Open
fgenellini wants to merge 1 commit into
paleg:masterfrom
fgenellini:patch-2
Open

Update notify_operator.py#2
fgenellini wants to merge 1 commit into
paleg:masterfrom
fgenellini:patch-2

Conversation

@fgenellini

Copy link
Copy Markdown

According https://github.com/pistolero/zbxsend syntax, the function is not wearing a zbxsend third parameter after the port. In my case, I had to remove the "20" at the end of the line No. 141 to work with Zabbix sender.

According https://github.com/pistolero/zbxsend syntax, the function is not wearing a zbxsend third parameter after the port. In my case, I had to remove the "20" at the end of the line No. 141 to work with Zabbix sender.
@paleg

paleg commented Oct 22, 2015

Copy link
Copy Markdown
Owner

You are probably using old version of zbxsend (<0.1.5).
Recent versions of send_to_zabbix accepts 4 parameters.
https://github.com/pistolero/zbxsend/blob/master/zbxsend.py#L23

@fgenellini

Copy link
Copy Markdown
Author

@paleg
I installed it with pip (zbx_send 1.6) (https://pypi.python.org/pypi/zbxsend/0.1.6) and to work with Zabbix 2.4 had to take the third parameter. Finally, seeing the code you are right, but I have not cleansed from end to end.

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.

2 participants