ZABBIX FEATURE REQUESTS

expose access to templated screens

Details

  • Zabbix ID:
    Reviewed 2.0

Description

new feature - templated screens - should be exposed to users in a better way than the mostly-unusable "monitoring -> hosts" page.

specification at https://zabbix.org/wiki/Docs/specs/ZBXNEXT-1066

Issue Links

Activity

Hide
Oleksiy Zagorskyi added a comment - - edited

According to specification: "Monitoring -> Hosts should be removed" (repeated twice)
What it means? Whole page Monitoring -> Hosts will be removed? or only the column "Screens" ?

<richlv> whole page is removed and it's functionality provided in other pages

<zalex> ok. then how about the idea about ability to see hosts (agent, snmp, ipmi, jmx) Availability for not "Zabbix Admin" users, i.e. for "Zabbix User" ?
It will be dropped?

<richlv>, yes, host availability for non-admin users won't be available
a) it probably appeared there only because that page was created as a copy of the configuration page
b) host availability is confusing enough for admin users. it is not ready to be exposed to users
c) some time ago it was decided (but not really documented, as far as i know) that host availability is more of an internal, admin-level thing and should not be exposed to users. this decision might be revisited later.... but later

<zalex> Many thanks for explanation! Now I haven't any objection.
The question CLOSED

Show
Oleksiy Zagorskyi added a comment - - edited According to specification: "Monitoring -> Hosts should be removed" (repeated twice) What it means? Whole page Monitoring -> Hosts will be removed? or only the column "Screens" ? <richlv> whole page is removed and it's functionality provided in other pages <zalex> ok. then how about the idea about ability to see hosts (agent, snmp, ipmi, jmx) Availability for not "Zabbix Admin" users, i.e. for "Zabbix User" ? It will be dropped? <richlv>, yes, host availability for non-admin users won't be available a) it probably appeared there only because that page was created as a copy of the configuration page b) host availability is confusing enough for admin users. it is not ready to be exposed to users c) some time ago it was decided (but not really documented, as far as i know) that host availability is more of an internal, admin-level thing and should not be exposed to users. this decision might be revisited later.... but later <zalex> Many thanks for explanation! Now I haven't any objection. The question CLOSED
Hide
Pavels Jelisejevs added a comment - - edited

(1) [DOCS]

Describe the JS translation implementation.

<pavels> RESOLVED http://www.zabbix.com/documentation/2.0/contrib/frontend_localisation?&#translatable_strings

<Vedmak> Originally, strings that should be translated w"H"ere kept in...

<pavels> Fixed.

<pavels> I guess it can be CLOSED now.

Show
Pavels Jelisejevs added a comment - - edited (1) [DOCS] Describe the JS translation implementation. <pavels> RESOLVED http://www.zabbix.com/documentation/2.0/contrib/frontend_localisation?&#translatable_strings <Vedmak> Originally, strings that should be translated w"H"ere kept in... <pavels> Fixed. <pavels> I guess it can be CLOSED now.
Hide
Oleksiy Zagorskyi added a comment - - edited

(2) [GUI] Dev branch briefly tested.
In the spec a term "Host screens" is used but I see just "Screens" in popup menus. I think the word "Host" in this case is very important.

<pavels> RESOLVED.
<zalex> Thanks. CLOSED

Show
Oleksiy Zagorskyi added a comment - - edited (2) [GUI] Dev branch briefly tested. In the spec a term "Host screens" is used but I see just "Screens" in popup menus. I think the word "Host" in this case is very important. <pavels> RESOLVED. <zalex> Thanks. CLOSED
Hide
Alexey Fukalov added a comment - - edited

(3) [GUI]
Configuration -> It services seems broken in branch

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (3) [GUI] Configuration -> It services seems broken in branch <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(4) [GUI]
If script confirmation message contains single quote popup menu is broken.

<pavels> RESOLVED.

<Vedmak> Now menu is working, but if script confirmation contains double quotes it's now working and script is not executed.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (4) [GUI] If script confirmation message contains single quote popup menu is broken. <pavels> RESOLVED. <Vedmak> Now menu is working, but if script confirmation contains double quotes it's now working and script is not executed. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(5) [GUI]
As js translation helper you added function _, though it's similar to PHP functioon i'd strongly suggest to change it as there is quite popular js library underscore.js which utilizes same name. Also i think one letter function names should be preserved for some libraries.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (5) [GUI] As js translation helper you added function _, though it's similar to PHP functioon i'd strongly suggest to change it as there is quite popular js library underscore.js which utilizes same name. Also i think one letter function names should be preserved for some libraries. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(6) [GUI]
I suggest move code for // fetch scripts for the host JS menu and // host JS menu link to separate function i saw it at least 3 times.

<pavels> RESOLVED.

<Vedmak> One place left in latest.php:546.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (6) [GUI] I suggest move code for // fetch scripts for the host JS menu and // host JS menu link to separate function i saw it at least 3 times. <pavels> RESOLVED. <Vedmak> One place left in latest.php:546. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(7) [GUI]
Maybe it's possible not to pass groupid to popup menu in order to decrease sql count.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (7) [GUI] Maybe it's possible not to pass groupid to popup menu in order to decrease sql count. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(8) [GUI]
In search.php in profile names like "CProfile::get('web.dashboard.hats.search_hostgroup.state', true)" word "dashboard" should be removed.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (8) [GUI] In search.php in profile names like "CProfile::get('web.dashboard.hats.search_hostgroup.state', true)" word "dashboard" should be removed. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(9) [GUI]
In host_screen.php GET parameter "groupid" seems not used, so it should be removed from validation and all links to host_screen.php.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (9) [GUI] In host_screen.php GET parameter "groupid" seems not used, so it should be removed from validation and all links to host_screen.php. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(10) [GUI]
Please review my changes in rev. 24818.

<pavels> Thnx, CLOSED.

Show
Alexey Fukalov added a comment - - edited (10) [GUI] Please review my changes in rev. 24818. <pavels> Thnx, CLOSED.
Hide
Oleksiy Zagorskyi added a comment - - edited

(11) For the record: ZBXNEXT-754 (Reviewed 2.0, btw) could be be resolved if add context menu to "overview" page too.

<pavels> We'll add it, but only when the hosts are displayed in the left column.

<pavels> RESOLVED.
<zalex> Thanks. TESTED, it works.

<Vedmak> CLOSED

Show
Oleksiy Zagorskyi added a comment - - edited (11) For the record: ZBXNEXT-754 (Reviewed 2.0, btw) could be be resolved if add context menu to "overview" page too. <pavels> We'll add it, but only when the hosts are displayed in the left column. <pavels> RESOLVED. <zalex> Thanks. TESTED, it works. <Vedmak> CLOSED
Hide
Pavels Jelisejevs added a comment - - edited

(12) We've decided to also add the menu to the event details page.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Pavels Jelisejevs added a comment - - edited (12) We've decided to also add the menu to the event details page. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Pavels Jelisejevs added a comment - - edited

(13) All of the refactoring, that was done in ZBX-4518 will be merged into this branch.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Pavels Jelisejevs added a comment - - edited (13) All of the refactoring, that was done in ZBX-4518 will be merged into this branch. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Oleksiy Zagorskyi added a comment - - edited

(14) r24900. From any page when I try to open "Latest data" I receive:
Invalid argument supplied for foreach() [include/forms.inc.php:2955]

<pavels> RESOLVED.
<zalex> TESTED. The problem is gone.

<Vedmak> CLOSED

Show
Oleksiy Zagorskyi added a comment - - edited (14) r24900. From any page when I try to open "Latest data" I receive: Invalid argument supplied for foreach() [include/forms.inc.php:2955] <pavels> RESOLVED. <zalex> TESTED. The problem is gone. <Vedmak> CLOSED
Hide
Alexey Fukalov added a comment - - edited

(15) [GUI]
Checkbox selection in hosts list is not saved after page refresh.

<pavels> RESOLVED.

<Vedmak> CLOSED

Show
Alexey Fukalov added a comment - - edited (15) [GUI] Checkbox selection in hosts list is not saved after page refresh. <pavels> RESOLVED. <Vedmak> CLOSED
Hide
Pavels Jelisejevs added a comment - - edited

Merged to trunk r24986.

Could someone review the docs in (1)?

<Vedmak> looks good!

Show
Pavels Jelisejevs added a comment - - edited Merged to trunk r24986. Could someone review the docs in (1)? <Vedmak> looks good!
Hide
Alexei Vladishev added a comment -

I believe it has been merged into trunk, revision 25005.

Show
Alexei Vladishev added a comment - I believe it has been merged into trunk, revision 25005.
Hide
Pavels Jelisejevs added a comment -

According to Vedmak's comment the docs are ok, so I guess I it can be closed now.

Show
Pavels Jelisejevs added a comment - According to Vedmak's comment the docs are ok, so I guess I it can be closed now.
Hide
Jens Berthold added a comment -

Maybe this issue should be reopened.

The currently available ways (Zabbix 1.9.9) to access a template based screen for a specific host (independent of currently active triggers or events) are the following:
1. Monitoring -> Overview -> Popup of Host
--> This access is only possible if the hosts are listed on the left side. The page is very slow if you have many hosts/items/triggers.
2. Monitoring -> Latest data
--> Only if there is no host filter, but then every host is listed several times for every application defined (no good overview)
3. via Search
--> OK, but you have to type

All other ways are only available if there are active triggers etc.

I read https://zabbix.org/wiki/Docs/specs/ZBXNEXT-1066 and I think the current ways of access are not intuitive and user friendly.
All other things like items, graphs, maps, "global" screens etc. can be accessed via menu. But template based screens can't.

A very intuitive way for me would be to integrate links to these screens under "Monitoring - Screens".
There you could show them in the currently available drop down field beneath the "global" screens (maybe with a special marker, or the template name in brackets).
When a template based screen is selected, there could be a second drop down field "Host" with all hosts that have the selected screen.

What do you say?

Show
Jens Berthold added a comment - Maybe this issue should be reopened. The currently available ways (Zabbix 1.9.9) to access a template based screen for a specific host (independent of currently active triggers or events) are the following: 1. Monitoring -> Overview -> Popup of Host --> This access is only possible if the hosts are listed on the left side. The page is very slow if you have many hosts/items/triggers. 2. Monitoring -> Latest data --> Only if there is no host filter, but then every host is listed several times for every application defined (no good overview) 3. via Search --> OK, but you have to type All other ways are only available if there are active triggers etc. I read https://zabbix.org/wiki/Docs/specs/ZBXNEXT-1066 and I think the current ways of access are not intuitive and user friendly. All other things like items, graphs, maps, "global" screens etc. can be accessed via menu. But template based screens can't. A very intuitive way for me would be to integrate links to these screens under "Monitoring - Screens". There you could show them in the currently available drop down field beneath the "global" screens (maybe with a special marker, or the template name in brackets). When a template based screen is selected, there could be a second drop down field "Host" with all hosts that have the selected screen. What do you say?
Hide
Jens Berthold added a comment -

I just saw what Alexei Vladishev wrote in https://support.zabbix.com/browse/ZBX-4120:
"The templated screens [...] are not supposed to be in Monitoring->Screens."

My question: why aren't they supposed to be there?

Show
Jens Berthold added a comment - I just saw what Alexei Vladishev wrote in https://support.zabbix.com/browse/ZBX-4120: "The templated screens [...] are not supposed to be in Monitoring->Screens." My question: why aren't they supposed to be there?
Hide
richlv added a comment -

because the current design doesn't really allow to incorporate them nicely.
your suggested way might work, but i suspect it might be confusing for users...
in the future a proper host-oriented monitoring section might be introduced, and this would hold links to host screens as well

Show
richlv added a comment - because the current design doesn't really allow to incorporate them nicely. your suggested way might work, but i suspect it might be confusing for users... in the future a proper host-oriented monitoring section might be introduced, and this would hold links to host screens as well
Hide
Jens Berthold added a comment -

@richlv: I would be very pleased if you could answer the following 3 questions:
Why do you think my suggested way would be confusing?
We have a customer who is just starting to use Zabbix - can you give an advice about the preferred way to select a particular templated screen?
Do you think I should open a feature request with my ideas above to have a discussion platform?

Show
Jens Berthold added a comment - @richlv: I would be very pleased if you could answer the following 3 questions: Why do you think my suggested way would be confusing? We have a customer who is just starting to use Zabbix - can you give an advice about the preferred way to select a particular templated screen? Do you think I should open a feature request with my ideas above to have a discussion platform?
Hide
richlv added a comment -

1) it might be unclear how those entries differ; also, it would not scale to bigger setups (think thousands of hosts linked to same template)
2) i'd probably go with global search for now;
3) as noted, i don't think current situation is perfect, and in future a host-centric monitoring page is sort of planned. it would include host related info like ip, dns, some trigger status/item values, graphs and, of course, templated screens. but that's not for 2.0, but for some release after that

Show
richlv added a comment - 1) it might be unclear how those entries differ; also, it would not scale to bigger setups (think thousands of hosts linked to same template) 2) i'd probably go with global search for now; 3) as noted, i don't think current situation is perfect, and in future a host-centric monitoring page is sort of planned. it would include host related info like ip, dns, some trigger status/item values, graphs and, of course, templated screens. but that's not for 2.0, but for some release after that
Hide
richlv added a comment -

interestingly, this development caused a regression with map context menus when french locale is selected : ZBX-5355

Show
richlv added a comment - interestingly, this development caused a regression with map context menus when french locale is selected : ZBX-5355
Hide
richlv added a comment -

in 2.2, host "inventory" page links to templated screens as well

Show
richlv added a comment - in 2.2, host "inventory" page links to templated screens as well

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: