Skip to content
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

[Bug]: Warning: nothing to update for the connection when updating notifications on the equinix_fabric_connection resource #835

Closed
ogierussell opened this issue Jan 2, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@ogierussell
Copy link

Terraform Version

Terraform v1.5.5 on darwin_arm64

Equinix Provider Version

version = "3.1.0"

Effected Terraform Resources

equinix_fabric_connection

Terraform Config Files

Relevant config:

resource "equinix_fabric_connection" "connection" {
  for_each = var.connection_details

  name = each.key

  notifications {
    type   = "ALL"
    emails = var.notification_list
  }
  ...
}

Debug Output

No response

Panic Output

No response

Expected Behavior

The notifications list for the connection is updated when additional e-mails are added to the list after initial creation.

Actual Behavior

The notifications list is not updated and the following warning is displayed:

│ Warning: nothing to update for the connection xxxx
│ 
│   with equinix_fabric_connection.connections["xxxx"],
│   on equinix-connection.tf line 13, in resource "equinix_fabric_connection" "connection":
│   13: resource "equinix_fabric_connection" "connection" {
│ 

Steps to Reproduce

Create a equinix_fabric_connection resource with an initial set of e-mails.
Add additional e-mails and attempt to apply that change.

@ogierussell ogierussell added the bug Something isn't working label Jan 2, 2025
@thogarty
Copy link
Contributor

thogarty commented Jan 7, 2025

Hi @ogierussell , we're in progress on fixing this for you. We hadn't made the functionality to update notifications available, but are adding it in now.

I'll update you with the PR # once it's available.

@thogarty
Copy link
Contributor

thogarty commented Jan 7, 2025

PR Created: #837

@thogarty
Copy link
Contributor

thogarty commented Jan 7, 2025

Hi @ogierussell , v3.1.1 was released with the fix for the issue you've informed us about. Please try your update with the latest provider version.

I'll wait for your testing before closing the issue.

@ogierussell
Copy link
Author

Hey, this is now working correctly so will close the issue. Thanks again for the support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants