[ZBX-9968] monitoring -> maps & discovery not updated automatically Created: 2015 Oct 19  Updated: 2017 May 30  Resolved: 2016 May 09

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Frontend (F)
Affects Version/s: 3.0.0alpha3
Fix Version/s: 3.0.3rc1, 3.2.0alpha1

Type: Incident report Priority: Blocker
Reporter: richlv Assignee: Unassigned
Resolution: Fixed Votes: 5
Labels: maps, networkdiscovery, regression
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File history_refresh_broken.png    
Issue Links:
Duplicate
is duplicated by ZBX-10790 maps are not refreshed according to r... Closed

 Description   

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

 Comments   
Comment by Gunars Pujats (Inactive) [ 2015 Dec 09 ]

(1) No translation strings changed.

oleg.egorov CLOSED

Comment by Gunars Pujats (Inactive) [ 2015 Dec 09 ]

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

Comment by Oleg Egorov (Inactive) [ 2016 Feb 22 ]

(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
];

gunarspujats 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
*/

gunarspujats RESOLVED in r58923.

oleg.egorov CLOSED

Comment by Oleg Egorov (Inactive) [ 2016 Feb 23 ]

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

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

RESOLVED in r58681

oleg.egorov CLOSED

Comment by Gunars Pujats (Inactive) [ 2016 Mar 17 ]

Fixed in:

  • pre-3.0.3rc1 r59778
  • pre-3.1.0 (trunk) r59951
Comment by Alexander Vladishev [ 2016 Apr 06 ]

(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]

gunarspujats RESOLVED in r59411

iivs CLOSED

Comment by Alexander Vladishev [ 2016 Apr 06 ]

(5) Broken automatic refresh of the history

history_refresh_broken.png

gunarspujats Due to PostgreSQL query error.

gunarspujats RESOLVED in r59411

iivs CLOSED

Comment by Alexander Vladishev [ 2016 Apr 11 ]

Undoing all changes in 3.0 and trunk branches.

Comment by Gunars Pujats (Inactive) [ 2016 Apr 12 ]

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

gunarspujats RESOLVED in r59411

iivs CLOSED

Comment by Ivo Kurzemnieks [ 2016 Apr 15 ]

TESTED

Comment by Oleg Egorov (Inactive) [ 2016 Apr 29 ]

(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]

gunarspujats Bug exists only in trunk.

gunarspujats RESOLVED in r59834

sasha TESTED

sasha Fixed in pre-3.1.0 r59951.

CLOSED

Comment by aleksey [ 2016 May 08 ]

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]

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

Comment by richlv [ 2016 May 13 ]

same problem in monitoring -> web, too

Generated at Fri Apr 26 12:32:17 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.