Skip to content

Conversation

@keerthi-gp
Copy link
Collaborator

No description provided.

@keerthi-gp keerthi-gp changed the title UI 40655 ppnlb reserved ip feat(ppnlb): Add support for Reserved IP as member target of PPNLB Jan 23, 2025
@keerthi-gp keerthi-gp force-pushed the UI-40655-PPNLB-Reserved-IP branch from 211c2fc to 1d9cd1c Compare January 23, 2025 12:03
scripts/build.sh Outdated
echo "==> Packaging..."
for PLATFORM in $(find ./pkg -mindepth 1 -maxdepth 1 -type d); do
OSARCH=$(basename ${PLATFORM})
ZIPNAME="terraform-provider-ibm_${VERSION}_${OSARCH}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same, this file needs to be restored to master. This is just change to have the version in the binary generated

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert this change

Copy link
Collaborator

@SunithaGudisagarIBM1 SunithaGudisagarIBM1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned #414 doc needs to be changed in alphanumaric format and also attach the test results in the PR..

make testacc TEST=./ibm/service/vpc TESTARGS='-run='
to run acceptance test and also attach the results in the PR

Also Upload Binary to Artifactory and inform CSI team once you have tested in staging env
https://github.com/ibm-vpc/terraform-provider-ibm/wiki/Upload-Binary-to-Artifactory

Otherwise PR looks good to me.

@keerthi-gp
Copy link
Collaborator Author

keerthigp@Keerthis-MacBook-Pro terraform-provider-ibm % make testacc TEST=./ibm/service/vpc TESTARGS='-run=TestAccIBMISLBPoolMember_basic_network_target_reservedIP'

— PASS: TestAccIBMISLBPoolMember_basic_network_target_reservedIP (324.07s)
PASS
ok github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/vpc 325.623s

keerthigp@Keerthis-MacBook-Pro terraform-provider-ibm % make testacc TEST=./ibm/service/vpc TESTARGS='-run=TestAccIBMISLBProfileDatasource_basic'

--- PASS: TestAccIBMISLBProfileDatasource_basic (18.24s)
PASS
ok github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/vpc 19.500s

keerthigp@Keerthis-MacBook-Pro terraform-provider-ibm % make testacc TEST=./ibm/service/vpc TESTARGS='-run=TestAccIBMISLBProfilesDatasource_basic'

--- PASS: TestAccIBMISLBProfilesDatasource_basic (22.67s)
PASS
ok github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/vpc 24.225s

keerthigp@Keerthis-MacBook-Pro terraform-provider-ibm % make testacc TEST=./ibm/service/vpc TESTARGS='-run=TestAccIBMISLBProfilesDatasource_filter'

--- PASS: TestAccIBMISLBProfilesDatasource_filter (20.06s)
PASS
ok github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/vpc 21.634s

Copy link
Collaborator

@uibm uibm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@keerthi-gp keerthi-gp force-pushed the UI-40655-PPNLB-Reserved-IP branch from c13f17f to 26ec8b0 Compare September 25, 2025 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants