Merge ~bryce/ubuntu/+source/logwatch:sru.1797836-bionic into ubuntu/+source/logwatch:ubuntu/bionic-devel

Proposed by Bryce Harrington
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: 4985770802e6e86928eaee8ef1d37b6dfad1b7f0
Merge reported by: Christian Ehrhardt 
Merged at revision: 4985770802e6e86928eaee8ef1d37b6dfad1b7f0
Proposed branch: ~bryce/ubuntu/+source/logwatch:sru.1797836-bionic
Merge into: ubuntu/+source/logwatch:ubuntu/bionic-devel
Diff against target: 63 lines (+41/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/ignore-ras-correctable-errors.patch (+31/-0)
debian/patches/series (+2/-0)
Reviewer Review Type Date Requested Status
Christian Ehrhardt  (community) Approve
Canonical Server packageset reviewers Pending
Canonical Server Pending
Review via email: mp+370585@code.launchpad.net

Description of the change

Fixes LP: #1797836 for Ubuntu bionic by adding a backported patch from
logwatch 7.5.0.

PPA with test package: https://launchpad.net/~bryce/+archive/ubuntu/logwatch-sru-1797836/+packages
  $ sudo add-apt-repository ppa:bryce/logwatch-sru-1797836
  $ sudo apt-get update
  $ sudo apt-get install logwatch

Test case:
  $ now=$(date +"%b %d %T")
  $ uptime=$(cat /proc/uptime | cut -d' ' -f1)
  $ fake_message="${now} ${HOSTNAME} kernel: [${uptime}] RAS: Correctable Errors collector initialized"
  $ echo "${fake_message}" >> /var/log/kern.log
  $ logwatch --service kernel --range Today

  With the current Ubuntu version, logwatch will display a report
  showing the RAS error.

  With the PPA version, logwatch should show no output.

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Current state:
 logwatch | 7.4.2-1ubuntu1 | xenial
 logwatch | 7.4.3+git20161207-2ubuntu1 | bionic
 logwatch | 7.4.3+git20161207-2ubuntu2 | cosmic
 logwatch | 7.5.0-1ubuntu1 | disco

Since we are not pushing the same version to Cosmic there is no need to extend it with per-release numbers.
This would do it:
  7.4.3+git20161207-2ubuntu1.1

All the rest (headers, patch, patch source, build, SRU, test) LGTM.
+1

You can fix up the version when sponsoring this yourself right?

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks for the review, tagged and uploaded:

$ git push pkg upload/7.4.3+git20161207-2ubuntu1.1
Counting objects: 11, done.
Delta compression using up to 6 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 1.98 KiB | 253.00 KiB/s, done.
Total 11 (delta 7), reused 0 (delta 0)
To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/logwatch
 * [new tag] upload/7.4.3+git20161207-2ubuntu1.1 -> upload/7.4.3+git20161207-2ubuntu1.1

$ dput ubuntu logwatch_7.4.3+git20161207-2ubuntu1.1_source.changes
Checking signature on .changes
gpg: /home/bryce/ubuntu/Logwatch/sru.1797836/logwatch_7.4.3+git20161207-2ubuntu1.1_source.changes: Valid signature from E603B2578FB8F0FB
Checking signature on .dsc
gpg: /home/bryce/ubuntu/Logwatch/sru.1797836/logwatch_7.4.3+git20161207-2ubuntu1.1.dsc: Valid signature from E603B2578FB8F0FB
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading logwatch_7.4.3+git20161207-2ubuntu1.1.dsc: done.
  Uploading logwatch_7.4.3+git20161207-2ubuntu1.1.debian.tar.xz: done.
  Uploading logwatch_7.4.3+git20161207-2ubuntu1.1_source.buildinfo: done.
  Uploading logwatch_7.4.3+git20161207-2ubuntu1.1_source.changes: done.
Successfully uploaded packages.

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

This is in -proposed, thereby setting merged

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 af5fcfc..ae1c30b 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,11 @@
6+logwatch (7.4.3+git20161207-2ubuntu1.18.04.1) bionic; urgency=medium
7+
8+ * Add d/patches/ignore-ras-correctable-errors.patch to fix false
9+ reporting of kernel error "RAS: Correctable Errors collector..."
10+ (LP: #1797836)
11+
12+ -- Bryce Harrington <bryce@canonical.com> Tue, 09 Jul 2019 15:39:57 -0700
13+
14 logwatch (7.4.3+git20161207-2ubuntu1) zesty; urgency=medium
15
16 * Merge from Debian testing. Remaining changes:
17diff --git a/debian/patches/ignore-ras-correctable-errors.patch b/debian/patches/ignore-ras-correctable-errors.patch
18new file mode 100644
19index 0000000..9c3d36c
20--- /dev/null
21+++ b/debian/patches/ignore-ras-correctable-errors.patch
22@@ -0,0 +1,31 @@
23+From ce8cd7421f34722b3c5411cb2544e34f627384cc Mon Sep 17 00:00:00 2001
24+From: Orion Poplawski <orion@nwra.com>
25+Date: Fri, 1 Jun 2018 11:23:56 -0600
26+Subject: [PATCH] [kernel] Ignore RAS: Correctable Errors collector initialized
27+ message
28+
29+Author: Orion Poplawski <orion@nwra.com>
30+Origin: upstream, https://sourceforge.net/p/logwatch/git/ci/ce8cd7421f34722b3c5411cb2544e34f627384cc
31+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/logwatch/+bug/1797836
32+Reviewed-By: Bryce Harrington <bryce@canonical.com>
33+Applied-Upstream: 7.5.0, commit:ce8cd7421f34722b3c5411cb2544e34f627384cc
34+Last-Updated: 2019-06-27
35+---
36+ scripts/services/kernel | 1 +
37+ 1 file changed, 1 insertion(+)
38+
39+diff --git a/scripts/services/kernel b/scripts/services/kernel
40+index 7c4ae26..9803c14 100755
41+--- a/scripts/services/kernel
42++++ b/scripts/services/kernel
43+@@ -85,6 +85,7 @@ while (defined(my $ThisLine = <STDIN>)) {
44+ # following now in iptables service
45+ or ($ThisLine =~ /^Packet log: .*PROTO=/)
46+ or ($ThisLine =~ /IN=.*OUT=.*SRC=.*DST=.*PROTO=/)
47++ or ($ThisLine =~ /RAS: Correctable Errors collector initialized/)
48+ # user specified ignore messages, lower cased
49+ or ($ThisLine =~ /$Ignore_messages/i)
50+ ) { # ignore the above strings
51+--
52+2.17.1
53+
54diff --git a/debian/patches/series b/debian/patches/series
55index ce06425..35fbd8b 100644
56--- a/debian/patches/series
57+++ b/debian/patches/series
58@@ -7,3 +7,5 @@
59 0007-dovecot-Ignore-Debug-output.patch
60 0008-postfix-Fix-unaligned-output.patch
61 0009-Revert-Changed-encoding-to-UTF-8-as-suggested-by-G-r.patch
62+
63+ignore-ras-correctable-errors.patch

Subscribers

People subscribed via source and target branches