Feature #9220
Read test suite configuration files from the local.d directory
100%
Subtasks
Related issues
Blocks Tails - |
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