Feature #9220

Read test suite configuration files from the local.d directory

Added by anonym 2015-04-10 12:09:16 . Updated 2015-05-12 18:38:44 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Test suite
Target version:
Start date:
2015-04-10
Due date:
% Done:

100%

Feature Branch:
test/9220-local-dot-d
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

As proposed in Feature #8920.


Subtasks


Related issues

Blocks Tails - Feature #9222: Document how to use shared test suite secrets for release testing Resolved 2015-04-10

History

#1 Updated by anonym 2015-04-10 12:12:02

  • Status changed from Confirmed to In Progress

Applied in changeset commit:57b9a093ade61f4ae49d43a091f2c073d92f2eeb.

#2 Updated by anonym 2015-04-10 12:16:01

  • Assignee changed from anonym to intrigeri
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA
  • Feature Branch set to test/9220-local-dot-d

In this implementation I left local.yml and have it take precedence over local.d/*.yml (internally they use lexical order). I’m not sure exactly what we want here. Possibly we could even remove local.yml.

#3 Updated by anonym 2015-04-10 12:25:35

  • blocks Feature #9222: Document how to use shared test suite secrets for release testing added

#4 Updated by intrigeri 2015-04-14 16:02:33

  • Assignee changed from intrigeri to kytv

I think that Kill Your TV is supposed to review and test such things first, before it lands onto my plate.

#5 Updated by kytv 2015-04-15 00:27:11

  • Assignee changed from kytv to intrigeri

Code looks good. Looks like a typo on wiki/src/contribute/release_process/test/usage.mdwn

 51    internal precende according to lexical order), which takes

I tested this by splitting my local.yml into multiple files, e.g. tor.yml, ssh.yml, etc, and running tests that need those secrets. All went well, the code does what it should.

Would merge after fixing the typo.

#6 Updated by intrigeri 2015-04-15 07:59:35

  • Assignee changed from intrigeri to anonym
  • % Done changed from 50 to 60
  • QA Check changed from Ready for QA to Dev Needed

Kill Your TV: in general, please assign such stuff to me for merging only once the problems your review has identified have been solved and reviewed again.

#7 Updated by anonym 2015-04-17 13:56:49

  • Assignee changed from anonym to intrigeri
  • QA Check changed from Dev Needed to Ready for QA

Typo fixed. Assigning to you, intrigeri, for the final review’n’merge since kytv is happy with the current state.

#8 Updated by intrigeri 2015-04-17 17:19:02

  • Status changed from In Progress to Fix committed
  • % Done changed from 60 to 100

Applied in changeset commit:ab25c0b953769f5e432d8794c7dad6df725ad9f4.

#9 Updated by intrigeri 2015-04-17 17:19:58

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

Merged, added the nitpicking commit:3dabc8b on top.

#10 Updated by BitingBird 2015-05-12 18:38:44

  • Status changed from Fix committed to Resolved