Feature #11752
Introduce a branch naming convention for which isotesters ignore the @fragile tag
100%
Description
I propose that branches ending with -with-fragile
will result in this behavior.
Subtasks
History
#1 Updated by anonym 2016-08-30 09:37:54
- Status changed from Confirmed to In Progress
- Assignee set to intrigeri
- % Done changed from 0 to 50
- QA Check set to Ready for QA
- Feature Branch set to feature/11752-isotesters-with-fragile
Please review’n’merge the feature branch from git.tails.boum.org:anonym/puppet-tails.
#2 Updated by intrigeri 2016-08-30 10:50:53
- Assignee deleted (
intrigeri) - % Done changed from 50 to 70
Merged, deployed on all isotesters. Let’s see how it goes in practice.
#3 Updated by anonym 2016-08-30 19:26:28
- Assignee set to intrigeri
intrigeri wrote:
> Merged, deployed on all isotesters. Let’s see how it goes in practice.
Not so well! Please pull again and have a look at my proposed fix! (I hope I didn’t screw up the indentation when trying to fix it this time! :D)
I suppose -with-fragile
doesn’t match the semantics sufficiently any more, though. It really is something like -force-all-tests
now. Any better suffix?
#4 Updated by intrigeri 2016-08-31 00:44:54
> I suppose -with-fragile
doesn’t match the semantics sufficiently any more, though. It really is something like -force-all-tests
now. Any better suffix?
Let’s go for -force-all-tests
. The code looks OK once we change the suffix, so I’m going to apply the change now.
#5 Updated by intrigeri 2016-08-31 05:49:14
- % Done changed from 70 to 80
There was another bug, that we’ve fixed, and then we changed the convention to +force-all-tests
. Now testing again.
#6 Updated by intrigeri 2016-09-01 02:31:49
- Status changed from In Progress to Resolved
- Assignee deleted (
intrigeri) - % Done changed from 80 to 100
- QA Check changed from Ready for QA to Pass
Works fine: https://jenkins.tails.boum.org/view/Tails_ISO/job/test_Tails_ISO_feature-stretch-force-all-tests/