Bug #16145
Upgrade Linux to 4.18.20
100%
Description
Subtasks
Related issues
Related to Tails - |
Resolved | 2018-10-25 | |
Related to Tails - |
Resolved | 2018-12-13 | |
Blocks Tails - |
Resolved | 2018-04-08 |
History
#1 Updated by intrigeri 2018-11-22 09:00:50
- blocks
Feature #15506: Core work 2018Q4: Foundations Team added
#2 Updated by intrigeri 2018-11-22 09:01:17
- related to
Bug #16073: Upgrade Linux to 4.19 added
#3 Updated by intrigeri 2018-11-22 09:04:17
- Assignee set to segfault
Hi segfault! For 3.11 it’s not clear yet whether we’ll have a 4.18.x with all the security fixes we want (and then we will need to do at least what this ticket is about) or whether we’ll need to go 4.19 (Bug #16073), hence this ticket. It’ll depend on when 4.19 is uploaded sid and whether it brings security fixes we want.
#4 Updated by intrigeri 2018-12-07 10:27:00
Hi segfault! The timing for getting this done in time for 3.11 is starting to be a bit tight, so I wonder: do you need help with this? If you do, please say so ASAP and I’ll make time for it :)
#5 Updated by anonym 2018-12-07 12:42:17
Worst case, I take this over tomorrow (2018-12-07) so there’s some minimal amount of time for QA at least.
#6 Updated by anonym 2018-12-08 15:50:12
- Assignee changed from segfault to anonym
anonym wrote:
> Worst case, I take this over tomorrow (2018-12-07) so there’s some minimal amount of time for QA at least.
Hah, it should have said 2018-12-08. Any way, taking this one over so it has a chance to get into Tails 3.11.
#7 Updated by anonym 2018-12-08 16:43:26
- Status changed from Confirmed to In Progress
Applied in changeset commit:tails|5f7db2ada323f04a8f25d81f16fe5c9975766e89.
#8 Updated by anonym 2018-12-08 23:07:56
- % Done changed from 0 to 30
- QA Check set to Ready for QA
- Feature Branch set to feature/16145-linux-4.18.20-2 feature/16145-linux-4.18.20-2+force-all-tests
Let’s see what jenkins thinks: https://jenkins.tails.boum.org/job/build_Tails_ISO_feature-16145-linux-4.18.20-2-force-all-tests/1/
#9 Updated by anonym 2018-12-09 10:26:31
- % Done changed from 30 to 40
First full test suite done: https://jenkins.tails.boum.org/job/test_Tails_ISO_feature-16145-linux-4.18.20-2-force-all-tests/1/
There are six failures. I didn’t bother looking at them yet as I started two more runs — I’ll triage failures later today once I have all three runs.
Cyril, I added you as a watcher so you will get a notification about this and can anticipate the imminent review’n’merge. It will be tight! Sadly I had a bit more trouble getting this branch (well, the APT suite) in order.
#10 Updated by anonym 2018-12-09 21:13:27
- Assignee changed from anonym to CyrilBrulebois
- % Done changed from 40 to 50
Look at these two runs of different branches with +force-all-tests:
- This ticket’s feature branch: https://jenkins.tails.boum.org/job/test_Tails_ISO_feature-16193-tor-browser-8.0.4-force-all-tests/3/
- Another feature branch: https://jenkins.tails.boum.org/job/test_Tails_ISO_feature-16193-tor-browser-8.0.4-force-all-tests/1/
They fail the exact same tests, which to me indicates that these tests are just broken. So IMHO we’re good re: tests.
On XMPP intrigeri offered to take this one over to lighten our RM’s burden tomorrow, just so that’s clear. I’ll still assign it to the RM so it cannot be forgotten!
#11 Updated by intrigeri 2018-12-10 08:19:55
> * This ticket’s feature branch: https://jenkins.tails.boum.org/job/test_Tails_ISO_feature-16193-tor-browser-8.0.4-force-all-tests/3/
That link is wrong, which makes me wonder if you looked at the correct test results (if not, this would invalidate your conclusions). Anyway, it’s part of the reviewer’s job to check test results themselves, so worries.
#12 Updated by intrigeri 2018-12-10 08:36:20
- Assignee changed from CyrilBrulebois to intrigeri
#13 Updated by intrigeri 2018-12-10 08:44:03
https://jenkins.tails.boum.org/view/Tails_ISO/job/test_Tails_ISO_feature-16145-linux-4.18.20-2-force-all-tests/4/cucumberTestReport/emergency-shutdown/tails-erases-memory-and-shuts-down-on-usb-boot-medium-removal_-persistent-data/ is a bit concerning but it happened only once in 5 runs, and we’ll need to check what’s going on in this area soon (Bug #16097), so I’ll ignore it for now. Other than that, code review passes and Jenkins is happy enough, going to merge!
#14 Updated by intrigeri 2018-12-10 08:48:48
- Status changed from In Progress to Fix committed
- % Done changed from 50 to 100
Applied in changeset commit:tails|e2642e49a1598c9b239fad66cbee51f27d53b66f.
#15 Updated by intrigeri 2018-12-10 09:00:27
- Assignee deleted (
intrigeri) - QA Check changed from Ready for QA to Pass
#16 Updated by intrigeri 2018-12-15 18:12:01
- related to
Bug #16224: Black screen after the boot menu with Intel GPU (i915) added
#17 Updated by CyrilBrulebois 2018-12-16 13:23:51
- Status changed from Fix committed to Resolved
Marking as Resolved as required by the release process documentation, but we might need to revert that at some point, see Bug #16224#note-12.