From 877784d4d1feb992420d691177d948a19ac01386 Mon Sep 17 00:00:00 2001 From: Maggie Neterval Date: Wed, 5 Aug 2020 15:07:55 -0400 Subject: [PATCH] feat(config): add upperBound/tooHighMessage for mineCanary feature flag This flag gates legacy, pre-Kayenta stages in Deck's canary module that are likely not used by OSS users. To reduce confusion, deprecate this flag. --- .../spinnaker/halyard/config/model/v1/node/Features.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java b/halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java index 5eaea50fdc..367eb62529 100644 --- a/halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java +++ b/halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java @@ -64,8 +64,11 @@ public NodeIterator getChildren() { @ValidForSpinnakerVersion( lowerBound = "1.5.0", + upperBound = "1.22.0", tooLowMessage = - "Canary is not configurable prior to this release. Will be stable at a later release.") + "Canary is not configurable prior to this release. Will be stable at a later release.", + tooHighMessage = + "This flag gates legacy canary stages and does not need to be enabled for OSS canary analysis support.") private Boolean mineCanary; @ValidForSpinnakerVersion(