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/openstack: Add fixed ips to openstack ports #3772

Merged
merged 2 commits into from
Nov 6, 2015

Conversation

Fodoj
Copy link

@Fodoj Fodoj commented Nov 5, 2015

No description provided.

@jtopjian
Copy link
Contributor

jtopjian commented Nov 5, 2015

Thank you very much for this! I don't use ports, so the initial commit was going off of assumptions and devstack. Perfect timing so it gets cleaned up for the next release.

I'll review all commits in detail later today.

@Fodoj Fodoj force-pushed the add-fixed-ips-to-openstack-ports branch from 69f06c1 to 47762cf Compare November 5, 2015 18:22
@jtopjian
Copy link
Contributor

jtopjian commented Nov 5, 2015

Everything else looks good!

I made the appropriate changes to the subnets locally and the tests pass, so it's good to go after that. I'll commit once you take care of that.

Great addition - thank you!

@Fodoj
Copy link
Author

Fodoj commented Nov 5, 2015

@jtopjian do you mean ${openstack_networking_subnet_v2.foo.id} change? Is that enough to fix tests?

@jtopjian
Copy link
Contributor

jtopjian commented Nov 5, 2015

Yeah, for the modification in resource_openstack_networking_network_v2_tests.go and in the documentation, that's all.

For resource_openstack_networking_port_v2_tests.go, a few changes are needed:

  1. Import the subnet package
  2. Declare a subnet variable
  3. Add a subnet block to the test
  4. Change the hard-coded subnet ID to a variable reference
  5. Add a testAccCheckNetworkingV2SubnetExists alongside the other resources in that test.

Let me know if you need a hand 😄

@Fodoj
Copy link
Author

Fodoj commented Nov 6, 2015

@jtopjian thanks for your review! Changes to tests are pushed.

@jtopjian
Copy link
Contributor

jtopjian commented Nov 6, 2015

Looks good!

jtopjian added a commit that referenced this pull request Nov 6, 2015
provider/openstack: Add fixed ips to openstack ports
@jtopjian jtopjian merged commit 1656e28 into hashicorp:master Nov 6, 2015
@Fodoj Fodoj deleted the add-fixed-ips-to-openstack-ports branch November 8, 2015 16:54
@ghost
Copy link

ghost commented Apr 30, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants