Merge ~paelzer/update-notifier/+git/update-notifier:lp-1893716-reduce-login-overhead-JAMMY-nongu into update-notifier:master

Proposed by Christian Ehrhardt 
Status: Merged
Merge reported by: Christian Ehrhardt 
Merged at revision: 4a228abd8dd10cf7174bd5bb7bdfce2cd17d0a59
Proposed branch: ~paelzer/update-notifier/+git/update-notifier:lp-1893716-reduce-login-overhead-JAMMY-nongu
Merge into: update-notifier:master
Diff against target: 30 lines (+12/-0)
2 files modified
debian/95-hwe-eol (+6/-0)
debian/changelog (+6/-0)
Reviewer Review Type Date Requested Status
Brian Murray Pending
Review via email: mp+418122@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
Brian Murray (brian-murray) wrote :

This looks fine but I wonder if we extended the time frame see my inline comment.

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

Hi Brian,
I didn't want to be too aggressive with my proposal ...

On one hand we have the impact that is rather low after the change, no matter if we use 1h or 1d.

- manual login: humans are slow, makes no big difference even if checking every time.
- auto-logins: before any rate limiting this could go into the millions per hour which now get one check \o/, bumping this up to 1d doesn't make too much difference

On the other hand you are right it does try to tell you if

a) one of your -hwe packages is no more supported (think hirsute based but hirsute EOL)
b) about matching replacements

None of that changes on an hourly base, a day will work just as much indeed.

I'll bumped it to a day.
Will you merge and upload this later or should I?

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

I heard that you are away, given the time in the cycle and that you are generally ok with it I did an upload now and merged it in the repository.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/95-hwe-eol b/debian/95-hwe-eol
2index 3b4bd13..92d0c7d 100755
3--- a/debian/95-hwe-eol
4+++ b/debian/95-hwe-eol
5@@ -1,5 +1,11 @@
6 #!/bin/sh
7
8+# this stamp is created and updated by /usr/lib/update-notifier/update-motd-hwe-eol
9+stamp="/var/lib/update-notifier/hwe-eol"
10+
11+# do not try to refresh this more than once per day
12+[ -z "$(find "$stamp" -newermt 'now-1 days' 2> /dev/null)" ] && exit 0
13+
14 if [ -x /usr/lib/update-notifier/update-motd-hwe-eol ]; then
15 exec /usr/lib/update-notifier/update-motd-hwe-eol
16 fi
17diff --git a/debian/changelog b/debian/changelog
18index 731ead1..6184ddb 100644
19--- a/debian/changelog
20+++ b/debian/changelog
21@@ -1,3 +1,9 @@
22+update-notifier (3.192.54) jammy; urgency=medium
23+
24+ * d/95-hwe-eol: do not update eol data more than once per hour (LP: #1893716)
25+
26+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Wed, 30 Mar 2022 12:20:47 +0200
27+
28 update-notifier (3.192.53) jammy; urgency=medium
29
30 * data/list-oem-metapackages: Complete the transition to apt_pkg which was

Subscribers

People subscribed via source and target branches