ZABBIX BUGS AND ISSUES
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-9968

monitoring -> maps & discovery not updated automatically

    Details

      Description

      pages monitoring -> maps and monitoring -> discovery are not updated automatically anymore

        Issue Links

          Activity

          Hide
          Gunars Pujats (Inactive) added a comment - - edited

          (1) No translation strings changed.

          Oleg Egorov CLOSED

          Show
          Gunars Pujats (Inactive) added a comment - - edited (1) No translation strings changed. Oleg Egorov CLOSED
          Hide
          Gunars Pujats (Inactive) added a comment -

          RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-9968

          Show
          Gunars Pujats (Inactive) added a comment - RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-9968
          Hide
          Oleg Egorov added a comment - - edited

          (2) Coding style issues:

          • CControllerDiscoveryView.php:80, CSCreenBase:194,209,224 - unnecessary ",".

          As discussed, array is not readable, change to switch or to other array type

          var requiredParameters = [
          	// mode			screenid		hostid	pageFile		profileIdx2		screenitemid	timeline
          	//		resourcetype	groupid					profileIdx		updateProfile	data
          	// 0	1		2		3		4		5		6		7		8		9		10		11
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_GRAPH
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_SIMPLE_GRAPH
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_MAP
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_PLAIN_TEXT
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_HOSTS_INFO
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_TRIGGERS_INFO
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_SERVER_INFO
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_CLOCK
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_SCREEN
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_TRIGGERS_OVERVIEW
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_DATA_OVERVIEW
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_URL
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_ACTIONS
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_EVENTS
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_HOSTGROUP_TRIGGERS
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_SYSTEM_STATUS
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_HOST_TRIGGERS
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_HISTORY
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_CHART
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_LLD_SIMPLE_GRAPH
          	[true,	false,	true,	true,	true,	true,	true,	true,	true,	true,	false,	true], // SCREEN_RESOURCE_LLD_GRAPH
          	[true,	true,	false,	false,	false,	false,	false,	true,	false,	false,	false,	false], // SCREEN_RESOURCE_HTTPTEST_DETAILS
          	[true,	true,	false,	false,	false,	false,	false,	false,	false,	false,	true,	false] // SCREEN_RESOURCE_DISCOVERY
          ];
          

          Gunars Pujats RESOLVED in r58693.

          Oleg Egorov

          • Use reset in CControllerMapView.php.
            $data['map'] = $maps[0];
            
          • Required parameters should be defined before optional in CScreenBase.php.
            $this->required_parameters += [
            	'isFlickerfree'		=> true,
            	'mode'				=> true,
            	'timestamp'			=> true,
            	'resourcetype'		=> true,
            	'dataId'			=> true
            ];
            
          • Not good idea...
            $this->screenitem = (array) $options['screenitem'];
            
          • Everywhere use a dot at the end of a sentence.
            /**
             * Items ids
             *
             * @var array
            */
            

          Gunars Pujats RESOLVED in r58923.

          Oleg Egorov CLOSED

          Show
          Oleg Egorov added a comment - - edited (2) Coding style issues: CControllerDiscoveryView.php:80, CSCreenBase:194,209,224 - unnecessary ",". As discussed, array is not readable, change to switch or to other array type var requiredParameters = [ // mode screenid hostid pageFile profileIdx2 screenitemid timeline // resourcetype groupid profileIdx updateProfile data // 0 1 2 3 4 5 6 7 8 9 10 11 [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_GRAPH [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_SIMPLE_GRAPH [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_MAP [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_PLAIN_TEXT [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_HOSTS_INFO [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_TRIGGERS_INFO [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_SERVER_INFO [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_CLOCK [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_SCREEN [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_TRIGGERS_OVERVIEW [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_DATA_OVERVIEW [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_URL [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_ACTIONS [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_EVENTS [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_HOSTGROUP_TRIGGERS [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_SYSTEM_STATUS [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_HOST_TRIGGERS [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_HISTORY [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_CHART [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_LLD_SIMPLE_GRAPH [true, false, true, true, true, true, true, true, true, true, false, true], // SCREEN_RESOURCE_LLD_GRAPH [true, true, false, false, false, false, false, true, false, false, false, false], // SCREEN_RESOURCE_HTTPTEST_DETAILS [true, true, false, false, false, false, false, false, false, false, true, false] // SCREEN_RESOURCE_DISCOVERY ]; Gunars Pujats RESOLVED in r58693. Oleg Egorov Use reset in CControllerMapView.php. $data['map'] = $maps[0]; Required parameters should be defined before optional in CScreenBase.php. $this->required_parameters += [ 'isFlickerfree' => true, 'mode' => true, 'timestamp' => true, 'resourcetype' => true, 'dataId' => true ]; Not good idea... $this->screenitem = (array) $options['screenitem']; Everywhere use a dot at the end of a sentence. /** * Items ids * * @var array */ Gunars Pujats RESOLVED in r58923. Oleg Egorov CLOSED
          Hide
          Oleg Egorov added a comment - - edited

          (3) Check please make_sorting_header function
          In r56947 was added extra option $link, but seems is not used

          Gunars Pujats Extra parameter is used in screen for sorting link, added in r58681.

          RESOLVED in r58681

          Oleg Egorov CLOSED

          Show
          Oleg Egorov added a comment - - edited (3) Check please make_sorting_header function In r56947 was added extra option $link, but seems is not used Gunars Pujats Extra parameter is used in screen for sorting link, added in r58681. RESOLVED in r58681 Oleg Egorov CLOSED
          Hide
          Gunars Pujats (Inactive) added a comment - - edited

          Fixed in:

          • pre-3.0.3rc1 r59778
          • pre-3.1.0 (trunk) r59951
          Show
          Gunars Pujats (Inactive) added a comment - - edited Fixed in: pre-3.0.3rc1 r59778 pre-3.1.0 (trunk) r59951
          Hide
          Alexander Vladishev added a comment - - edited

          (4) Error messages on history page:

          mb_strpos(): Empty delimiter [history.php:124 → CView->render() → include() → CScreenHistory->get() → mb_strpos() in include/classes/screens/CScreenHistory.php:205]
          

          Gunars Pujats RESOLVED in r59411

          Ivo Kurzemnieks CLOSED

          Show
          Alexander Vladishev added a comment - - edited (4) Error messages on history page: mb_strpos(): Empty delimiter [history.php:124 → CView->render() → include() → CScreenHistory->get() → mb_strpos() in include/classes/screens/CScreenHistory.php:205] Gunars Pujats RESOLVED in r59411 Ivo Kurzemnieks CLOSED
          Hide
          Alexander Vladishev added a comment - - edited

          (5) Broken automatic refresh of the history

          history_refresh_broken.png

          Gunars Pujats Due to PostgreSQL query error.

          Gunars Pujats RESOLVED in r59411

          Ivo Kurzemnieks CLOSED

          Show
          Alexander Vladishev added a comment - - edited (5) Broken automatic refresh of the history history_refresh_broken.png Gunars Pujats Due to PostgreSQL query error. Gunars Pujats RESOLVED in r59411 Ivo Kurzemnieks CLOSED
          Hide
          Alexander Vladishev added a comment -

          Undoing all changes in 3.0 and trunk branches.

          Show
          Alexander Vladishev added a comment - Undoing all changes in 3.0 and trunk branches.
          Hide
          Gunars Pujats (Inactive) added a comment - - edited

          (6) Timeline do not update in history.php, therefore latest data are not shown.

          Gunars Pujats RESOLVED in r59411

          Ivo Kurzemnieks CLOSED

          Show
          Gunars Pujats (Inactive) added a comment - - edited (6) Timeline do not update in history.php, therefore latest data are not shown. Gunars Pujats RESOLVED in r59411 Ivo Kurzemnieks CLOSED
          Hide
          Ivo Kurzemnieks added a comment -

          TESTED

          Show
          Ivo Kurzemnieks added a comment - TESTED
          Hide
          Oleg Egorov added a comment - - edited

          (9) Undefined index: history [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:202]
          Undefined index: trends [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:203]

          Gunars Pujats Bug exists only in trunk.

          Gunars Pujats RESOLVED in r59834

          Alexander Vladishev TESTED

          Alexander Vladishev Fixed in pre-3.1.0 r59951.

          CLOSED

          Show
          Oleg Egorov added a comment - - edited (9) Undefined index: history [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:202] Undefined index: trends [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:203] Gunars Pujats Bug exists only in trunk. Gunars Pujats RESOLVED in r59834 Alexander Vladishev TESTED Alexander Vladishev Fixed in pre-3.1.0 r59951. CLOSED
          Hide
          aleksey added a comment - - edited

          Trunk, r59937, at history page still get errors:
          Undefined index: history [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:202]
          Undefined index: trends [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:203]

          Alexander Vladishev Thanks for a report! This fix is not merged into a trunk. Please wait a closing of (9).

          Show
          aleksey added a comment - - edited Trunk, r59937, at history page still get errors: Undefined index: history [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:202] Undefined index: trends [history.php:124 → CView->render() → include() → CScreenHistory->get() → get_min_itemclock_by_itemid() in include/graphs.inc.php:203] Alexander Vladishev Thanks for a report! This fix is not merged into a trunk. Please wait a closing of (9) .
          Hide
          richlv added a comment -

          same problem in monitoring -> web, too

          Show
          richlv added a comment - same problem in monitoring -> web, too

            People

            • Assignee:
              Unassigned
              Reporter:
              richlv
            • Votes:
              5 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: