zalando-incubator / stackset-controller / 674

46%

master: 56%

Pull #123

travis-ci

Changing the semantics of StackLifeCycle.Limit and enforcing it to >= 1

Changing the semantics of StackLifeCycle.Limit to exclude the stack
that's getting traffic. So `StackLifeCycle.Limit: 1` would now mean, keep
at least 2 stacks. This is more intuitive because the stack that's getting
traffic would never be deleted anyways.

Also modifying the CRD definition to enforce StackLifeCycle.Limit >= 1.
This is so people have at least 1 stack around to safely switch to. Since
HPAs are now not deleted, this extra stack should get safely downscaled
without costing too much.

Signed-off-by: Muhammad Muaaz Saleem <muhammad.muaaz.saleem@zalando.de>

22 of 22 new or added lines in 2 files covered. (100.0%)

773 of 1684 relevant lines covered (45.9%)

1.82 hits per line