fix: removes redundant required field from DisplayVariable (#1959)

GoogleCloudPlatform · Nov 10, 2023 · f23b405 · f23b405
1 parent 91faee5
commit f23b405
Show file tree

Hide file tree

Showing 4 changed files with 58 additions and 72 deletions.
diff --git a/cli/Makefile b/cli/Makefile
@@ -1,7 +1,7 @@
 SHELL := /bin/bash
 
 # Changing this value will trigger a new release
-VERSION=v1.2.8
+VERSION=v1.2.9
 BINARY=bin/cft
 GITHUB_REPO=github.com/GoogleCloudPlatform/cloud-foundation-toolkit
 PLATFORMS := linux windows darwin

diff --git a/cli/bpmetadata/bpmetadata_ui.pb.go b/cli/bpmetadata/bpmetadata_ui.pb.go
diff --git a/cli/bpmetadata/proto/bpmetadata_ui.proto b/cli/bpmetadata/proto/bpmetadata_ui.proto
@@ -21,6 +21,8 @@ message BlueprintUIInput {
 // Additional display specific metadata pertaining to a particular
 // input variable.
 message DisplayVariable {
+  reserved 19;
+
   // The variable name from the corresponding standard metadata file.
   // Gen: auto-generated - the Terraform variable name
   string name = 1; // @gotags: json:"name" yaml:"name"
@@ -112,9 +114,6 @@ message DisplayVariable {
   // Optional.
   // Gen: manually-authored
   int32 level = 18; // @gotags: json:"level,omitempty" yaml:"level,omitempty"
-
-  // Whether the property is required for the deployment or not.
-  bool required = 19; // @gotags: json:"required,omitempty" yaml:"required,omitempty"
 }
 
 message ValueLabel {

diff --git a/cli/bpmetadata/schema/gcp-blueprint-metadata.json b/cli/bpmetadata/schema/gcp-blueprint-metadata.json
@@ -711,9 +711,6 @@
         },
         "level": {
           "type": "integer"
-        },
-        "required": {
-          "type": "boolean"
         }
       },
       "additionalProperties": false,