Skip to content

Remove the exclude_test_cases list and vNIC‑specific test filtering.#387

Open
FarooqAbdulla02 wants to merge 1 commit intolop-devops:masterfrom
FarooqAbdulla02:fix/pci-info-improvements
Open

Remove the exclude_test_cases list and vNIC‑specific test filtering.#387
FarooqAbdulla02 wants to merge 1 commit intolop-devops:masterfrom
FarooqAbdulla02:fix/pci-info-improvements

Conversation

@FarooqAbdulla02
Copy link
Contributor

  • Remove the commented exclude_inputs_params block.
  • Simplify configuration file handling by using shutil.copy() for all interface types.
  • Reduce overall code complexity from 688 to 661 lines.
  • All configuration files (vnic, veth, hnv, pci) are now copied directly without any filtering, making the code more maintainable and consistent.
  • All existing functionality remains intact, including HMC integration (manageSystem, VIOS information), credential handling (hmc_pwd, vios_pwd), remote peer discovery through SSH, and dynamic parameter injection from files.

- Remove the commented exclude_inputs_params block.
- Simplify configuration file handling by using shutil.copy() for all interface types.
- Reduce overall code complexity from 688 to 661 lines.
- All configuration files (vnic, veth, hnv, pci) are now copied directly without any filtering,
  making the code more maintainable and consistent.
- All existing functionality remains intact, including HMC integration (manageSystem, VIOS information),
  credential handling (hmc_pwd, vios_pwd), remote peer discovery through SSH,
  and dynamic parameter injection from files.

Signed-off-by: “Shaik <“shaik.abdulla1@ibm.com”>
@FarooqAbdulla02
Copy link
Contributor Author

FarooqAbdulla02 commented Mar 5, 2026

command output:
[root@ltcden7-lp23-new tests]# python pci_info.py --vnic --params-file adapter_list.txt --create-config
23:25:25 INFO : Loaded 3 param(s) from params file: adapter_list.txt
23:25:25 INFO : HMC IP detected via lsrsrc IBM.MCP: 10.48.33.38
23:25:25 INFO : Auto-detected HMC IP: 10.48.33.38
23:25:25 INFO : Remote mode: connecting to 10.48.35.44 as root
23:25:25 INFO : RemoteRunner ready for root@10.48.35.44:22 (sshpass mode)
23:25:25 INFO : Local LPAR name detected: ltcden7-lp23-Farooq_RH_10.2
23:25:25 INFO : HMCClient ready for hscroot@10.48.33.38:22
23:25:26 INFO : Managed systems on 10.48.33.38: ['ltcfuji9ys', 'ltcden7', 'ltcden4', 'ltcden3', 'ltcd41']
23:25:26 INFO : LPARs on ltcfuji9ys: ['ltcfuji9ys-vios1', 'ltcfuji9ys-lp1', 'ltcfuji9ys-lp2', 'ltcfuji9ys-lp3', 'ltcfuji9ys-lp4', 'ltcfuji9ys-lp5', 'ltcfuji9ys-lp6']
23:25:26 INFO : LPARs on ltcden7: ['ltcden7-lp23-Farooq_RH_10.2', 'ltcden7-lp10-nvme-tcp-soft-target', 'ltcden7-lp22-Vaishnavi', 'ltcden7-lp20-jenkins-backup', 'ltcden7-lp18-CR', 'ltcden7-lp17-CR', 'ltcden7-lp14-Naresh', 'ltcden7-lp15-Vaishnavi_RH10', 'ltcden7-lp21-Pavaman', 'ltcden7-lp13-Pavaman-MOFED', 'ltcden7-lp12-nvmf', 'ltcden7-lp11-Pavaman', 'ltcden7-lp16-Naresh', 'ltcden7-lp9-Pavaman', 'ltcden7-lp8-Pavaman', 'ltcden7-lp7-Pavaman', 'ltcden7-vios1', 'ltcden7-lp6-Pavaman', 'ltcden7-lp5-Farooq_RH_10', 'ltcden7-lp4-gcov-rh96-Naresh', 'ltcden7-lp3-CI-LPM', 'ltcden7-lp2-Bluefin2-Murthy', 'ltcden7-lp1-Farooq_vNIC_bug', 'ltcden7-lp19-Vaishnavi_rh10']
23:25:26 INFO : LPAR 'ltcden7-lp23-Farooq_RH_10.2' found on managed system 'ltcden7'
23:25:26 INFO : Managed system resolved: ltcden7
23:25:26 INFO : HMCClient ready for hscroot@10.48.33.38:22
23:25:27 INFO : VIOS on 'ltcden7': names=ltcden7-vios1 ltcden7-vios1 ips=10.48.35.39
23:25:27 INFO : VIOS names: ltcden7-vios1 ltcden7-vios1 VIOS IPs: 10.48.35.39
23:25:27 INFO : Derived host_ip=192.168.10.62 host_ips=192.168.10.62 192.168.20.62 from public_interface_ip=10.48.35.62
23:25:27 INFO : Gathering peer vNIC details from remote host 10.48.35.44
23:25:28 INFO : Peer info injected: peer_ip=192.168.10.30 peer_ips=192.168.10.30 192.168.150.177 peer_interfaces=env6 env8 peer_public_ip=10.48.35.44
23:25:28 INFO : python avocado-setup.py --run-suite host_io_vnic_stress_fvt --input-file io_input.txt

@FarooqAbdulla02
Copy link
Contributor Author

we can hold and close this as #388 has improved code base.

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.

1 participant