poddm commented on code in PR #138:
URL:
https://github.com/apache/cloudstack-terraform-provider/pull/138#discussion_r1821403742
##########
cloudstack/service_offering_constrained_resource.go:
##########
@@ -0,0 +1,270 @@
+package cloudstack
+
+import (
+ "context"
+ "fmt"
+
+ "github.com/apache/cloudstack-go/v2/cloudstack"
+ "github.com/hashicorp/terraform-plugin-framework/resource"
+ "github.com/hashicorp/terraform-plugin-framework/resource/schema"
+
"github.com/hashicorp/terraform-plugin-framework/resource/schema/int32planmodifier"
+
"github.com/hashicorp/terraform-plugin-framework/resource/schema/planmodifier"
+ "github.com/hashicorp/terraform-plugin-framework/types"
+ "github.com/hashicorp/terraform-plugin-framework/types/basetypes"
+)
+
+var (
+ _ resource.Resource = &serviceOfferingConstrainedResource{}
+ _ resource.ResourceWithConfigure = &serviceOfferingConstrainedResource{}
+)
+
+func NewserviceOfferingConstrainedResource() resource.Resource {
+ return &serviceOfferingConstrainedResource{}
+}
+
+type serviceOfferingConstrainedResource struct {
+ client *cloudstack.CloudStackClient
+}
+
+func (r *serviceOfferingConstrainedResource) Schema(_ context.Context, _
resource.SchemaRequest, resp *resource.SchemaResponse) {
+ resp.Schema = schema.Schema{
+ Attributes:
serviceOfferingMergeCommonSchema(map[string]schema.Attribute{
+ "cpu_speed": schema.Int32Attribute{
+ Description: "Speed of CPU in Mhz. This does
not apply to kvm.",
+ Required: true,
+ PlanModifiers: []planmodifier.Int32{
+ int32planmodifier.RequiresReplace(),
+ },
+ },
+ "max_cpu_number": schema.Int32Attribute{
+ Description: "The maximum number of CPUs to be
set with Custom Computer Offering",
+ Required: true,
+ PlanModifiers: []planmodifier.Int32{
+ int32planmodifier.RequiresReplace(),
+ },
+ },
+ "max_memory": schema.Int32Attribute{
+ Description: "The maximum memory size of the
custom service offering in MB",
+ Required: true,
+ PlanModifiers: []planmodifier.Int32{
+ int32planmodifier.RequiresReplace(),
+ },
+ },
+ "min_cpu_number": schema.Int32Attribute{
+ Description: "The minimum number of CPUs to be
set with Custom Computer Offering",
+ Required: true,
+ PlanModifiers: []planmodifier.Int32{
+ int32planmodifier.RequiresReplace(),
+ },
+ },
+ "min_memory": schema.Int32Attribute{
+ Description: "The minimum memory size of the
custom service offering in MB",
+ Required: true,
+ PlanModifiers: []planmodifier.Int32{
+ int32planmodifier.RequiresReplace(),
+ },
+ },
+ }),
+ }
+}
+
+func (r *serviceOfferingConstrainedResource) Create(ctx context.Context, req
resource.CreateRequest, resp *resource.CreateResponse) {
+ //
+ var plan serviceOfferingConstrainedResourceModel
+ var planDiskQosHypervisor ServiceOfferingDiskQosHypervisor
+ var planDiskOffering ServiceOfferingDiskOffering
+ var planDiskQosStorage ServiceOfferingDiskQosStorage
+
+ //
+ resp.Diagnostics.Append(req.Plan.Get(ctx, &plan)...)
+ if !plan.ServiceOfferingDiskQosHypervisor.IsNull() {
+
resp.Diagnostics.Append(plan.ServiceOfferingDiskQosHypervisor.As(ctx,
&planDiskQosHypervisor, basetypes.ObjectAsOptions{})...)
+ }
+ if !plan.ServiceOfferingDiskOffering.IsNull() {
+
resp.Diagnostics.Append(plan.ServiceOfferingDiskOffering.As(ctx,
&planDiskOffering, basetypes.ObjectAsOptions{})...)
+ }
+ if !plan.ServiceOfferingDiskQosStorage.IsNull() {
+
resp.Diagnostics.Append(plan.ServiceOfferingDiskQosStorage.As(ctx,
&planDiskQosStorage, basetypes.ObjectAsOptions{})...)
+ }
+ if resp.Diagnostics.HasError() {
+ return
+ }
+
+ // common params
+ params :=
r.client.ServiceOffering.NewCreateServiceOfferingParams(plan.DisplayText.ValueString(),
plan.Name.ValueString())
+ plan.commonCreateParams(ctx, params)
+ planDiskQosHypervisor.commonCreateParams(ctx, params)
+ planDiskOffering.commonCreateParams(ctx, params)
+ planDiskQosStorage.commonCreateParams(ctx, params)
+
+ // resource specific params
+ if !plan.CpuSpeed.IsNull() {
+ params.SetCpuspeed(int(plan.CpuSpeed.ValueInt32()))
+ }
+ if !plan.MaxCpuNumber.IsNull() {
+ params.SetMaxcpunumber(int(plan.MaxCpuNumber.ValueInt32()))
+ }
+ if !plan.MaxMemory.IsNull() {
+ params.SetMaxmemory(int(plan.MaxMemory.ValueInt32()))
+ }
+ if !plan.MinCpuNumber.IsNull() {
+ params.SetMincpunumber(int(plan.MinCpuNumber.ValueInt32()))
+ }
+ if !plan.MinMemory.IsNull() {
+ params.SetMinmemory(int(plan.MinMemory.ValueInt32()))
+ }
+
+ // create offering
+ cs, err := r.client.ServiceOffering.CreateServiceOffering(params)
+ if err != nil {
+ resp.Diagnostics.AddError(
+ "Error creating service offering",
+ "Could not create constrained offering, unexpected
error: "+err.Error(),
+ )
+ return
+ }
+
+ //
+ plan.Id = types.StringValue(cs.Id)
+
+ //
+ resp.Diagnostics.Append(resp.State.Set(ctx, plan)...)
+}
+
+func (r *serviceOfferingConstrainedResource) Read(ctx context.Context, req
resource.ReadRequest, resp *resource.ReadResponse) {
+ //
+ var state serviceOfferingConstrainedResourceModel
+ var stateDiskQosHypervisor ServiceOfferingDiskQosHypervisor
+ var stateDiskOffering ServiceOfferingDiskOffering
+ var stateDiskQosStorage ServiceOfferingDiskQosStorage
+
+ //
+ resp.Diagnostics.Append(req.State.Get(ctx, &state)...)
+ if !state.ServiceOfferingDiskQosHypervisor.IsNull() {
+
resp.Diagnostics.Append(state.ServiceOfferingDiskQosHypervisor.As(ctx,
&stateDiskQosHypervisor, basetypes.ObjectAsOptions{})...)
+ }
+ if !state.ServiceOfferingDiskOffering.IsNull() {
+
resp.Diagnostics.Append(state.ServiceOfferingDiskOffering.As(ctx,
&stateDiskOffering, basetypes.ObjectAsOptions{})...)
+ }
+ if !state.ServiceOfferingDiskQosStorage.IsNull() {
+
resp.Diagnostics.Append(state.ServiceOfferingDiskQosStorage.As(ctx,
&stateDiskQosStorage, basetypes.ObjectAsOptions{})...)
+ }
+ if resp.Diagnostics.HasError() {
+ return
+ }
+
+ //
+ cs, _, err :=
r.client.ServiceOffering.GetServiceOfferingByID(state.Id.ValueString())
+ if err != nil {
+ resp.Diagnostics.AddError(
+ "Error reading service offering",
+ "Could not read constrained service offering,
unexpected error: "+err.Error(),
+ )
+ return
+ }
+
+ // resource specific
+ if cs.Cpuspeed > 0 {
+ state.CpuSpeed = types.Int32Value(int32(cs.Cpuspeed))
+ }
+ // These fields arent returned from list
+ // max_cpu_number
Review Comment:
You're right. Added these into the state.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
