Skip to content

Commit c729899

Browse files
authored
INTMDB-405: [Terraform] Add cluster label to advanced clusters (#857)
* Update CHANGELOG.md (#849) * INTMDB-405: Add default label if none supplied * Add test for default label exising * Flatten nested IF statement
1 parent 770ebc4 commit c729899

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

mongodbatlas/resource_mongodbatlas_advanced_cluster.go

+3-5
Original file line numberDiff line numberDiff line change
@@ -315,12 +315,10 @@ func resourceMongoDBAtlasAdvancedClusterCreate(ctx context.Context, d *schema.Re
315315
request.EncryptionAtRestProvider = v.(string)
316316
}
317317

318-
if _, ok := d.GetOk("labels"); ok {
319-
if containsLabelOrKey(expandLabelSliceFromSetSchema(d), defaultLabel) {
320-
return diag.FromErr(fmt.Errorf("you should not set `Infrastructure Tool` label, it is used for internal purposes"))
321-
}
322-
request.Labels = append(expandLabelSliceFromSetSchema(d), defaultLabel)
318+
if _, ok := d.GetOk("labels"); ok && containsLabelOrKey(expandLabelSliceFromSetSchema(d), defaultLabel) {
319+
return diag.FromErr(fmt.Errorf("you should not set `Infrastructure Tool` label, it is used for internal purposes"))
323320
}
321+
request.Labels = append(expandLabelSliceFromSetSchema(d), defaultLabel)
324322

325323
if v, ok := d.GetOk("mongo_db_major_version"); ok {
326324
request.MongoDBMajorVersion = formatMongoDBMajorVersion(v.(string))

mongodbatlas/resource_mongodbatlas_advanced_cluster_test.go

+1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ func TestAccResourceMongoDBAtlasAdvancedCluster_basicTenant(t *testing.T) {
4848
resource.TestCheckResourceAttr(resourceName, "name", rNameUpdated),
4949
resource.TestCheckResourceAttrSet(resourceName, "replication_specs.#"),
5050
resource.TestCheckResourceAttrSet(resourceName, "replication_specs.0.region_configs.#"),
51+
resource.TestCheckResourceAttr(resourceName, "labels.#", "0"),
5152
),
5253
},
5354
{

0 commit comments

Comments
 (0)