Tails

#1 Updated by Anonymous 2018-09-28 10:50:30

• blocked by Feature #15319: Grow system partition during boot when started from USB added

#2 Updated by Anonymous 2018-09-28 10:54:58

• blocks Bug #16005: Code review & rubber-duck added

#3 Updated by intrigeri 2018-10-12 11:58:21

• blocked by deleted (Bug #16005: Code review & rubber-duck)

#4 Updated by intrigeri 2018-11-27 15:50:04

• Description updated

#5 Updated by CyrilBrulebois 2018-11-28 15:26:58

• Assignee changed from CyrilBrulebois to intrigeri
• QA Check set to Ready for QA
• Feature Branch set to test/16003-growing-system-partition

Here’s the new scenario:

Scenario: Starting Tails from an USB image, and making sure it is correctly set up and resized
  Given a computer
  And I temporarily create a 7200 MiB disk named "usbimage"
  And I write the Tails USB image to disk "usbimage"
  And I start Tails from USB drive "usbimage" with network unplugged and I login
  Then Tails is running from USB drive "usbimage"
  And the label of the FAT filesystem on the system partition on "usbimage" is "Tails"
  And the system partition on "usbimage" is an ESP
  And the FAT filesystem on the system partition on "usbimage" is at least 4096M large
  And the FS UUID for the system partition on "usbimage" was randomized

And I’ve managed to test both positive and negative results with an IMG file built from the 3.10.1 ISO (so without any repartition code), from the current feature branches for Feature #15292, with or without the prospective workaround mentioned in Bug #16153.

#6 Updated by intrigeri 2018-11-28 16:26:07

• Status changed from Confirmed to In Progress

Applied in changeset commit:tails|c81ef61ea9a852f5d6559cea83cf87443ae4dbbc.

#7 Updated by intrigeri 2018-11-28 17:39:40

• Assignee changed from intrigeri to CyrilBrulebois
• % Done changed from 0 to 30
• QA Check changed from Ready for QA to Dev Needed

Great job!

• I think we need to adjust a few things now that we support TAILS_IMG: look for TAILS_ISO in features/support/hooks.rb.
• In the “the FAT filesystem on the system partition” step, I’m not sure about using guestfs on the device while the system is booted. I trust you it works right now but it feels risky to rely on it. I’d rather query this info from the system under test (as you do for the filesystem size). Alternatively, let the system boot, then shut it down, then do all the partition and FS checks offline with guestfs.
• I’ve pushed a few commits where I explain why I’m changing things, as agreed elsewhere.

#9 Updated by intrigeri 2018-12-02 14:56:56

• Description updated

Given Bug #16178 got reported I’m adding a new test to the list in the ticket description.

#11 Updated by intrigeri 2018-12-07 13:29:41

• blocks Feature #16199: Publish a beta for USB images added

#12 Updated by intrigeri 2018-12-16 11:43:06

• Target version changed from Tails_3.11 to Tails_3.12

#17 Updated by intrigeri 2019-01-04 14:01:39

• blocked by deleted (Feature #16199: Publish a beta for USB images)

#19 Updated by CyrilBrulebois 2019-01-05 10:28:30

• QA Check changed from Dev Needed to Ready for QA

System partition flags check just pushed:

   48bd529ad0..8beba325ad  HEAD -> test/16003-growing-system-partition

which I think was the last missing piece for this bug report? Switching to Ready for QA.

#20 Updated by intrigeri 2019-01-05 11:31:50

• QA Check changed from Ready for QA to Dev Needed

• Nitpicking: to be more consistent with how we write things elsewhere in our test suite, please replace "(.+)" with "([^"]+)" for matching captured bits of step titles.
• API clarity: at least one test takes a on "$image" argument but does not use it: /dev/sda1 is hard-coded in the test implementation. I think it’s fine to hard-code /dev/sda1 there (assuming we don’t have code yet to find this out dynamically, or we do the same elsewhere already) but then let’s not pretend we’re supporting checking an arbitrary drive, and instead drop that argument.
• Maybe you missed the first part of my review i.e. Bug #16003#note-7?

#22 Updated by CyrilBrulebois 2019-01-05 21:28:32

• Assignee deleted (CyrilBrulebois)
• QA Check changed from Dev Needed to Ready for QA

Just pushed a few more commits:

   cbbaf4dad1..eef12f0fb5  HEAD -> test/16003-growing-system-partition

I’ve fixed the guestfs issue for all tests I worked on, introduced a wrapper as discussed elsewhere to organize udisks-provided info into a little tree (I almost stripped the common prefix for the keys for brevity, to get Partition, Block, etc. keys, but finally decided against it). Note the XXX, as we have entries spanning across several lines. I haven’t seen enough examples of udisksctl info output to conclude it’s safe enough for a default case… We should probably match for leading spaces, strip all leading spaces, and concat that to the previous value with an extra \n character, and leave the default character as a catch-all which would log/generate an error when encountering an unknown pattern?

Some other tests might benefit from this wrapper, instead of just trying to match the whole output of this command, but that looks like out of scope for this specific task.

#23 Updated by intrigeri 2019-01-06 07:07:29

• Assignee set to intrigeri

(As your designated reviewer / rubber-duck for this task :)

#26 Updated by intrigeri 2019-01-09 18:27:49

• Status changed from In Progress to Resolved
• % Done changed from 30 to 100

Applied in changeset commit:tails|64d58e436c958d28de92c1acfb7240cbb9ccdcf2.

#27 Updated by intrigeri 2019-01-09 18:29:07

• Assignee deleted (intrigeri)
• QA Check changed from Ready for QA to Pass