config(core): Remove MPTV2 UI feature flag (spinnaker#1510)

louisjimenez · Jan 24, 2020 · 759e988 · 759e988
1 parent fc08f83
commit 759e988
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 23 deletions.
diff --git a/docs/commands.md b/docs/commands.md
@@ -6153,7 +6153,6 @@ hal config features edit [parameters]
  * `--deployment`: If supplied, use this Halyard deployment. This will _not_ create a new deployment.
  * `--gremlin`: Enable Gremlin fault-injection support.
  * `--infrastructure-stages`: Enable infrastructure stages. Allows for creating Load Balancers as part of pipelines.
- * `--managed-pipeline-templates-v2-ui`: Enable managed pipeline templates v2 UI support.
  * `--mine-canary`: Enable canary support. For this to work, you'll need a canary judge configured. Currently, Halyard does not configure canary judge for you.
  * `--no-validate`: (*Default*: `false`) Skip validation.
  * `--pipeline-templates`: Enable pipeline template support. Read more at [https://github.com/spinnaker/dcd-spec](https://github.com/spinnaker/dcd-spec).

diff --git a/...rc/main/java/com/netflix/spinnaker/halyard/cli/command/v1/config/EditFeaturesCommand.java b/...rc/main/java/com/netflix/spinnaker/halyard/cli/command/v1/config/EditFeaturesCommand.java
@@ -90,12 +90,6 @@ public class EditFeaturesCommand extends AbstractConfigCommand {
   @Parameter(names = "--wercker", description = "Enable the Wercker CI stage.", arity = 1)
   private Boolean wercker = null;
 
-  @Parameter(
-      names = "--managed-pipeline-templates-v2-ui",
-      description = "Enable managed pipeline templates v2 UI support.",
-      arity = 1)
-  private Boolean managedPipelineTemplatesV2UI = null;
-
   @Parameter(
       names = "--gremlin",
       description = "Enable Gremlin fault-injection support.",
@@ -129,10 +123,6 @@ protected void executeThis() {
             : features.getAppengineContainerImageUrlDeployments());
     features.setTravis(travis != null ? travis : features.getTravis());
     features.setWercker(wercker != null ? wercker : features.getWercker());
-    features.setManagedPipelineTemplatesV2UI(
-        managedPipelineTemplatesV2UI != null
-            ? managedPipelineTemplatesV2UI
-            : features.getManagedPipelineTemplatesV2UI());
     features.setGremlin(gremlin != null ? gremlin : features.getGremlin());
 
     if (originalHash == features.hashCode()) {

diff --git a/...ard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java b/...ard-config/src/main/java/com/netflix/spinnaker/halyard/config/model/v1/node/Features.java
@@ -22,7 +22,7 @@
 
 @Data
 @EqualsAndHashCode(callSuper = false)
-@JsonIgnoreProperties({"jobs"})
+@JsonIgnoreProperties({"jobs", "managedPipelineTemplatesV2UI"})
 public class Features extends Node {
   @Override
   public String getNodeName() {
@@ -87,11 +87,6 @@ public NodeIterator getChildren() {
       tooLowMessage = "Wercker stage is not available prior to this release.")
   private Boolean wercker;
 
-  @ValidForSpinnakerVersion(
-      lowerBound = "1.13.0",
-      tooLowMessage = "Managed Pipeline Templates v2 UI is not available prior to this release.")
-  private Boolean managedPipelineTemplatesV2UI;
-
   @ValidForSpinnakerVersion(
       lowerBound = "1.13.0",
       tooLowMessage = "Gremlin is not available prior to this release.")

diff --git a/...va/com/netflix/spinnaker/halyard/deploy/spinnaker/v1/profile/deck/DeckProfileFactory.java b/...va/com/netflix/spinnaker/halyard/deploy/spinnaker/v1/profile/deck/DeckProfileFactory.java
@@ -137,12 +137,6 @@ protected void setProfile(
     bindings.put(
         "features.wercker",
         Boolean.toString(features.getWercker() != null ? features.getWercker() : false));
-    bindings.put(
-        "features.managedPipelineTemplatesV2UI",
-        Boolean.toString(
-            features.getManagedPipelineTemplatesV2UI() != null
-                ? features.getManagedPipelineTemplatesV2UI()
-                : false));
     bindings.put(
         "features.gremlin",
         Boolean.toString(features.getGremlin() != null ? features.getGremlin() : false));