Skip to content

Conversation

@ismirlia
Copy link
Collaborator

This pull request deprecates name fields for id fields. In either case (before and after this change), the user can put either the name or id and resource will be fetched. The reason for this change is that APP Config team is fetching tons of resources and sometimes if you use the name the cache could fail or fetch stale data. The ID is more reliable. This change is to encourage the user to use IDs instead of names.

Deprecate the use of pi_instance_name on the ibm_pi_instance, ibm_pi_instance_console_languages, ibm_pi_instance_volumes, ibm_pi_instance_ip, and ibm_pi_pvm_snapshot data sources for pi_instance_id.

Output from acceptance testing:

--- PASS: TestAccIBMPIInstanceDataSource_basic (44.64s)
PASS

--- PASS: TestAccIBMPIInstanceConsoleLanguages (18.56s)
PASS

--- PASS: TestAccIBMPIInstanceVolumesDataSource_basic (23.19s)
PASS

--- PASS: TestAccIBMPISnapshotDataSource_basic (18.25s)
PASS

--- PASS: TestAccIBMPIInstanceIPDataSource_basic (22.42s)
PASS

Update documentation

Add ibm_pi_instance_ip data source

Remove non zero check
@ismirlia ismirlia requested a review from yussufsh October 16, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

service/Power Systems Issues related to Power Systems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants