Merge ~paelzer/ubuntu/+source/irqbalance:lp-2046470-fix-log-spam-NOBLE into ubuntu/+source/irqbalance:ubuntu/noble-devel

Proposed by Christian Ehrhardt 
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merged at revision: 2529baa2b3478c6937652898f47e8a471995c08a
Proposed branch: ~paelzer/ubuntu/+source/irqbalance:lp-2046470-fix-log-spam-NOBLE
Merge into: ubuntu/+source/irqbalance:ubuntu/noble-devel
Diff against target: 86 lines (+54/-1)
4 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch (+44/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Athos Ribeiro (community) Approve
Canonical Server Reporter Pending
Review via email: mp+458083@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Thanks, Christian!

This looks reasonable to me (+1).

The only comment I have here is regarding the "Origin" DEP3 header. Since upstream did not merge (or comment) the patch yet, it seems that the PR or commit URL should go in the "Forwarded" field. From the "Origin" field definition, removing the "upstream, " part of it should be enough to comply to the definition as well.

The reason I am pointing to this header is that it could lead to another person touching the package to overlook that patch in case it is rejected or changed by upstream.

I will not block this on that header though. Once you fix it or disagree with the assessment, go ahead and upload this one :)

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: paelzer, athos-ribeiro
Uploaders: paelzer, athos-ribeiro
MP auto-approved

review: Approve
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Thanks, uploaded

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index c192dd9..6cd1abd 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+irqbalance (1.9.3-2ubuntu1) noble; urgency=medium
7+
8+ * d/p/lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch:
9+ reduce log spam by debug message (LP: #2046470)
10+
11+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Fri, 05 Jan 2024 14:30:02 +0100
12+
13 irqbalance (1.9.3-2) unstable; urgency=medium
14
15 * d/p/filter-console-only-output-when-using-journal-mode.debpatch:
16diff --git a/debian/control b/debian/control
17index 8524c3d..38ebba3 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -1,7 +1,8 @@
21 Source: irqbalance
22 Section: utils
23 Priority: optional
24-Maintainer: Paride Legovini <paride@debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: Paride Legovini <paride@debian.org>
27 Build-Depends:
28 debhelper-compat (= 13),
29 dh-exec,
30diff --git a/debian/patches/lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch b/debian/patches/lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch
31new file mode 100644
32index 0000000..4466e53
33--- /dev/null
34+++ b/debian/patches/lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch
35@@ -0,0 +1,44 @@
36+From a27df6e7a29fad744494a5a1432bb3ed6799cb7d Mon Sep 17 00:00:00 2001
37+From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
38+Date: Fri, 5 Jan 2024 14:11:49 +0100
39+Subject: [PATCH] Stop log spam due to debug message from scan
40+
41+Currently systems encounter log spam every 10 seconds due to a
42+message meant for debugging now being forwarded to journald.
43+Users get three empty lines followed by 78 "-" without any further
44+info what this is about.
45+
46+Due to its original debug-only purpose the message also grabs
47+a lot of screen-estate. So while touching it anyway let us also make
48+it a bit less agressive (no three lead line breaks, less width),
49+now it is like:
50+ ----- Enter: scan -----
51+
52+Fixes: #292
53+
54+Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
55+
56+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/irqbalance/+bug/2046470
57+Origin: upstream, https://github.com/Irqbalance/irqbalance/pull/293/commits/a27df6e7a29fad744494a5a1432bb3ed6799cb7d
58+Last-Update: 2024-01-05
59+
60+---
61+ irqbalance.c | 2 +-
62+ 1 file changed, 1 insertion(+), 1 deletion(-)
63+
64+diff --git a/irqbalance.c b/irqbalance.c
65+index f5f2c51..d38c153 100644
66+--- a/irqbalance.c
67++++ b/irqbalance.c
68+@@ -289,7 +289,7 @@ gboolean force_rescan(gpointer data __attribute__((unused)))
69+
70+ gboolean scan(gpointer data __attribute__((unused)))
71+ {
72+- log(TO_CONSOLE, LOG_INFO, "\n\n\n-----------------------------------------------------------------------------\n");
73++ log(TO_CONSOLE, LOG_DEBUG, "----- Enter: %s -----\n", __func__);
74+ clear_work_stats();
75+ parse_proc_interrupts();
76+
77+--
78+2.34.1
79+
80diff --git a/debian/patches/series b/debian/patches/series
81index 41e93c6..27b1cf5 100644
82--- a/debian/patches/series
83+++ b/debian/patches/series
84@@ -1 +1,2 @@
85 filter-console-only-output-when-using-journal-mode.debpatch
86+lp-2046470-Stop-log-spam-due-to-debug-message-from-scan.patch

Subscribers

People subscribed via source and target branches