[ZBX-8820] Not working double Bulk acknowledge triggers Created: 2014 Sep 25 Updated: 2017 May 30 Resolved: 2015 Oct 30 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 2.2.6, 2.4.0 |
Fix Version/s: | 3.0.0alpha2 |
Type: | Incident report | Priority: | Major |
Reporter: | Evgeny Molchanov | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 1 |
Labels: | acknowledgements | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: | ack_step1.png ack_step2.png ack_step3.png ack_step4.png ack_step5.png ack_step6.png | ||||||||||||||||||||
Issue Links: |
|
Description |
Not working double Bulk acknowledge triggers. If acknowledge triggers one, the message is saved. |
Comments |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Sep 25 ] |
Could you please provide more information on how exactly to reproduce the problem? |
Comment by Evgeny Molchanov [ 2014 Sep 25 ] |
2 trigger a confirmation message test1 |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Sep 26 ] |
Ah, thank you for the detailed description, now I understand. The thing is that the acknowledge controls are currently inconsistent. When you use bulk acknowledge for triggers, it acknowledges all unacknowledged problem events for the selected triggers. Since the second time you bulk acknowledge a trigger, all of its events are acknowledged, it doesn't do anything. As for the "Acknowledge" link. If a trigger has unacknowledged problem events, it will act the same way as the bulk acknowledged button, that is, acknowledge all unacknowledged events. But if it doesn't have any unacknowledged problem events, it will acknowledge the last event of the trigger. I guess we'll need to rethink this feature in the next release, this is extremely confusing. And document this in the current releases. |
Comment by Raj R Kumar [ 2014 Oct 16 ] |
Is there any workaround for this? This is really needed by our project. |
Comment by Oleksii Zagorskyi [ 2014 Dec 22 ] |
|
Comment by Alexander Vladishev [ 2015 Feb 08 ] |
Related issue: |
Comment by Alexander Vladishev [ 2015 Aug 25 ] |
Available in the development branch svn://svn.zabbix.com/branches/dev/ZBX-8820 |
Comment by Oleg Egorov (Inactive) [ 2015 Aug 25 ] |
(1) String changes? Strings added:
Strings deleted:
oleg.egorov CLOSED |
Comment by Oleg Egorov (Inactive) [ 2015 Aug 26 ] |
API validation issue added under ZBX-3783 (20) Also { "eventids": [ 219883, 219856 ], "message": 0, "acknowledge_type": "test" } Return success message, and action it the same as acknowledge_type = null |
Comment by Oleg Egorov (Inactive) [ 2015 Aug 27 ] |
(2) Minor coding style improvement in r55204 sasha Thanks! CLOSED |
Comment by Oleg Egorov (Inactive) [ 2015 Aug 27 ] |
(3) Performance issue Fatal error: Allowed memory size of 1073741824 bytes exhausted (tried to allocate 41 bytes) in C:\xampp\htdocs\ZBX-8820\frontends\php\include\classes\debug\CProfiler.php on line 211 sasha Works fine for 500K events and memory_limit = 1GB. Please try again without Debug mode. sasha RESOLVED in r55212. 3:30 minutes for 1M acknowledges and memory_limit = 128M. oleg.egorov CLOSED |
Comment by Oleg Egorov (Inactive) [ 2015 Aug 28 ] |
TESTED |
Comment by Alexander Vladishev [ 2015 Aug 28 ] |
Available in:
|
Comment by Alexander Vladishev [ 2015 Aug 28 ] |
(4) Documentation martins-v Updated sections:
Please review. RESOLVED.
REOPENED martins-v Updated: RESOLVED. sasha Thanks! CLOSED <richlv> let's mention the removal of the hardcoded bulk acknowledge message in the upgrade notes martins-v RESOLVED: https://www.zabbix.com/documentation/3.0/manual/installation/upgrade_notes_300#frontend_changes <richlv> thank you, looks good CLOSED |
Comment by Alexander Vladishev [ 2015 Sep 23 ] |
(5) An empty acknowledge can be created now. It should be fixed. sasha TESTED Take a look at my changes in r55751. vitalijs.cemeris CLOSED |
Comment by vitalijs.cemeris (Inactive) [ 2015 Sep 25 ] |
FIXED IN 3.0.0alpha3(trunk) r55762 |
Comment by Aleksandrs Saveljevs [ 2015 Sep 28 ] |
(6) The following comment has a couple of typos: Trims all elements values that have currect selector and are inside Jquery object first element. The word "currect" is somewhere between "current" and "correct", and the proper capitalization for "Jquery" is "jQuery". sasha RESOLVED in r55821. asaveljevs According to martins-v, "Trims selected element values." is more correct than "Trims selected elements values." sasha Thanks for pointing me to the right side. RESOLVED in r56001. CLOSED |
Comment by richlv [ 2015 Sep 29 ] |
(7) the changelog entry currently talks about "functionality to trim multiple JS elements" - was there no user-visible angle to this change, for example, didn't it disallow adding empty acknowledgement messages ? if it did, that would be nice to use in the changelog entry as per the development guidelines :
sasha RESOLVED in r56002. <richlv> for the record, changelog message was changed to "fixed trimming of new acknowledge messages" - thanks |
Comment by Ivo Kurzemnieks [ 2015 Dec 08 ] |
The message handling caused |