Added adapter details update and improve DRC validation#3071
Added adapter details update and improve DRC validation#3071maramsmurthy wants to merge 1 commit intoavocado-framework-tests:masterfrom
Conversation
- Added calls to `set_adapter_details(pci)` after each DLPAR add/move operation and after DRMGR PCI/PHB operations to ensure adapter details are refreshed before OS validation. - Updated DRC validation logic in `dlpar_remove` to check for specific `drc_index` in `listhwres` output, preventing false positives when other resources are listed. Signed-off-by: Maram Srimannarayana Murthy <msmurthy@linux.vnet.ibm.com>
| for _ in range(self.num_of_dlpar): | ||
| self.do_drmgr_phb('r') | ||
| self.do_drmgr_phb('a') | ||
| self.set_adapter_details(pci) |
There was a problem hiding this comment.
@maramsmurthy where are we adding the details for /dev/disk/by-id/
There was a problem hiding this comment.
I hope we no need to perform /dev/disk/by-id as it is just needed a namespace check.
Name space check is already available as follows in dlpar.py only
current_namespaces = nvme.get_current_ns_ids(self.contr_name)
if current_namespaces == self.ns_list:
for ns_id in current_namespaces:
status = nvme.get_ns_status(self.contr_name, ns_id)
if not status[0] == 'live' and status[1] == 'optimized':
err_ns.append(ns_id)
else:
self.log.info("following ns not back listing after hot_plug" %
(self.ns_list - current_namespaces))
Please check and let me know if any other changes are required.
|
Test is successful without any issues avocado run --max-parallel-tasks=1 dlpar.py -m dlpar.py.data/dlpar.yamlJOB ID : 7c056a37a32c8ede6713e202d9b735762e2c75af (2/3) dlpar.py:DlparPci.test_drmgr_pci;run-lpar_2-peer_ip-1de0: PASS (647.49 s) |
|
@maramsmurthy @Naresh-ibm any update ? |
|
@maramsmurthy where are we on this PR? |
3 participants
set_adapter_details(pci)after each DLPAR add/move operation and after DRMGR PCI/PHB operations to ensure adapter details are refreshed before OS validation.dlpar_removeto check for specificdrc_indexinlisthwresoutput, preventing false positives when other resources are listed.