launchdarkly / terraform-provider-launchdarkly

Terraform LaunchDarkly provider
https://www.terraform.io/docs/providers/launchdarkly/
Mozilla Public License 2.0
22 stars 24 forks source link

benign drift on fallthrough.rollout_weights #148

Open nhomble opened 1 year ago

nhomble commented 1 year ago

Using provider version 2.0

We see that terraform detects drift from our LD instance because fallthrough.rollout_weights becomes []. To avoid the noise in detection, I think the provider could handle nil better instead of waiting for the reconciliation.

My best guess is that https://github.com/launchdarkly/terraform-provider-launchdarkly/blob/main/launchdarkly/fallthrough_helper.go#L73 could be enhanced to default to [] before the percent rollout check.

ldhenry commented 1 year ago

Hey @nhomble,

Thanks for bringing this to our attention. I've created an internal ticket to investigate a solution. We'll be sure to update this issue as we learn more.

Thanks, Henry