-
Notifications
You must be signed in to change notification settings - Fork 184
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
Munin mod_tile_* hardcoded URL #418
Comments
Yes, I see that. Although Munin usage has been discontinued (openstreetmap/chef@1751b31) by OSM, perhaps something like this master...hummeltech:MuninScripts will do the trick for you. Thanks |
I understand the principle of not maintaining code here that is not actually being used by OSMF, but it's a pity nevertheless. The plugin has been packaged by Debian and Fedora for a long time, so its removal will leave orphans all over the place. For my part, for a new installation, I dismissed prometheus and turned to munin as soon as I saw this. Do you know whether anyone else is taking over the munin plugin maintainance?
Thank you. Especially for patching something that has already been obsoleted. |
Sorry about that, I didn't mean to imply that these scripts were actually going to be removed from here as I'm sure there are still plenty of Munin users. The problem I was trying to convey is that I don't have much familiarity with Munin and that these scripts will likely not be getting many updates in the future. I will submit a pull request with the above patch for you to look over and test out and if the solution works for you, I'd be happy to merge it in. I can see absolutely no reason that it would break any existing deployments. Thanks |
There is a lot of code here that is not used by the OSMF, I wouldn't say that's a huge consideration. |
The misunderstanding is entirely my fault. I just glanced at 1751b31 and, without even looking at what file that was and where, concluded that the munin plugins had already been removed. My only defence is that I hadn't had my morning coffee yet. I'll test the dynamically configured scripts and report back in a couple of days. |
All four munin mod_tile scripts use "data=$(wget -q http://localhost/mod_tile -O -)". There are many cases where this cannot work, e.g. "Listen 10.22.33.44" in httpd.conf, mod_tile running in a virtual host, mod_tile only being accessible by https, or https with a self-signed certificate.
Changing the wget command in the scripts is easy, but if they have been installed by rpm or deb such changes will be lost at the first update of the munin plugin. A much better solution would be to add a configuration file /etc/munin/plugin-conf.d/mod_tile and let the scripts read the URL and any additional wget arguments (e.g. --no-check-certificate) from there.
The text was updated successfully, but these errors were encountered: