-
Notifications
You must be signed in to change notification settings - Fork 154
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
feat(apig): add new datasource to get list of signatures #4818
Conversation
4e5b246
to
f49ead3
Compare
7d2b9d4
to
c8bb049
Compare
docs/data-sources/apig_signatures.md
Outdated
variable "instance_id" {} | ||
variable "signature_name" {} | ||
|
||
data "huaweicloud_apig_api_associated_acl_policies" "test" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data "huaweicloud_apig_api_associated_acl_policies" "test" { | |
data "huaweicloud_apig_signatures" "test" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
signSecret := utils.Reverse(signKey) | ||
|
||
return fmt.Sprintf(` | ||
%s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
%s | |
%s | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
fcbb17f
to
5f2c93e
Compare
docs/data-sources/apig_signatures.md
Outdated
|
||
* `bind_num` - The number of bound APIs. | ||
|
||
* `created_at` - The creation time of the signature. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* `created_at` - The creation time of the signature. | |
* `created_at` - The creation time of the signature, in RFC3339 format. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
5f2c93e
to
759be3e
Compare
/approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR has been approved by: @Lance52259
/approve
/lgtm
What this PR does / why we need it:
add new datasource to get list of signatures.
Which issue this PR fixes:
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)fixes #xxx
Special notes for your reviewer:
Release note:
PR Checklist
Acceptance Steps Performed