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

Provider crashes when UUID resource entries have duplicated names #427

Open
kklimonda-cl opened this issue Feb 4, 2025 · 0 comments
Open
Labels
bug Something isn't working component/terraform

Comments

@kklimonda-cl
Copy link
Contributor

When UUID non-exhaustive resource in the plan has multiple rules with the same name, SDK manager crashes when building a list of entries to use for MoveGroup() call:

goroutine 10 [running]:
[github.com/PaloAltoNetworks/terraform-provider-panos/internal/manager.(*UuidObjectManager[...]).moveNonExhaustive(0x103ece600](http://github.com/PaloAltoNetworks/terraform-provider-panos/internal/manager.(*UuidObjectManager[...]).moveNonExhaustive(0x103ece600), {0x103e9cbc8, 0x14000c3e030}, {0x14000d25350, 0x0?, 0x0?}, {0x0?, 0x1400050c1a0?, 0x1b?}, 0x140004a98f0?, ...)
        [...]/generated/terraform-provider-panos/internal/manager/uuid.go:208 +0x1f4
[github.com/PaloAltoNetworks/terraform-provider-panos/internal/manager.(*UuidObjectManager[...]).CreateMany(0x103ece600](http://github.com/PaloAltoNetworks/terraform-provider-panos/internal/manager.(*UuidObjectManager[...]).CreateMany(0x103ece600), {0x103e9cbc8, 0x14000c3e030}, {0x14000d25350, 0x0?, 0x0?}, {0x1400019fb08?, 0x5c, 0x5c}, 0x1, ...)
        [...]/generated/terraform-provider-panos/internal/manager/uuid.go:286 +0x614

We should validate resource in during plan to avoid that, and properly handle this in SDK manager.

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

No branches or pull requests

1 participant