Bug #12621

The 'Tor is ready' step fails when any systemd service fails

Added by anonym 2017-05-31 10:36:38 . Updated 2017-12-07 12:44:18 .

Status:
Confirmed
Priority:
Normal
Assignee:
anonym
Category:
Test suite
Target version:
Start date:
2017-05-31
Due date:
% Done:

0%

Feature Branch:
Type of work:
Discuss
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

Extracted from Bug #12617:

intrigeri wrote:
> anonym wrote:
> > [Any systemd service failing] breaks all automated tests using Tor, since the Tor is ready step runs systemctl is-system-running. The placement of that check in that step is my second issue: it doesn’t belong. It should be moved into a dedicated step, run in a scenario about the health of the system. A failure would then in itself immediately signal something quite descriptive about the actual failure, instead of falsely signaling that something about little-t tor is utterly broken, blocking a huge part of the test suite.
>
> Well, IMO it’s good that we run systemctl is-system-running as often as we can, before testing other stuff, to identify issues as early as possible. IIRC we can’t run it for offline scenarios since this check will fail if Tor hasn’t started or bootstrapped. But I agree the current way this is implemented gives suboptimal feedback. Perhaps renaming the “Tor is ready” step to describe more accurately what we test in it would help.


Files


Subtasks


History

#1 Updated by anonym 2017-05-31 10:37:01

  • Description updated

#2 Updated by anonym 2017-06-29 13:32:56

  • blocks Feature #13239: Core work 2017Q3: Test suite maintenance added

#3 Updated by anonym 2017-07-06 14:15:08

  • Target version changed from Tails_3.1 to Tails_3.2

#7 Updated by intrigeri 2017-09-07 06:35:28

  • Target version changed from Tails_3.2 to Tails_3.3

#8 Updated by intrigeri 2017-10-01 09:56:58

  • blocks Feature #13240: Core work 2017Q4: Test suite maintenance added

#9 Updated by intrigeri 2017-10-01 09:57:02

  • blocked by deleted (Feature #13239: Core work 2017Q3: Test suite maintenance)

#10 Updated by intrigeri 2017-11-10 14:16:33

  • Target version changed from Tails_3.3 to Tails_3.5

#11 Updated by intrigeri 2017-12-07 12:44:18

  • Target version deleted (Tails_3.5)

#12 Updated by intrigeri 2018-01-01 16:57:13

  • blocked by deleted (Feature #13240: Core work 2017Q4: Test suite maintenance)