Simple update resource migration to Terraform Plugin Framework

.golangci.yaml

@@ -38,6 +38,16 @@ run:
   # If it's not please let us know.
   # "/" will be replaced by current OS file path separator to properly work on Windows.
   skip-files:
+    - "redfish/provider/data_source_redfish_bios.go"
+    - "redfish/provider/data_source_redfish_firmware_inventory.go"
+    - "redfish/provider/data_source_redfish_storage.go"
+    - "redfish/provider/data_source_redfish_system_boot.go"
+    - "redfish/provider/data_source_redfish_virtual_media.go"
+    - "redfish/provider/resource_redfish_storage_volume.go"
+    - "redfish/provider/resource_redfish_storage_volume.go"
+    - "redfish/provider/resource_redfish_user_account.go"
+    - "redfish/provider/resource_redfish_virtual_media.go"
+    - "redfish/provider/resource_redfish_bios.go"
   #   - ".*\\.my\\.go$"
   # If set we pass it to "go list -mod={option}". From "go help modules":
   # If invoked with -mod=readonly, the go command is disallowed from the implicit
@@ -164,7 +174,7 @@ linters-settings:
       - name: cyclomatic
         severity: warning
         disabled: false
-        arguments: [10] # TBD
+        arguments: [20] # TBD 10
       # https://github.com/mgechev/revive/blob/master/RULES_DESCRIPTIONS.md#datarace
       - name: datarace
         severity: warning

common/job_management.go

@@ -21,7 +21,7 @@ const (
 //   - jobURI -> URI for the job to check.
 //   - timeBetweenAttempts -> time to wait between attempts. I.e. 30 means 30 seconds.
 //   - timeout -> maximun time to wait until job is considered failed.
-func WaitForJobToFinish(service *gofish.Service, jobURI string, timeBetweenAttempts int, timeout int) error {
+func WaitForJobToFinish(service *gofish.Service, jobURI string, timeBetweenAttempts int64, timeout int64) error {
 	// Create tickers
 	attemptTick := time.NewTicker(time.Duration(timeBetweenAttempts) * time.Second)
 	timeoutTick := time.NewTicker(time.Duration(timeout) * time.Second)
@@ -44,7 +44,7 @@ func WaitForJobToFinish(service *gofish.Service, jobURI string, timeBetweenAttem
 			}
 		case <-timeoutTick.C:
 			log.Printf("[DEBUG] - Error. Timeout reached\n")
-			return fmt.Errorf("Timeout waiting for the job to finish")
+			return fmt.Errorf("timeout waiting for the job to finish")
 		}
 	}
 }

docs/resources/simple_update.md

