-
Notifications
You must be signed in to change notification settings - Fork 676
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
DS: VPC Floating IP #1794
DS: VPC Floating IP #1794
Conversation
floatingIPZone: { | ||
Type: schema.TypeString, | ||
Computed: true, | ||
ConflictsWith: []string{isFloatingIPTarget}, |
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.
Remove ConflictsWith..its a computed field
floatingIPTarget: { | ||
Type: schema.TypeString, | ||
Computed: true, | ||
ConflictsWith: []string{isFloatingIPZone}, |
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.
Remove ConflictsWith
return err | ||
} | ||
|
||
floatingIPOptions := &vpcclassicv1.ListFloatingIpsOptions{} |
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.
Use the option start parameter....which we added for other datasources for List
start
string
A server-supplied token determining what resource to start the page on
check other datasources
return err | ||
} | ||
|
||
floatingIPOptions := &vpcv1.ListFloatingIpsOptions{} |
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.
Same as above comment
} | ||
} | ||
|
||
return nil |
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.
If not name matches return fmt.Error("No floatingIP found with name....")
} | ||
} | ||
|
||
return nil |
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.
If not name matches return fmt.Error("No floatingIP found with name....")
} | ||
|
||
``` | ||
|
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.
Add Argument Reference section (required variables)
|
||
The following attributes are exported: | ||
* `id` - An alphanumeric value identifying the floating ip. | ||
* `name` - Name for the floating IP address. |
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.
Remove name from attribute reference section
keys = [ibm_is_ssh_key.testacc_sshkey.id] | ||
} | ||
|
||
data "ibm_is_floating_ip" "test" { |
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.
Remove all other resources just keep data "ibm_is_floating_ip"
5b5cbda
to
0371f94
Compare
Test Results: