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.

Sign up for GitHub

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

Jump to bottom

Deprecated ibm_pi_network_port resource warning #437

Closed
christopher-horn opened this issue Aug 10, 2022 · 0 comments · Fixed by #438
Closed

Deprecated ibm_pi_network_port resource warning #437

christopher-horn opened this issue Aug 10, 2022 · 0 comments · Fixed by #438
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Comments

@christopher-horn
Copy link
Contributor

Warning: Deprecated Resource

with module.prepare.ibm_pi_network_port.bastion_vip,
on modules/1_prepare/prepare.tf line 392, in resource "ibm_pi_network_port" "bastion_vip":
392: resource "ibm_pi_network_port" "bastion_vip" {

Resource ibm_pi_network_port is deprecated. Use ibm_pi_network_port_attach to
create & attach a network port to a pvm instance
@yussufsh yussufsh added good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. labels Aug 10, 2022
prb112 added a commit to prb112/ocp4-upi-powervs that referenced this issue Aug 23, 2022
@prb112
Update the bastion network name to pi_network_name ocp-power-automati…
f348e83 
…on#437

As identified in issue ocp-power-automation#437, the bastion network name results in a deprecation warning. 
The PR updates to support the new PowerVS terraform field `pi_network_name` 

Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
@prb112 prb112 mentioned this issue Aug 23, 2022
Merged
ppc64le-cloud-bot pushed a commit that referenced this issue Aug 26, 2022
Merge pull request #438 from prb112/issue-437
f93756a 
Update the bastion network name to pi_network_name #437
@yussufsh yussufsh linked a pull request Nov 7, 2022 that will close this issue
Update the bastion network name to pi_network_name #437 #438
Merged
@yussufsh yussufsh closed this as completed Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update the bastion network name to pi_network_name #437 prb112/ocp4-upi-powervs
2 participants
@yussufsh @christopher-horn