@@ -0,0 +1,202 @@
+---
+# Copyright (c) 2023 Dell Inc., or its subsidiaries. All Rights Reserved.
+#
+# Licensed under the Mozilla Public License Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://mozilla.org/MPL/2.0/
+#
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+title: "redfish_simple_update resource"
+linkTitle: "redfish_simple_update"
+page_title: "redfish_simple_update Resource - terraform-provider-redfish"
+subcategory: ""
+description: |-
+  Resource for managing power.
+---
+
+# redfish_simple_update (Resource)
+
+Resource for managing power.
+This Terraform resource is used to Update the iDRAC Server. We can Read the existing version or update the same using this resource.
+
+## Example Usage
+
+variables.tf
+```terraform
+/*
+Copyright (c) 2023 Dell Inc., or its subsidiaries. All Rights Reserved.
+
+Licensed under the Mozilla Public License Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://mozilla.org/MPL/2.0/
+
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+variable "rack1" {
+  type = map(object({
+    user         = string
+    password     = string
+    endpoint     = string
+    ssl_insecure = bool
+  }))
+}
+```
+
+terraform.tfvars
+```terraform
+/*
+Copyright (c) 2023 Dell Inc., or its subsidiaries. All Rights Reserved.
+
+Licensed under the Mozilla Public License Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://mozilla.org/MPL/2.0/
+
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+rack1 = {
+  "my-server-1" = {
+    user         = "admin"
+    password     = "passw0rd"
+    endpoint     = "https://my-server-1.myawesomecompany.org"
+    ssl_insecure = true
+  },
+  "my-server-2" = {
+    user         = "admin"
+    password     = "passw0rd"
+    endpoint     = "https://my-server-2.myawesomecompany.org"
+    ssl_insecure = true
+  },
+}
+```
+
+provider.tf
+```terraform
+/*
+Copyright (c) 2023 Dell Inc., or its subsidiaries. All Rights Reserved.
+
+Licensed under the Mozilla Public License Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://mozilla.org/MPL/2.0/
+
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+terraform {
+  required_providers {
+    redfish = {
+      version = "1.0.0"
+      source  = "registry.terraform.io/dell/redfish"
+    }
+  }
+}
+```
+
+main.tf
+```terraform
+/*
+Copyright (c) 2023 Dell Inc., or its subsidiaries. All Rights Reserved.
+
+Licensed under the Mozilla Public License Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://mozilla.org/MPL/2.0/
+
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+resource "redfish_simple_update" "update" {
+  for_each = var.rack1
+
+  redfish_server {
+    user         = each.value.user
+    password     = each.value.password
+    endpoint     = each.value.endpoint
+    ssl_insecure = each.value.ssl_insecure
+  }
+
+  // The network protocols and image for firmware update
+  transfer_protocol     = "HTTP"
+  target_firmware_image = "/home/mikeletux/Downloads/BIOS_FXC54_WN64_1.15.0.EXE"
+  // Reset parameters to be applied when upgrade is completed
+  reset_type    = "ForceRestart"
+  reset_timeout = 120 // If not set, by default will be 120s
+  // The maximum amount of time to wait for the simple update job to be completed
+  simple_update_job_timeout = 1200 // If not set, by default will be 1200s
+}
+```
+
+After the successful execution of the above resource block, firmware would have got updated. It can be verified through state file.
+
+<!-- schema generated by tfplugindocs -->
+## Schema
+
+### Required
+
+- `redfish_server` (Attributes) Redfish Server (see [below for nested schema](#nestedatt--redfish_server))
+- `reset_type` (String) Reset type allows to choose the type of restart to apply when firmware upgrade is scheduled. Possible values are: "ForceRestart", "GracefulRestart" or "PowerCycle"
+- `target_firmware_image` (String) Target firmware image used for firmware update on the redfish instance. Make sure you place your firmware packages in the same folder as the module and set it as follows: "${path.module}/BIOS_FXC54_WN64_1.15.0.EXE"
+- `transfer_protocol` (String) The network protocol that the Update Service uses to retrieve the software image file located at the URI provided in ImageURI, if the URI does not contain a scheme. Accepted values: CIFS, FTP, SFTP, HTTP, HTTPS, NSF, SCP, TFTP, OEM, NFS. Currently only HTTP, HTTPS and NFS are supported with local file path or HTTP(s)/NFS link.
+
+### Optional
+
+- `reset_timeout` (Number) Time in seconds that the provider waits for the server to be reset before timing out.
+- `simple_update_job_timeout` (Number) Time in seconds that the provider waits for the simple update job to be completed before timing out.
+
+### Read-Only
+
+- `id` (String) ID of the simple update resource
+- `software_id` (String) Software ID from the firmware package uploaded
+- `version` (String) Software version from the firmware package uploaded
+
+<a id="nestedatt--redfish_server"></a>
+### Nested Schema for `redfish_server`
+
+Required:
+
+- `endpoint` (String) Server BMC IP address or hostname
+
+Optional:
+
+- `password` (String, Sensitive) User password for login
+- `user` (String) User name for login
+- `validate_cert` (Boolean) This field indicates whether the SSL/TLS certificate must be verified or not
+
+
+

go.mod

@@ -9,6 +9,8 @@ require (
 	github.com/stmcginnis/gofish v0.14.1-0.20230828052805-4738a5dd9470
 )
 
+require github.com/apparentlymart/go-textseg/v15 v15.0.0 // indirect
+
 require (
 	github.com/hashicorp/terraform-plugin-framework-validators v0.12.0
 	github.com/hashicorp/terraform-plugin-go v0.19.0
@@ -22,7 +24,6 @@ require (
 	github.com/Masterminds/sprig/v3 v3.2.2 // indirect
 	github.com/ProtonMail/go-crypto v0.0.0-20230717121422-5aa5874ade95 // indirect
 	github.com/agext/levenshtein v1.2.2 // indirect
-	github.com/apparentlymart/go-textseg/v15 v15.0.0 // indirect
 	github.com/armon/go-radix v1.0.0 // indirect
 	github.com/bgentry/speakeasy v0.1.0 // indirect
 	github.com/cloudflare/circl v1.3.3 // indirect

redfish/models/simpleUpdate.go

@@ -0,0 +1,18 @@
+package models
+
+import (
+	"github.com/hashicorp/terraform-plugin-framework/types"
+)
+
+// SimpleUpdateRes is struct for simple update resource
+type SimpleUpdateRes struct {
+	Id            types.String  `tfsdk:"id"`
+	RedfishServer RedfishServer `tfsdk:"redfish_server"`
+	Protocol      types.String  `tfsdk:"transfer_protocol"`
+	Image         types.String  `tfsdk:"target_firmware_image"`
+	ResetType     types.String  `tfsdk:"reset_type"`
+	ResetTimeout  types.Int64   `tfsdk:"reset_timeout"`
+	JobTimeout    types.Int64   `tfsdk:"simple_update_job_timeout"`
+	SoftwareId    types.String  `tfsdk:"software_id"`
+	Version       types.String  `tfsdk:"version"`
+}