-
Notifications
You must be signed in to change notification settings - Fork 47
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
Keep value of metal_connection.vlans in sync with API #271
Conversation
The TestCheckAttr functions cannot be used on list values, so we have to validate the `vlans` length and each individual element instead.
Codecov ReportBase: 59.43% // Head: 60.17% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #271 +/- ##
==========================================
+ Coverage 59.43% 60.17% +0.74%
==========================================
Files 74 74
Lines 12889 12911 +22
==========================================
+ Hits 7660 7769 +109
+ Misses 4937 4839 -98
- Partials 292 303 +11
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Keep value of metal_connection.vlans in sync with API
This updates the
metal_connection
resource and data source to read the value ofvlans
from the API response to ensure that terraform state stays up-to-date with the real infrastructure if someone modifies the VLANs for a terraform-managedmetal_connection
outside of terraform.Closes #270