Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

respect vpc2 order for resource_vultr_instance #551

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

quantimnot
Copy link

@quantimnot quantimnot commented Feb 3, 2025

Description

The vpc2_ids were stored in a set type, so the order wasn't preserved. I changed it to a list type.

NOTE: This is an incomplete change. A check for duplicate values should be added. This change should be done to other resources like bare metal.

NOTE: I am not a Go programmer.

NOTE: I am not doing anymore with this PR. You can merge it, or close it.

Related Issues

Fixes #550

Checklist:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you linted your code locally prior to submission?
  • Have you successfully ran tests with your changes locally?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] - VPC 2.0 bind order not respected
1 participant