Hi Wakko, On Donnerstag, 13. Juni 2013, Wakko Warner wrote: > I didn't add this to the bug tracking. If you feel it's appropriate, go > ahead and add it.
absolutly, this seems like very worthwhile progres. Also, please reply to the bug and attach the code you currently have (and/or provide the link to a git repo which has it :) > Holger Levsen wrote: > > On Montag, 3. Juni 2013, Wakko Warner wrote: > > > I'll attach a patch when I'm ready. > > > > great! thanks for the update! > > > > > I should mention that I'm using version 2.1.1-1 for my testing. It > > > should apply to 2.0 as well when I'm finished. > > > > np. > > > > feel free to join #munin on oftc or freenode to chat with us. > > I'm still playing with it. I noticed that dynazoom didn't work with IE8 or > older at all. I've also found some other problems with it. I'm fixing > those first and then adding the stuff in that I wrote. My original worked, > but I wasn't pleased with it yet. > > What I've added is this so far: > munin-cgi-graph: > Added filename length checking and use a hash if name too long. > Added field and nototal parameter from query string. > GraphOld.pm: > Added --nototal to prevent totals from being shown in graph. > Added --field to show only this field (multiples permitted) > I tried to make it work like --service but it didn't work > like that. > Added code to work with the options. > One bug: if a non-stack field was skipped and the next was a stack, > the stack is converted to line1 instead of what the > non-stack field was. I'll fix this. > The graph colors for the field are not effected. > HTMLConfig.pm: > Generate a list of fields with labels for the zoom urls. > Add the field list and hastotal options to zoom urls. > > I didn't change a whole lot in the perl code. The dynazoom.html has been > the focus of most of my attention (and the fact I'm weak in the web design > dept and especially JS) > > The features of the dynazoom (and I have this working, just need to > refactor): > 1. Checkboxes to enable/disable fields, default is all enabled. > 2. Radio button for the total field (only if hastotal=1). > 3. When a field gets unchecked, a small grey bar will be placed over the > field in the image. > 4. Fields in the image, including total, can be clicked on and will toggle > the checkbox for that field. > 5. If the scale is 1 or less, forbid zooming. At this point, each pixel > (or vertical line in the graph) is 1 second. I understand there is now a > way for munin to record with less than a 300 second granularity. I've not > seen how this works. > 6. Fix problems with IE8. I don't have access to anything older. It > should work in older versions. IE9 and 10 worked fine. I'm developing > this with Firefox 0.9, 3.6, 21 and chromium 26. So far all those work > fine. > > If you're interested in the perl patch, I can send it to you. The dynazoom > felt like spagetti and I have a few versions laying around. > > One bug in the stuff for the dynazoom.html is the fact that the graph > placement in the image is hard coded. I don't know if there is any > possibility to get the information that rrd graphv displays to be included > in the URL. thanks & cheers, Holger
signature.asc
Description: This is a digitally signed message part.