Skip to content

Conversation

@r-patange
Copy link
Contributor

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Output from acceptance testing:

--- PASS: TestAccIBMPIInstanceNetworkDataSource_basic (19.50s)
PASS
--- PASS: TestAccIBMPIInstanceNetworksDataSource_basic (18.58s)
PASS
...

Bump github.com/IBM-Cloud/power-go-client to pick up the InstanceNetworks client needed by the new data sources.
Register ibm_pi_instance_network and ibm_pi_instance_networks in the provider. Add new arg/attr constants (ip_address, href, version) used by these data sources.
Add data sources for a single instance network (Get) and all networks (GetAll). Include flattenPvmInstanceNetworksv2 to expose external_ip, ip_address, mac_address, network IDs, href, and version. Single-network id is <cloud>/<instance>/<network>; list id is <cloud>/<instance>.
Cover id composition and presence of key attributes (network_id, network_interface_id, ip_address).
Add docs for d/pi_instance_network and d/pi_instance_networks with example usage, arguments, and attributes.
…wervs-ibm/terraform-provider-ibm into remove-pi-instance-network-datasource
@ismirlia
Copy link
Collaborator

This PR needs to be merged on December 9th after Q4 have been released in the service broker. Please start reviewing them now so that they'll be ready to merge by then. @hkantare and @yussufsh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge service/Power Systems Issues related to Power Systems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants