[SOLVED] cacti_graph_id=0

[sh0x]

After an upgrade from weathermap .96a to to .97test4 my maps are setting cacti_graph_id=0 so the overlibgraphs break. I'm using cacti 0.8.7e with dsstats 1.1-1 and boost 1.5 with weathermap and rrd_use_poller_output set to 1.

DEBUG: [add_note@Weathermap.class.php:106] [Map 28] Webseal: Adding note cacti_graph_id='0'

Everything else seems to work ok and the other hints set correctly. I copy back .96a and its all good. What am I missing? Thanks.

[Howie]

After an upgrade from weathermap .96a to to .97test4 my maps are setting cacti_graph_id=0 so the overlibgraphs break. I'm using cacti 0.8.7e with dsstats 1.1-1 and boost 1.5 with weathermap and rrd_use_poller_output set to 1.

DEBUG: [add_note@Weathermap.class.php:106] [Map 28] Webseal: Adding note cacti_graph_id='0'

Everything else seems to work ok and the other hints set correctly. I copy back .96a and its all good. What am I missing? Thanks.

Are you using dsstats or rrdtool for your TARGETs?

[Howie]

I think I have found this one. It affects both poller_output and dsstats support.

The fix is simple though.

Change the $ldi to $local_data_id on the very end of line 45 of lib/ds-common.php

This will be in the next release (I expect 0.97a in a week or so)

[sh0x]

Finally got back to this, thanks Howie that seemed to do the trick.. though I see you made an official release of test4 yesterday so I should probably grab it at this point.

Thanks again Howie!

[sh0x]

Just tried the official .97 release, looks like the change is still needed to ds-common.php for $ldi to $local_data_id.

[Howie]

This is actually fixed in svn, and will be in 0.97a. I think I have cleared my reported buglist now, so 0.97a should be very soon, with about a dozen bugs fixed and a few small features.

(test4 came out in November, 0.97 came out Jan 18th, 3 days before I said 'I have found this one'... so neither would have had the fix)