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.

Sign up for GitHub

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

Jump to bottom

WeatherMap OverLib - cacti_graph_id [UNKNOWN] #142

Open
HerrDrSt opened this issue Mar 15, 2024 · 31 comments
Open

WeatherMap OverLib - cacti_graph_id [UNKNOWN] #142

HerrDrSt opened this issue Mar 15, 2024 · 31 comments

Comments

@HerrDrSt
Copy link

Having problems with this Weathermap for Cacti 1.x

It's not resolving {link:this:cacti_graph_id} - instead I get [UNKNOWN] - for the graph-ids.

================================================================
Graph OverLib Mouse Hover shows:

Link

https://IP-ADDRESS/cacti/graph.php?local_graph_id=[UNKNOWN]

Images

https:///IP-ADDRESS/cacti/graph_image.php?local_graph_id=[UNKNOWN]&rra_id=0&graph_height=125&graph_width=400&random=35352

https://IP-ADDRESS/cacti/graph_image.php?local_graph_id=[UNKNOWN]&rra_id=0&graph_height=125&graph_width=400&random=17813

===WeatherMap Global link configuration====================================

SET rrd_use_poller_output 1

TEMPLATE-only LINKs:

LINK DEFAULT
INFOURL {map:cacti_url}/graph.php?local_graph_id={link:this:cacti_graph_id}
OVERLIBGRAPH {map:cacti_url}/graph_image.php?local_graph_id={link:this:cacti_graph_id}&rra_id=0
BWLABEL bits
WIDTH 4
BWFONT 3
SET cacti_use_ifspeed 1

==========================================
Using the following versions:
[viewed from changelogs]

Cacti - 1.2.22
cacti-spine-1.2.24
Weathermap - 1.1

==images=======================================

overlib-fail-new-weathermap-unknown
graphs-on-old-weathermap-ok
@TheWitness
Copy link
Member

TheWitness commented Mar 15, 2024 •
edited
Loading

Was this working before? I'm guessing that this was true. I have to do a code review to figure this one out. Can you Email me your map configuration? Mail to thewitness at cacti dot net.

@TheWitness
Copy link
Member

@HerrDrSt, would you be up for a Zoom call to trace this? What Timezone are you in?

@HerrDrSt
Copy link
Author

HerrDrSt commented Mar 19, 2024 •
edited
Loading

Hallo @TheWitness

Yes, I would be willing to do a Zoom Call, my Timezone is +3 GMT.

Mayhaps when you see this message - we can agree on the appropriate time and you can send a link.

Regarding your first question:
✓ No, it was *not working before - from setup
✓ The screenshot I have shared of the working version is our older cacti with older weathermap 0.98

✓ I will mail you the map config soonest at that address you shared

PS: I am also getting lots of those implicit conversion, float to int errors when I build the weathermap

@TheWitness
Copy link
Member

Are you using the latest develop? GMT+3 is going to be hard for me tomorrow as I have some early morning meeting. Thursday morning my time EST (GMT+5) will work out. I'll look for your Email.

@HerrDrSt
Copy link
Author

I am using the following versions .
Are they compatible? Where can I get the latest develop?
At one point, I resorted to downloading individual php pages from this fork to update the ones in my server with latest files, but the error persisted.

+5GMT is difference of only 2 hours, that is workable, share zoom link day before when you are ready.

[viewed from changelogs]

Cacti - 1.2.22
cacti-spine-1.2.24
Weathermap - 1.1

@TheWitness
Copy link
Member

Sorry I tied to type -5 and somehow my fingers let me down. We are 8 hours apart. How about Saturday in my morning (EST)?

@TheWitness
Copy link
Member

I have mainly usage questions so that I can verify the current logic to ensure the process is working. Make sure you are on the latest develop too.

@TheWitness
Copy link
Member

TheWitness commented Mar 20, 2024 •
edited
Loading

So, mainly, what config name in the config file represents the SNMP index?

@TheWitness
Copy link
Member

@HerrDrSt, I'm here in my office now if you read this and have a few minutes. We can get this issue resolved.

@HerrDrSt
Copy link
Author

HerrDrSt commented Mar 24, 2024 via email •
edited
Loading

@TheWitness
Copy link
Member

Back at the office now. I know it's 8:00pm your time, but are you still around?

@HerrDrSt

This comment was marked as off-topic.

Sign in to view
@TheWitness
Copy link
Member

How about 9:00 am EDT tomorrow? It's too late now for you.

@HerrDrSt
Copy link
Author

HerrDrSt commented Mar 31, 2024 via email •
edited
Loading

@TheWitness
Copy link
Member

TheWitness commented Mar 31, 2024 •
edited
Loading

I use Zoom if thats okay.

@HerrDrSt
Copy link
Author

HerrDrSt commented Mar 31, 2024 via email

TheWitness added a commit that referenced this issue Mar 31, 2024
@TheWitness
Fixing #142 - Fix Programmatic Links
e62d9dd
@TheWitness
Copy link
Member

Okay, two files to update. Please provide feedback. This was a shotgun approach.

@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 1, 2024 via email •
edited
Loading

@TheWitness
Copy link
Member

If you were running the latest develop, just replace the two files. If not, just replace them all.

@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 2, 2024 via email •
edited
Loading

@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 2, 2024

Hallo @TheWitness

I have just replaced the two files - and tested by rebuilding the
weathermap.

Still showing "UNKNOWN - graph-id" with a broken-image.

Please find attached screenshot of the error.

feedback_02042024_unknown

BR/
HerrDrSt

@TheWitness
Copy link
Member

Okay. I'll setup a local map. Busy week though.

@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 3, 2024 via email

@TheWitness
Copy link
Member

Anything in the Cacti log to speak of?

@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 7, 2024 via email

@TheWitness
Copy link
Member

It looks like you are not running the latest develop. A few of your files are out of date.

@HerrDrSt
Copy link
Author

Hallo, You mean Cacti itself or Weathermap?

@TheWitness
Copy link
Member

WeatherMap. You should not be getting those deprecation warnings any longer. I've confirmed in the code.

@TheWitness
Copy link
Member

@HerrDrSt , weird that I closed this. Can you update to the latest develop and see if it's fixed. If not, can you do a Zoom on Saturday?

@TheWitness TheWitness reopened this Apr 26, 2024
@HerrDrSt
Copy link
Author

HerrDrSt commented Apr 27, 2024 via email

@TheWitness
Copy link
Member

https://github.com/Cacti/plugin_weathermap.git

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TheWitness @HerrDrSt