[ZBX-9928] Triple SMS for each trigger action Cinterion MC35i Created: 2015 Oct 02  Updated: 2017 May 30  Resolved: 2015 Nov 24

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Server (S)
Affects Version/s: 2.2.10
Fix Version/s: 2.0.17rc1, 2.2.12rc1, 2.4.8rc1, 3.0.0alpha5

Type: Incident report Priority: Minor
Reporter: anton Assignee: Unassigned
Resolution: Fixed Votes: 0
Labels: regression, sms
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

CentOS release 6.7 64

Attachments: Text File gsm.txt     File sms.patch     Text File zabbix_server.log    

 Description   

After update Zabbix from 2.2.9 to 2.2.10 I have a problem with sms.
Every time sms action send out three sms, because zabbix interpreting modem response as an error:
Expected [+CMGS: ] received [+CMGS:]

Device : /dev/ttyS0
Manufacturer : Cinterion
Model : MC35i

 Comments   
Comment by Aleksandrs Saveljevs [ 2015 Oct 02 ]

Apparently, a regression from ZBX-3507.

Comment by Andris Zeila [ 2015 Nov 06 ]

Currently the only scenario I can imagine is that this modem sends the +CMGS:<space><number>\r in two packets - +CMGS: and <space><number>\r. But if it's true - then there was also a good chance failing before ZBX-3507 fix.

Comment by Andris Zeila [ 2015 Nov 09 ]

Would it be possible to attach log fragment created with DebugLevel 4 while sending SMS ? Also would You be willing to test a patch (when it will be ready) ? Otherwise we can only make a good guess.

Comment by Andris Zeila [ 2015 Nov 10 ]

Thanks!

I attached the patch that basically uses pre ZBX-3507 processing logic while keeping the Huawei E1752 compatibility added in ZBX-3507. Could you please test it with DebugLevel 4.

Comment by anton [ 2015 Nov 14 ]

I tested the patched version, at first glance, sms actions works fine.
Log fragment in Attachments.
Thanks.

Comment by Andris Zeila [ 2015 Nov 17 ]

Great, then we will proceed with this fix. Thanks!

Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-9928 .

Comment by Aleksandrs Saveljevs [ 2015 Nov 20 ]

Looks good, but please see r56815 before merging (it guards against empty read()).

wiper Thanks!

Comment by Andris Zeila [ 2015 Nov 20 ]

Released in:

  • pre-2.0.17rc1 r56817
  • pre-2.2.12rc1 r56818
  • pre-2.4.8rc1 r56821
  • pre-3.0.0alpha5 r56822
Generated at Fri Apr 26 01:03:42 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.