builder: fix linter issues

With the 1.60.1 update to golangci-lint and the configuration changes,
some more linting errors were reported by the linter now, so we fix
those that appeared.

Author: lbajolet-hashicorp

builder/classic/step_list_images.go

@@ -28,7 +28,7 @@ func (s *stepListImages) Run(ctx context.Context, state multistep.StateBag) mult
 	imList, err := imageListClient.GetImageList(&getInput)
 	if err != nil {
 		// If the list didn't exist, create it.
-		ui.Say(fmt.Sprintf(err.Error()))
+		ui.Say(err.Error())
 		ui.Say(fmt.Sprintf("Destination image list %s does not exist; Creating it...",
 			config.DestImageList))
 

builder/oci/config.go

@@ -193,28 +193,28 @@ func (c *Config) Prepare(raws ...interface{}) error {
 		// key involved.
 		var message string = " cannot be present when use_instance_principals is set to true."
 		if c.AccessCfgFile != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("access_cfg_file"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("access_cfg_file"+message))
 		}
 		if c.AccessCfgFileAccount != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("access_cfg_file_account"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("access_cfg_file_account"+message))
 		}
 		if c.UserID != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("user_ocid"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("user_ocid"+message))
 		}
 		if c.TenancyID != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("tenancy_ocid"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("tenancy_ocid"+message))
 		}
 		if c.Region != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("region"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("region"+message))
 		}
 		if c.Fingerprint != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("fingerprint"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("fingerprint"+message))
 		}
 		if c.KeyFile != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("key_file"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("key_file"+message))
 		}
 		if c.PassPhrase != "" {
-			errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("pass_phrase"+message))
+			errs = packersdk.MultiErrorAppend(errs, errors.New("pass_phrase"+message))
 		}
 		// This check is used to facilitate testing. During testing a Mock struct
 		// is assigned to c.configProvider otherwise testing fails because Instance