-
Notifications
You must be signed in to change notification settings - Fork 359
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
Networking v2: Add no_fixed_ip to openstack_networking_port_v2 #433
Networking v2: Add no_fixed_ip to openstack_networking_port_v2 #433
Conversation
This commit adds the `no_fixed_ip` argument to the openstack_networking_port_v2 resource which will enable a port to be created without a Fixed IP address set.
Build succeeded.
|
// | ||
// Since we're only concerned about a value of "true", | ||
// GetOk is used. | ||
if _, ok := d.GetOk("no_fixed_ip"); ok { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it's better to hide this logic in the resourcePortFixedIpsV2
? Because it doesn't make sense to run resourcePortFixedIpsV2
if no_fixed_ip
was set.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good to me. See the latest commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a single question. But it doesn't seem to be critical.
Build succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This commit adds the
no_fixed_ip
argument to theopenstack_networking_port_v2 resource which will enable a port to
be created without a Fixed IP address set.
For #429