Skip to content
This repository has been archived by the owner on Feb 3, 2020. It is now read-only.

Dashboard draws fail if API call takes too long. #17

Closed
aletson opened this issue Mar 3, 2015 · 3 comments
Closed

Dashboard draws fail if API call takes too long. #17

aletson opened this issue Mar 3, 2015 · 3 comments
Assignees

Comments

@aletson
Copy link
Owner

aletson commented Mar 3, 2015

Let's think about solutions here.

The best one that I can come up with immediately is to have each function (filterHostsAndOutput and getHostsFromZabbix) call each other, but I don't know what impact that would have if settings get changed (since filterHostsAndOutput is called to redraw the screen when settings change).

@eburgueno
Copy link
Collaborator

Would using proper AJAX (#3) have an impact in this?

@aletson
Copy link
Owner Author

aletson commented Mar 3, 2015

That's a really good question. I'm still convinced that #3 is mainly a result of my incompetence, but wracking my brains for a solution hasn't helped.

I think it would fix the problem, but could cause the dashboard to refresh at irregular intervals, since the dashboard simply refreshes as soon as the API call is finished. If there's variance in API calls...

Eh. Dunno. It's certainly something to think about. I'll spend some time in the indefinite future trying to get async to work properly; we'll see how it goes!

I definitely think if we do that, we have to call filterHostsAndOutput as a success callback from getHostsFromZabbix since regardless of async, filterHostsAndOutput requires data in the hosts variable.

Feel free to correct any incorrect assumptions! :)

@aletson aletson self-assigned this May 6, 2015
@aletson
Copy link
Owner Author

aletson commented Nov 27, 2015

Going to go ahead and close this. Excessive load times appears to be an issue only with multiple hundreds of hosts or with Zabbix 2.2.0, which had a bug where results would take progressively longer to return from calls to the API.

@aletson aletson closed this as completed Nov 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants