Details

      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

            • Assignee:
              Alexei Vladishev
              Reporter:
              richlv
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: