You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by su...@apache.org on 2026/08/18 12:32:51 UTC
(cloudstack-terraform-provider) 01/01: Address review comments
This is an automated email from the ASF dual-hosted git repository.
sureshanaparti pushed a commit to branch mp/service_gpus_clean
in repository https://gitbox.apache.org/repos/asf/cloudstack-terraform-provider.git
commit b6d25072ca80d3792ff42f3158271d8e680a5c57
Author: Suresh Kumar Anaparti <su...@shapeblue.com>
AuthorDate: Tue Aug 18 18:02:20 2026 +0530
Address review comments
- Add Computed: true to count field to indicate server-managed attribute
- Add Computed: true and Default: false to display field for consistency
with other boolean attributes in the schema
---
cloudstack/service_offering_schema.go | 3 +++
1 file changed, 3 insertions(+)
diff --git a/cloudstack/service_offering_schema.go b/cloudstack/service_offering_schema.go
index e0d47df..19c019f 100644
--- a/cloudstack/service_offering_schema.go
+++ b/cloudstack/service_offering_schema.go
@@ -250,6 +250,7 @@ func serviceOfferingMergeCommonSchema(s1 map[string]schema.Attribute) map[string
"count": schema.Int32Attribute{
Description: "the number of GPUs to assign to the guest VM",
Optional: true,
+ Computed: true,
PlanModifiers: []planmodifier.Int32{
int32planmodifier.RequiresReplace(),
},
@@ -257,9 +258,11 @@ func serviceOfferingMergeCommonSchema(s1 map[string]schema.Attribute) map[string
"display": schema.BoolAttribute{
Description: "whether the GPU is presented as a display device to the guest VM",
Optional: true,
+ Computed: true,
PlanModifiers: []planmodifier.Bool{
boolplanmodifier.RequiresReplace(),
},
+ Default: booldefault.StaticBool(false),
},
},
},