Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Graph Management to show source type #1670

Closed
TheNetworkIsDown opened this issue Jun 15, 2018 · 6 comments
Closed

Enhance Graph Management to show source type #1670

TheNetworkIsDown opened this issue Jun 15, 2018 · 6 comments
Labels
question A question not a bug

Comments

@TheNetworkIsDown
Copy link
Contributor

Graph Management dialog shows template names that don't correspond to anything.

Also, the template names shown when using "Change graph template" don't correspond to any actual graph template either.

This is Cacti 1.1.38 upgraded from early 1.x. (Not sure if this bug existed previously, as only currently I'm again intensively using Cacti.)

image

@TheNetworkIsDown
Copy link
Contributor Author

I've another example.

I've got two graphs
"web02 - Disk Space - /dev/sda1"
and
"data02 - Used Space - /"

and for both the GUI refers to a template called "Available Disk Space".

graph_templates_graph -> title_cache = "web02 - Disk Space - /dev/sda1" --> graph_template_id 21
graph_templates: graph_template_id = 21 --> Unix - Available Disk Space

graph_templates_graph -> title_cache = "data02 - Used Space - /" --> graph_template_id 26
graph_templates: graph_template_id = 26 --> Host MIB - Available Disk Space

For both items the GUI however shows a template called "Available Disk Space" which seems to be a string occuring in the "snmp_query_graph" table in the "name" column, where it refers to graph_template_id 21 and 26, respectively....

I've tried to reproduce in a vanilla 1.2.0 install where I added disk space checks to localhost. While there are no duplicates (because the new install has less queries configured), it is also not showing the actual template name, but the stuff from the snmp_queries table.....

@cigamit
Copy link
Member

cigamit commented Jun 16, 2018

In Cacti 1.x, for Data Queries, we use the Data Query associated Graph Template name and not the actual Graph Template name. In prior versions of Cacti it was very easy for the names to be different as there was no helper function. This is a change from Cacti 0.x releases.

@cigamit cigamit added the question A question not a bug label Jun 16, 2018
@TheNetworkIsDown
Copy link
Contributor Author

That is very confusing. I guess it is a consequence of retrofitting the data queries into the existing architecture at some point.

How can we make this more understandable to newbies and existing users alike? I propose the following:

  1. The top of the page containing the "Template" dropdown (and all others places reading "Template" where actually "Template/Query" is meant, like table headers etc.) should be changed and read what it is: "Template/Query" (or a new description should be devised).

  2. "Templates" derived indirectly from Queries should be displayed differently than those derived directly from Data Input Methods. I propose a simple measure: adding for example an asterisk (*) behind those derived from queries. At least that would help distinguish what is what. You could even add a tooltip (text appears when hovering the element) displaying the actual graph template behind that associated query name. I guess the data is there, we just need to display it.

What do you think?

This really needs to be discussed.

@cigamit
Copy link
Member

cigamit commented Jun 17, 2018

Yea, having some visual indicator that it's Data Query based vs. a Raw Template sounds like a good idea. Yes, and it's a little confusion, I have to admit that.

@netniV netniV changed the title Graph Management dialog shows bad template names Improve Graph Management to show type of source for a graph Jul 19, 2018
netniV added a commit to netniV/cacti that referenced this issue Jul 19, 2018
…raph

This commit adds functionality as requested in Cacti#1670 by adding an extra column to the graph management page to display the source type of a graph ('None', 'Data Query', 'Template')
@netniV
Copy link
Member

netniV commented Jul 19, 2018

I have made an update for v1.2 that will show the type of source for a given graph:

image

cigamit pushed a commit that referenced this issue Jul 21, 2018
…graph (#1725)

* Feature#1670: Improve Graph Management to show type of source for a graph

This commit adds functionality as requested in #1670 by adding an extra column to the graph management page to display the source type of a graph ('None', 'Data Query', 'Template')

* Minor update to make column sortable
@netniV
Copy link
Member

netniV commented Jul 21, 2018

We have now merged this into 1.2's development.

@netniV netniV closed this as completed Jul 21, 2018
cigamit added a commit that referenced this issue Jul 21, 2018
Change alignment, add some i18n
@netniV netniV changed the title Improve Graph Management to show type of source for a graph Enhance Graph Management to show source type Dec 31, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question A question not a bug
Projects
None yet
Development

No branches or pull requests

3 participants