Skip to content

Commit 6aba792

Browse files
committed
'recording_mode_overrides' -> 'recording_mode_override'.
1 parent 618b643 commit 6aba792

5 files changed

+17
-19
lines changed

internal/service/configservice/configuration_recorder.go

+7-7
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,11 @@ func ResourceConfigurationRecorder() *schema.Resource {
108108
Schema: map[string]*schema.Schema{
109109
"recording_frequency": {
110110
Type: schema.TypeString,
111-
Default: configservice.RecordingFrequencyContinuous,
112111
Optional: true,
112+
Default: configservice.RecordingFrequencyContinuous,
113113
ValidateFunc: validation.StringInSlice(configservice.RecordingFrequency_Values(), false),
114114
},
115-
"recording_mode_overrides": {
115+
"recording_mode_override": {
116116
Type: schema.TypeList,
117117
Optional: true,
118118
// Even though the name is plural, the API only allows one override:
@@ -124,17 +124,17 @@ func ResourceConfigurationRecorder() *schema.Resource {
124124
Type: schema.TypeString,
125125
Optional: true,
126126
},
127+
"recording_frequency": {
128+
Type: schema.TypeString,
129+
Required: true,
130+
ValidateFunc: validation.StringInSlice(configservice.RecordingFrequency_Values(), false),
131+
},
127132
"resource_types": {
128133
Type: schema.TypeSet,
129134
Required: true,
130135
MinItems: 1,
131136
Elem: &schema.Schema{Type: schema.TypeString},
132137
},
133-
"recording_frequency": {
134-
Type: schema.TypeString,
135-
Required: true,
136-
ValidateFunc: validation.StringInSlice(configservice.RecordingFrequency_Values(), false),
137-
},
138138
},
139139
},
140140
},

internal/service/configservice/configuration_recorder_test.go

+5-6
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ func testAccConfigurationRecorder_disappears(t *testing.T) {
6464
Check: resource.ComposeTestCheckFunc(
6565
testAccCheckConfigurationRecorderExists(ctx, resourceName, &cr),
6666
acctest.CheckResourceDisappears(ctx, acctest.Provider, tfconfig.ResourceConfigurationRecorder(), resourceName),
67-
acctest.CheckResourceDisappears(ctx, acctest.Provider, tfconfig.ResourceConfigurationRecorder(), resourceName),
6867
),
6968
ExpectNonEmptyPlan: true,
7069
},
@@ -95,10 +94,10 @@ func testAccConfigurationRecorder_allParams(t *testing.T) {
9594
resource.TestCheckResourceAttr(resourceName, "recording_group.0.resource_types.#", "2"),
9695
resource.TestCheckResourceAttr(resourceName, "recording_mode.#", "1"),
9796
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_frequency", "DAILY"),
98-
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_overrides.#", "1"),
99-
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_overrides.0.recording_frequency", "CONTINUOUS"),
100-
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_overrides.0.resource_types.#", "1"),
101-
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_overrides.0.resource_types.0", "AWS::EC2::Instance"),
97+
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_override.#", "1"),
98+
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_override.0.recording_frequency", "CONTINUOUS"),
99+
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_override.0.resource_types.#", "1"),
100+
resource.TestCheckResourceAttr(resourceName, "recording_mode.0.recording_mode_override.0.resource_types.0", "AWS::EC2::Instance"),
102101
acctest.CheckResourceAttrGlobalARN(resourceName, "role_arn", "iam", fmt.Sprintf("role/%s", rName)),
103102
),
104103
},
@@ -262,7 +261,7 @@ resource "aws_config_configuration_recorder" "test" {
262261
recording_mode {
263262
recording_frequency = "DAILY"
264263
265-
recording_mode_overrides {
264+
recording_mode_override {
266265
resource_types = ["AWS::EC2::Instance"]
267266
recording_frequency = "CONTINUOUS"
268267
}

internal/service/configservice/flex.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ func expandRecordingMode(mode map[string]interface{}) *configservice.RecordingMo
107107
recordingMode.RecordingFrequency = aws.String(v)
108108
}
109109

110-
if v, ok := mode["recording_mode_overrides"]; ok {
110+
if v, ok := mode["recording_mode_override"]; ok {
111111
recordingMode.RecordingModeOverrides = expandRecordingModeRecordingModeOverrides(v.([]interface{}))
112112
}
113113

@@ -310,7 +310,7 @@ func flattenRecordingMode(g *configservice.RecordingMode) []map[string]interface
310310
}
311311

312312
if g.RecordingModeOverrides != nil && len(g.RecordingModeOverrides) > 0 {
313-
m["recording_mode_overrides"] = flattenRecordingModeRecordingModeOverrides(g.RecordingModeOverrides)
313+
m["recording_mode_override"] = flattenRecordingModeRecordingModeOverrides(g.RecordingModeOverrides)
314314
}
315315

316316
return []map[string]interface{}{m}

internal/service/configservice/remediation_configuration_test.go

-1
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,6 @@ func testAccRemediationConfiguration_disappears(t *testing.T) {
127127
Check: resource.ComposeTestCheckFunc(
128128
testAccCheckRemediationConfigurationExists(ctx, resourceName, &rc),
129129
acctest.CheckResourceDisappears(ctx, acctest.Provider, tfconfigservice.ResourceRemediationConfiguration(), resourceName),
130-
acctest.CheckResourceDisappears(ctx, acctest.Provider, tfconfigservice.ResourceRemediationConfiguration(), resourceName),
131130
),
132131
ExpectNonEmptyPlan: true,
133132
},

website/docs/r/config_configuration_recorder.html.markdown

+3-3
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ resource "aws_config_configuration_recorder" "foo" {
7878
recording_mode {
7979
recording_frequency = "CONTINUOUS"
8080
81-
recording_mode_overrides {
81+
recording_mode_override {
8282
description = "Only record EC2 network interfaces daily"
8383
resource_types = ["AWS::EC2::NetworkInterface"]
8484
recording_frequency = "DAILY"
@@ -115,9 +115,9 @@ This resource supports the following arguments:
115115
### recording_mode Configuration Block
116116

117117
* `recording_frequency` - (Required) Default reecording frequency. `CONTINUOUS` or `DAILY`.
118-
* `recording_mode_overrides` - (Optional) Recording mode overrides. Detailed below.
118+
* `recording_mode_override` - (Optional) Recording mode overrides. Detailed below.
119119

120-
#### recording_mode_overrides Configuration Block
120+
#### recording_mode_override Configuration Block
121121

122122
* `description` - (Optional) A description you provide of the override.
123123
* `resource_types` - (Required) A list that specifies the types of AWS resources for which the override applies to. See [restrictions in the AWS Docs](https://docs.aws.amazon.com/config/latest/APIReference/API_RecordingModeOverride.html)

0 commit comments

Comments
 (0)