Merge lp:~davewalker/ubuntu/natty/munin/uppercase-hostname-mismatch into lp:ubuntu/natty/munin

Proposed by Dave Walker
Status: Merged
Merge reported by: Daniel Holbach
Merged at revision: not available
Proposed branch: lp:~davewalker/ubuntu/natty/munin/uppercase-hostname-mismatch
Merge into: lp:ubuntu/natty/munin
Diff against target: 58 lines (+38/-0)
3 files modified
debian/changelog (+9/-0)
debian/patches/series (+1/-0)
debian/patches/uppercase-hostname-mismatch.patch (+28/-0)
To merge this branch: bzr merge lp:~davewalker/ubuntu/natty/munin/uppercase-hostname-mismatch
Reviewer Review Type Date Requested Status
Ubuntu Sponsors Pending
Review via email: mp+47391@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2010-10-15 14:16:41 +0000
+++ debian/changelog 2011-01-25 13:47:10 +0000
@@ -1,3 +1,12 @@
1munin (1.4.5-3ubuntu2) natty; urgency=low
2
3 * debian/patches/uppercase-hostname-mismatch.patch: Correctly match the
4 hostnames in munin-node, fixing a mismatch when the hostname contains
5 an uppercase character, resolving the failed discovery of plugins to
6 match. Patch courtesy of upstream svn trunk. (LP: #707099)
7
8 -- Dave Walker (Daviey) <DaveWalker@ubuntu.com> Tue, 25 Jan 2011 13:31:14 +0000
9
1munin (1.4.5-3ubuntu1) natty; urgency=low10munin (1.4.5-3ubuntu1) natty; urgency=low
211
3 * Merge from debian unstable. (LP: #661505) Remaining changes:12 * Merge from debian unstable. (LP: #661505) Remaining changes:
413
=== modified file 'debian/patches/series'
--- debian/patches/series 2010-10-15 14:16:41 +0000
+++ debian/patches/series 2011-01-25 13:47:10 +0000
@@ -3,3 +3,4 @@
3102-snort-bashism.patch3102-snort-bashism.patch
4103-ejabberd-cli.patch4103-ejabberd-cli.patch
5237-hddtemp_smartctl-sata-detect.patch5237-hddtemp_smartctl-sata-detect.patch
6uppercase-hostname-mismatch.patch
67
=== added file 'debian/patches/uppercase-hostname-mismatch.patch'
--- debian/patches/uppercase-hostname-mismatch.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/uppercase-hostname-mismatch.patch 2011-01-25 13:47:10 +0000
@@ -0,0 +1,28 @@
1Description: Correctly match the hostnames in munin-node, fixing a mismatch
2 when the hostname contains an uppercase character, resolving the failed
3 discovery of plugins to match.
4Origin: Cherry picked from upstream svn trunk, subset of changeset 3404.
5Bug: http://munin-monitoring.org/ticket/952
6Bug-Ubuntu: https://launchpad.net/bugs/707099
7--- a/node/lib/Munin/Node/Server.pm
8+++ b/node/lib/Munin/Node/Server.pm
9@@ -88,6 +88,10 @@
10 my $node = $config->{sconf}{$service}{host_name}
11 || (split /\s+/, ($host_name || ''))[1]
12 || $config->{fqdn};
13+
14+ # hostname checks are case in-sensitive,
15+ # so store everything in lowercase
16+ $node = lc($node);
17
18 # hostname checks are case insensitive, so store everything in lowercase
19 $node = lc($node);
20@@ -171,7 +175,7 @@
21
22 logger ("DEBUG: Running command \"$_\".") if $config->{DEBUG};
23 if (/^list\s*([0-9a-zA-Z\.\-]+)?/i) {
24- _list_services($session, $1);
25+ _list_services($session, lc($1));
26 }
27 elsif (/^cap\s?(.*)/i) {
28 _negotiate_session_capabilities($session, $1);

Subscribers

People subscribed via source and target branches

to all changes: