fix/ssm (#2440)

* fix/ssm

* fix/ssm

Author: SevenEarth

.changelog/2440.txt

@@ -0,0 +1,3 @@
+```release-note:enhancement
+resource/tencentcloud_ssm_ssh_key_pair_secret: Fix kms_key_id problem
+```

tencentcloud/services/ssm/resource_tc_ssm_ssh_key_pair_secret.go

@@ -38,6 +38,7 @@ func ResourceTencentCloudSsmSshKeyPairSecret() *schema.Resource {
 			},
 			"kms_key_id": {
 				Optional:    true,
+				Computed:    true,
 				Type:        schema.TypeString,
 				Description: "Specifies a KMS CMK to encrypt the secret.If this parameter is left empty, the CMK created by Secrets Manager by default will be used for encryption.You can also specify a custom KMS CMK created in the same region for encryption.",
 			},

tencentcloud/services/ssm/resource_tc_ssm_ssh_key_pair_secret_test.go

@@ -3,6 +3,7 @@ package ssm_test
 import (
 	"fmt"
 	"testing"
+	"time"
 
 	tcacctest "github.com/tencentcloudstack/terraform-provider-tencentcloud/tencentcloud/acctest"
 
@@ -14,21 +15,22 @@ import (
 func TestAccTencentCloudSsmSshKeyPairSecretResource_basic(t *testing.T) {
 	t.Parallel()
 	rName := fmt.Sprintf("tf-testacc-kms-key-%s", acctest.RandString(13))
+	rSshName := fmt.Sprintf("%d", time.Now().Unix())
 	resource.Test(t, resource.TestCase{
 		PreCheck: func() {
 			tcacctest.AccPreCheck(t)
 		},
 		Providers: tcacctest.AccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: fmt.Sprintf(testAccSsmSshKeyPairSecret, rName),
+				Config: fmt.Sprintf(testAccSsmSshKeyPairSecret, rName, rSshName),
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example", "description", "desc."),
 					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example", "status", "Disabled"),
 				),
 			},
 			{
-				Config: fmt.Sprintf(testAccSsmSshKeyPairSecretUpdate, rName),
+				Config: fmt.Sprintf(testAccSsmSshKeyPairSecretUpdate, rName, rSshName),
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example", "description", "update desc."),
 					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example", "status", "Enabled"),
@@ -40,6 +42,26 @@ func TestAccTencentCloudSsmSshKeyPairSecretResource_basic(t *testing.T) {
 				ImportStateVerify:       true,
 				ImportStateVerifyIgnore: []string{"clean_ssh_key"},
 			},
+			{
+				Config: fmt.Sprintf(testAccSsmSshKeyPairSecretNoId, rSshName),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example1", "description", "desc."),
+					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example1", "status", "Disabled"),
+				),
+			},
+			{
+				Config: fmt.Sprintf(testAccSsmSshKeyPairSecretNoIdUpdate, rSshName),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example1", "description", "update desc."),
+					resource.TestCheckResourceAttr("tencentcloud_ssm_ssh_key_pair_secret.example1", "status", "Enabled"),
+				),
+			},
+			{
+				ResourceName:            "tencentcloud_ssm_ssh_key_pair_secret.example1",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"clean_ssh_key"},
+			},
 		},
 	})
 }
@@ -61,7 +83,7 @@ resource "tencentcloud_ssm_ssh_key_pair_secret" "example" {
   project_id    = 0
   description   = "desc."
   kms_key_id    = tencentcloud_kms_key.example.id
-  ssh_key_name  = "tf_example_ssh"
+  ssh_key_name  = "tf_ssh_name_%s"
   status        = "Disabled"
   clean_ssh_key = true
 
@@ -88,7 +110,7 @@ resource "tencentcloud_ssm_ssh_key_pair_secret" "example" {
   project_id    = 0
   description   = "update desc."
   kms_key_id    = tencentcloud_kms_key.example.id
-  ssh_key_name  = "tf_example_ssh"
+  ssh_key_name  = "tf_ssh_name_%s"
   status        = "Enabled"
   clean_ssh_key = true
 
@@ -97,3 +119,33 @@ resource "tencentcloud_ssm_ssh_key_pair_secret" "example" {
   }
 }
 `
+
+const testAccSsmSshKeyPairSecretNoId = `
+resource "tencentcloud_ssm_ssh_key_pair_secret" "example1" {
+  secret_name   = "tf-example-ssh-test-no-id"
+  project_id    = 0
+  description   = "desc."
+  ssh_key_name  = "tf_noid_name_%s"
+  status        = "Disabled"
+  clean_ssh_key = true
+
+  tags = {
+    createdBy = "terraform"
+  }
+}
+`
+
+const testAccSsmSshKeyPairSecretNoIdUpdate = `
+resource "tencentcloud_ssm_ssh_key_pair_secret" "example1" {
+  secret_name   = "tf-example-ssh-test-no-id"
+  project_id    = 0
+  description   = "update desc."
+  ssh_key_name  = "tf_noid_name_%s"
+  status        = "Enabled"
+  clean_ssh_key = true
+
+  tags = {
+    createdBy = "terraform"
+  }
+}
+`