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

datasource/alicloud_config_aggregate_config_rules: add the Attributes aggregator_id, compliance datasource/alicloud_config_config_rules: removed the fields member_id, multi_account #4304

Merged
merged 1 commit into from Dec 3, 2021

Conversation

Wanghx0991
Copy link
Contributor

No description provided.

@Wanghx0991 Wanghx0991 force-pushed the config_fix branch 4 times, most recently from d897233 to 54a9113 Compare December 2, 2021 11:17
@@ -285,7 +285,7 @@ func TestAccAlicloudPvtzZone_syncTask(t *testing.T) {
ra := resourceAttrInit(resourceId, pvtzZoneBasicMap)
rc := resourceCheckInitWithDescribeMethod(resourceId, &v, func() interface{} {
return &PvtzService{testAccProvider.Meta().(*connectivity.AliyunClient)}
}, "DescribePvtzZoneWithSyncHostTask")
}, "DescribePvtzZone")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正了上一个的单测问题

@@ -70,6 +70,10 @@ func dataSourceAlicloudConfigAggregateConfigRules() *schema.Resource {
Type: schema.TypeString,
Computed: true,
},
"aggregator_id": {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

文档透出,但schema里未定义

@@ -70,6 +70,10 @@ func dataSourceAlicloudConfigAggregateConfigRules() *schema.Resource {
Type: schema.TypeString,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

标题写清楚,增加了什么字段,删除了什么字段

website/docs/d/config_rules.html.markdown Outdated Show resolved Hide resolved
@Wanghx0991 Wanghx0991 changed the title datasource/alicloud_config_aggregate_config_rules: add the Attributes aggregator_id datasource/alicloud_config_aggregate_config_rules: add the Attributes aggregator_id, compliance datasource/alicloud_config_config_rules: removed the fields member_id, multi_account Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants