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

Require each NIC on an instance to be in a unique subnet #1167

Merged
merged 2 commits into from
Jun 7, 2022

Conversation

bnaecker
Copy link
Collaborator

@bnaecker bnaecker commented Jun 7, 2022

  • Adds additional query logic the network_interface::InsertQuery that
    fails the query if the VPC Subnet for a new interface is the same as
    that for an existing interface on the instance.
  • Updates tests to conform to and verify new unique VPC Subnet behavior

- Adds additional query logic the `network_interface::InsertQuery` that
  fails the query if the VPC Subnet for a new interface is the same as
  that for an existing interface on the instance.
- Updates tests to conform to and verify new unique VPC Subnet behavior
@bnaecker bnaecker requested review from davepacheco and smklein June 7, 2022 03:44
@bnaecker
Copy link
Collaborator Author

bnaecker commented Jun 7, 2022

Addresses #1162

- More obvious non-UUID sentinel strings in error messages
- Simplify column aliases
- Link to COALESCE issue
@bnaecker bnaecker enabled auto-merge (squash) June 7, 2022 17:38
@bnaecker bnaecker merged commit 3ae28c7 into main Jun 7, 2022
@bnaecker bnaecker deleted the multiple-nics-unique-subnets branch June 7, 2022 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants