[ZBXNEXT-1343] Remove support of old style DM Created: 2012 Jul 31 Updated: 2016 Mar 29 Due: 2014 Apr 14 Resolved: 2014 Dec 23 |
|
Status: | Closed |
Project: | ZABBIX FEATURE REQUESTS |
Component/s: | Agent (G), API (A), Documentation (D), Frontend (F), Server (S) |
Affects Version/s: | None |
Fix Version/s: | 2.3.0 |
Type: | Change Request | Priority: | Major |
Reporter: | Alexei Vladishev | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | distributed, dm | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Issue Links: |
|
Description |
Support of distributed monitoring should be removed in preparation to work on new next generation DM. Draft of the specification is available at http://www.zabbix.org/wiki/Docs/specs/ZBXNEXT-1343 |
Comments |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 17 ] |
Removed all of the DM-related code from the frontend. Please test and review. |
Comment by richlv [ 2012 Aug 17 ] |
that should be in svn://svn.zabbix.com/branches/dev/ZBXNEXT-1343 , i guess |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 17 ] |
I've added a note to the specification about entries in the database, that should be deleted: http://www.zabbix.org/wiki/Docs/specs/ZBXNEXT-1343#Database_changes |
Comment by Alexander Vladishev [ 2012 Aug 21 ] |
(1) [S] ZBX_SYNC, ZBX_HISTORY and ZBX_HISTORY_SYNC should be removed from C code and schema.tmpl <dimir> RESOLVED in r29777, r29781. Please check also frontend change in r29782. <Sasha> CLOSED |
Comment by Alexander Vladishev [ 2012 Aug 21 ] |
(2) [S] Please review my changes in r29661, r29678, 29689 and 29690 <dimir> CLOSED |
Comment by dimir [ 2012 Aug 23 ] |
Removed TABLE_TYPE_CONFIG and TABLE_TYPE_HISTORY table constants from frontend in r29796. Also updated schema.inc.php, it was missing few changes from schema.tmpl . |
Comment by richlv [ 2012 Aug 31 ] |
(3) not directly related, but whenever we click on the "main" menu (like "monitoring", "configuration" etc), 'submenu' opens the first entry (like "dashboard", "host groups")... except in administration. here it previously always defaulted to "dm", now it defaults to "proxies". pointed out by andris Eduards RESOLVED r.44077 jelisejev CLOSED. |
Comment by Oleksii Zagorskyi [ 2012 Sep 07 ] |
(4) a small question - are we going to remove DM related tables from schema ? <dimir> http://www.zabbix.org/wiki/Docs/specs/ZBXNEXT-1343#Database_changes <zalex> dang, How I missed that ? :> |
Comment by Alexey Fukalov [ 2012 Sep 17 ] |
(5) <richlv> this was about PERM_READ_LIST, which was removed in that issue CLOSED |
Comment by Stefan [ 2013 Aug 06 ] |
why remove it? and what is the alternative? please dont say proxys! |
Comment by Gabriele Armao [ 2013 Oct 16 ] |
any news on this? I don't see any change in 2.2 roadmap |
Comment by Pavels Jelisejevs (Inactive) [ 2013 Oct 16 ] |
We decided not to remove in 2.2. |
Comment by Marc [ 2014 Jan 09 ] |
If it's still aimed to be removed, then update the documentation accordantly, please! There has to be a very clear indication that this feature is going to be removed. |
Comment by Stefan [ 2014 Mar 14 ] |
what would be the alternatve? PS: sorry for my bad english, but ih hope you understand me.. Solutions?! Proxy must send alerts, but its limited to get only data from 1day |
Comment by richlv [ 2014 Mar 16 ] |
(6) let's check whether this fixes sasha CLOSED it should be fixed separately. |
Comment by Norbert Bladt [ 2014 Mar 19 ] |
Proxy is the biggest part of the solution. It has many advantages. We use DM because nothing else was =0& {zabbix[proxy,Proxyname,lastaccess].last(0)}=0 |
Comment by Andris Zeila [ 2014 Mar 25 ] |
(7) Please review my changes in r43720 sasha Thanks! CLOSED |
Comment by Eduards Samersovs (Inactive) [ 2014 Mar 31 ] |
Frontend ready in r.43901 |
Comment by Gabriele Armao [ 2014 Mar 31 ] |
hello, since you're closing the ticket and planned the removal of old style DM in 2.4 roadmap (https://www.zabbix.org/wiki/Docs/roadmap#Planned), do you have any other information for your users (except the draft that only lists what you're going to remove) of how the "next generation" DM will be? it will be quite useful to know ahead what to do with DM in general. Thank you. |
Comment by dimir [ 2014 Mar 31 ] |
Unfortunately there are no decisions currently on what will the future DM look like and when will it appear. It's only known that it neither will be available in 2.4 nor in 2.6 . |
Comment by Gabriele Armao [ 2014 Mar 31 ] |
ok, just to get this correctly, since you planned and, from what I see, removed the old style DM support, this means that upgrading to the future 2.4 will fully remove master-slave setup and capability, if in place on the upgraded installation, while leaving the information of other nodes on the database (but not viewable on the frontend)? I'm not arguing nor complaining, I just want to be sure on what will happen, if I can rely or not on DM support for the near future. thanks again |
Comment by Oleksii Zagorskyi [ 2014 Mar 31 ] |
FYI - there is |
Comment by Eduards Samersovs (Inactive) [ 2014 Apr 01 ] |
(8) Strings removed:
jelisejev Some more strings that have been removed:
sasha CLOSED |
Comment by Alexander Vladishev [ 2014 Apr 01 ] |
Upgrade path isn't done. Reopening. |
Comment by Marc [ 2014 Apr 02 ] |
This issue should be considered in upcoming Webinars as well for the same reason previously mentioned. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 02 ] |
(9) [F] In CTrigger::getObjects() the "expression" key is unset from an unexisting variable $options. Eduards RESOLVED r.44077 jelisejev CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 03 ] |
(10)
Eduards RESOLVED r.44100 jelisejev Eduards RESOLVED r.44190 2. Please have someone from the C side review your changes to the C code. sasha C side successfully reviewed and tested. CLOSED jelisejev Currently in trunk, when we reach the maximum ID, we try to refresh them to check if any IDs have been freed. Please restore this feature. Eduards RESOLVED r.44363 jelisejev Still not resolved. Eduards RESOLVED r.44393 jelisejev CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 03 ] |
(11) [F] The name of the server defined in $ZBX_SERVER_NAME is no longer displayed. Eduards RESOLVED r.44077 jelisejev The class "serverName" does not match our CSS naming conventions:
Eduards RESOLVED r.44190 jelisejev CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 03 ] |
(12) [F] I've made a small change in r44045, please review. Eduards CLOSED jelisejev I've also restored the comment in CHostBase.php in r44058. It talks about nodes of a graph, not DM nodes. Eduards CLOSED |
Comment by Alexander Vladishev [ 2014 Apr 03 ] |
(13) I removed unused alias from SQL statement in r44046. Please review. Eduards CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 04 ] |
(14) Seems that some of the node related code remains in idens.inc.php: if (!isset($ident['node'])) { return reset($searchedImages); } elseif (isset($searchedImages[$ident['node']])) { return $searchedImages[$ident['node']]; } else { return 0; } Eduards RESOLVED r.44100 jelisejev CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 04 ] |
(15) The name of the "DM" menu link must be changed to "Proxies". Also please fix the old (3) issue. Eduards RESOLVED r.44077 jelisejev CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 04 ] |
(16) There are a bunch of places where we use the CMultiSelect class and passed the name of the node in the "prefix" parameter. In some places, like configuration.screen.constructor.edit.php:289 the node has been removed, but now an empty "prefix" parameter remains. It should be removed as well. Eduards RESOLVED r.44077 jelisejev CLOSED. |
Comment by Alexander Vladishev [ 2014 Apr 10 ] |
Upgrade patches are ready for testing. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 10 ] |
(17) I've updated to the latest /trunk in r44286 and there have been several conflicts on the PHP side, please review. Eduards CLOSED |
Comment by Andris Zeila [ 2014 Apr 11 ] |
(18) [S] graphs_theme table is not synchronized between nodes, so we don't need to process graph_theme.description fields. sasha RESOLVED in r44341 wiper CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 15 ] |
Frontend TESTED. |
Comment by Andris Zeila [ 2014 Apr 15 ] |
Server TESTED |
Comment by Alexander Vladishev [ 2014 Apr 16 ] |
Old style node-based distributed monitoring was removed in pre-2.3.0 (trunk) r44485. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 16 ] |
(20) The upgrade patch stopped with the following error
But we allow to have IT services with the same names. sasha Fixed in pre-2.3.0 (trunk) r44490. CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 16 ] |
(21) I've removed all node-related parameters from the API docs from all of the get, exists and getobjects methods and the reference commentary. <richlv> looks like https://www.zabbix.com/documentation/2.4/manual/api/reference/hostinterface/exists still has it, or have i misunderstood something ? <richlv> removed it. also removed node related info from : jelisejev Thanks. CLOSED. |
Comment by richlv [ 2014 Apr 17 ] |
(22) node related parameters have not been removed from zabbix_server.conf
<richlv> in r44604 removed NodeID, NodeNoEvents, NodeNoHistory parameters and node reference from StartTrappers. the config file in tests is a bit outdated, but let's handle it separately sasha Thank you! CLOSED |
Comment by richlv [ 2014 Apr 17 ] |
(23) "node watcher" busy rate item has not been removed from the default zabbix server template sasha RESOLVED in the development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-1343 r44606. jelisejev CLOSED. |
Comment by richlv [ 2014 Apr 18 ] |
(24) frontends/php/include/menu.inc.php has i suspect that this might not be needed anymore sasha RESOLVED in the development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-1343 r44607. jelisejev Seems that the "node_perm" and "force_disable_all_nodes" parameters are unused as well. I've removed them in r44689. Please review. sasha CLOSED |
Comment by richlv [ 2014 Apr 21 ] |
(25) frontends/php/include/defines.inc.php has : this does not seem to be used anywhere sasha RESOLVED in the development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-1343 r44635. jelisejev CLOSED. |
Comment by Martins Valkovskis [ 2014 Apr 22 ] |
(26) Updated documentation:
<sasha> thanks! CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Apr 23 ] |
TESTED, but please review (21) before merging. sasha (20), (21) and (22) are available in pre-2.3.0 (trunk) r44806. |
Comment by richlv [ 2014 Apr 30 ] |
(27) i removed node references from the server manpage directly in trunk in r45000 and regenerated https://www.zabbix.com/documentation/2.4/manpages/zabbix_server please review sasha CLOSED |
Comment by richlv [ 2014 Apr 30 ] |
(28) removed node reference from code comment in r45001, please review (function names and other things in there still reference nodes, but i guess we can ignore that for now ) sasha Function and module names will renamed later. CLOSED |
Comment by richlv [ 2014 Oct 16 ] |
(29) looks like template changes are missing from these documentation pages :
sasha RESOLVED <richlv> changed for -> from and in the upgrade notes added info on how to fix the affected template after the upgrade. please review these changes -> again RESOLVED martins-v I added some formatting for better readability plus some minor grammar changes. As far as I'm concerned, this can be CLOSED. |
Comment by richlv [ 2014 Oct 16 ] |
(30) once subissue 29 is closed, changed templates should be uploaded to the official template page sasha Updated Zabbix Templates/Official Templates RESOLVED <richlv> minor change to the heading level, otherwise looks great -> CLOSED |
Comment by richlv [ 2014 Oct 16 ] |
subissues still open : |