netascode / terraform-aci-nac-aci

Terraform Cisco ACI Nexus-as-Code Module
https://registry.terraform.io/modules/netascode/nac-aci/aci
Apache License 2.0
17 stars 14 forks source link

Suppport ipDPLearning on fvSubnet #74

Closed jgomezve closed 1 month ago

github-actions[bot] commented 2 months ago

Integration Test Results

8 tests  ±0   0 :white_check_mark:  - 8   0s :stopwatch: ±0s 1 suites ±0   8 :zzz: +8  1 files   ±0   0 :x: ±0 

Results for commit 6b2d1526. ± Comparison against base commit 8053176c.

:recycle: This comment has been updated with latest results.

jgomezve commented 2 months ago

Hey @danischm Wondering if we already faced this challenge. The attribute ipDPLearning of the MO fvSubnet was made available somewhere in version 5.2.x. Not sure how to tell Terraform to only include this in the payload if the attribute is defined

https://github.com/netascode/terraform-aci-nac-aci/blob/c33094d1df3b41194214d7c29ba412a2502a8362/modules/terraform-aci-bridge-domain/main.tf#L57