Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kms: Improves the invoking api method and supports refreshing credential automatically #8237

Merged
merged 1 commit into from
Feb 5, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 2 additions & 8 deletions alicloud/data_source_alicloud_kms_aliases.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -90,16 +89,11 @@ func dataSourceAlicloudKmsAliasesRead(d *schema.ResourceData, meta interface{})
}
}
var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
for {
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &runtime)
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, true)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
8 changes: 2 additions & 6 deletions alicloud/data_source_alicloud_kms_ciphertext.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"

"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
Expand Down Expand Up @@ -67,13 +66,10 @@ func dataSourceAlicloudKmsCiphertextRead(d *schema.ResourceData, meta interface{
}

var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
10 changes: 2 additions & 8 deletions alicloud/data_source_alicloud_kms_key_versions.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -84,16 +83,11 @@ func dataSourceAlicloudKmsKeyVersionsRead(d *schema.ResourceData, meta interface
}
}
var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
for {
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &runtime)
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, true)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
10 changes: 2 additions & 8 deletions alicloud/data_source_alicloud_kms_keys.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -165,16 +164,11 @@ func dataSourceAlicloudKmsKeysRead(d *schema.ResourceData, meta interface{}) err
}
status, statusOk := d.GetOk("status")
var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
for {
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &runtime)
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, true)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
8 changes: 2 additions & 6 deletions alicloud/data_source_alicloud_kms_plaintext.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"

"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
Expand Down Expand Up @@ -65,13 +64,10 @@ func dataSourceAlicloudKmsPlaintextRead(d *schema.ResourceData, meta interface{}
}

var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
10 changes: 3 additions & 7 deletions alicloud/data_source_alicloud_kms_secret_versions.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -107,14 +106,11 @@ func dataSourceAlicloudKmsSecretVersionsRead(d *schema.ResourceData, meta interf
var objects []map[string]interface{}

var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
for {
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down Expand Up @@ -191,7 +187,7 @@ func dataSourceAlicloudKmsSecretVersionsRead(d *schema.ResourceData, meta interf
request["SecretName"] = secretName
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
10 changes: 2 additions & 8 deletions alicloud/data_source_alicloud_kms_secrets.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (
"time"

"github.com/PaesslerAG/jsonpath"
util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -155,16 +154,11 @@ func dataSourceAlicloudKmsSecretsRead(d *schema.ResourceData, meta interface{})
}

var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
for {
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &runtime)
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, true)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
6 changes: 3 additions & 3 deletions alicloud/data_source_alicloud_kms_service.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package alicloud
import (
"time"

util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -37,12 +36,13 @@ func dataSourceAlicloudKmsServiceRead(d *schema.ResourceData, meta interface{})
}
action := "OpenKmsService"
request := map[string]interface{}{}
conn, err := meta.(*connectivity.AliyunClient).NewTeaCommonClient(connectivity.OpenKmsService)
var err error
client := meta.(*connectivity.AliyunClient)
if err != nil {
return WrapError(err)
}
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err := conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err := client.RpcPostWithEndpoint("Kms", "2016-01-20", action, nil, request, false, connectivity.OpenKmsService)
if err != nil {
if IsExpectedErrors(err, []string{"QPS Limit Exceeded"}) || NeedRetry(err) {
return resource.RetryableError(err)
Expand Down
13 changes: 2 additions & 11 deletions alicloud/resource_alicloud_db_instance.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ import (

"github.com/PaesslerAG/jsonpath"

util "github.com/alibabacloud-go/tea-utils/service"

"github.com/hashicorp/go-uuid"
"github.com/hashicorp/terraform-plugin-sdk/helper/hashcode"

Expand Down Expand Up @@ -2228,19 +2226,12 @@ func buildDBCreateRequest(d *schema.ResourceData, meta interface{}) (map[string]
func findKmsRoleArn(client *connectivity.AliyunClient, k string) (string, error) {
action := "DescribeKey"
var response map[string]interface{}

var err error
request := make(map[string]interface{})
request["KeyId"] = k

conn, err := client.NewKmsClient()
if err != nil {
return "", WrapError(err)
}
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(5*time.Minute, func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &runtime)
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, true)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
22 changes: 6 additions & 16 deletions alicloud/resource_alicloud_kms_alias.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"log"
"time"

util "github.com/alibabacloud-go/tea-utils/service"
"github.com/aliyun/terraform-provider-alicloud/alicloud/connectivity"
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
Expand Down Expand Up @@ -39,15 +38,12 @@ func resourceAlicloudKmsAliasCreate(d *schema.ResourceData, meta interface{}) er
var response map[string]interface{}
action := "CreateAlias"
request := make(map[string]interface{})
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
request["AliasName"] = d.Get("alias_name")
request["KeyId"] = d.Get("key_id")
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(d.Timeout(schema.TimeoutCreate), func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down Expand Up @@ -85,10 +81,7 @@ func resourceAlicloudKmsAliasRead(d *schema.ResourceData, meta interface{}) erro
}
func resourceAlicloudKmsAliasUpdate(d *schema.ResourceData, meta interface{}) error {
client := meta.(*connectivity.AliyunClient)
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
var response map[string]interface{}
if d.HasChange("key_id") {
request := map[string]interface{}{
Expand All @@ -98,7 +91,7 @@ func resourceAlicloudKmsAliasUpdate(d *schema.ResourceData, meta interface{}) er
action := "UpdateAlias"
wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(d.Timeout(schema.TimeoutUpdate), func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand All @@ -119,17 +112,14 @@ func resourceAlicloudKmsAliasDelete(d *schema.ResourceData, meta interface{}) er
client := meta.(*connectivity.AliyunClient)
action := "DeleteAlias"
var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
var err error
request := map[string]interface{}{
"AliasName": d.Id(),
}

wait := incrementalWait(3*time.Second, 3*time.Second)
err = resource.Retry(d.Timeout(schema.TimeoutDelete), func() *resource.RetryError {
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
if NeedRetry(err) {
wait()
Expand Down
10 changes: 2 additions & 8 deletions alicloud/resource_alicloud_kms_alias_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,15 +51,9 @@ func testSweepKmsAlias(region string) error {
action := "ListAliases"

var response map[string]interface{}
conn, err := client.NewKmsClient()
if err != nil {
return WrapError(err)
}
sweeped := false
for {
runtime := util.RuntimeOptions{}
runtime.SetAutoretry(true)
response, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
response, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
return WrapErrorf(err, DataDefaultErrorMsg, "alicloud_kms_alias", action, AlibabaCloudSdkGoERROR)
}
Expand Down Expand Up @@ -91,7 +85,7 @@ func testSweepKmsAlias(region string) error {
request := map[string]interface{}{
"AliasName": item["AliasName"],
}
_, err = conn.DoRequest(StringPointer(action), nil, StringPointer("POST"), StringPointer("2016-01-20"), StringPointer("AK"), nil, request, &util.RuntimeOptions{})
_, err = client.RpcPost("Kms", "2016-01-20", action, nil, request, false)
if err != nil {
log.Printf("[ERROR] Failed to delete Kms Alias (%s): %s", item["AliasName"], err)
}
Expand Down
Loading
Loading