Skip to content

Conversation

@PavamanSubramaniyam
Copy link
Contributor

Development is printing the lsvpd details in tuple format

So accordingly updating the handling in the test script

Development is printing the lsvpd details in tuple format

So accordingly updating the handling in the test script

Signed-off-by: Pavaman Subramaniyam <[email protected]>
@PavamanSubramaniyam
Copy link
Contributor Author

avocado run --max-parallel-tasks=1 ras_lsvpd.py:RASToolsLsvpd.test_pci_lsvpd
JOB ID : 436e08c0f10d181eee22f15c1f8e2147e36ef4dc
JOB LOG : /home/OpTest/avocado-fvt-wrapper/results/job-2025-11-14T08.58-436e08c/job.log
(1/1) ras_lsvpd.py:RASToolsLsvpd.test_pci_lsvpd: STARTED
(1/1) ras_lsvpd.py:RASToolsLsvpd.test_pci_lsvpd: PASS (1.30 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML : /home/OpTest/avocado-fvt-wrapper/results/job-2025-11-14T08.58-436e08c/results.html
JOB TIME : 40.78 s

@PavamanSubramaniyam
Copy link
Contributor Author

I have uploaded the complete result logs file to the Jira.

@Naresh-ibm Naresh-ibm self-assigned this Nov 14, 2025
@Naresh-ibm
Copy link
Collaborator

@PavamanSubramaniyam is this same for all the adpaters?
is not this might break for other adapters I feel
if this is from specific distro version , then please let the older code to be available for older distro and add the conditions accordingly.

@PavamanSubramaniyam
Copy link
Contributor Author

I have verified with FC adapter and Network adapters also and got the same results. The RAS development team confirmed that the new lsvpd versions will have the tuple format introduced and has been picked up by the RHEL distros like RHEL9.7, RHEL9.8, RHEL10.1 and RHEL10.2.

@Naresh-ibm
Copy link
Collaborator

I have verified with FC adapter and Network adapters also and got the same results. The RAS development team confirmed that the new lsvpd versions will have the tuple format introduced and has been picked up by the RHEL distros like RHEL9.7, RHEL9.8, RHEL10.1 and RHEL10.2.

In that case can we make a condition and append the changes, like
if distro.version < 9.7

else:
<old_code>

same for sles too.

let me know if you need any help here.
as people might be still running the older release for testing. So don't want this to break.

@PavamanSubramaniyam
Copy link
Contributor Author

We will any how test going forward with the latest RHEL distro's like RHEL9.8 and RHEL10.2 releases and will be testing mostly the pre-GA distros. Also, this enhancement is giving the full correct output now since the development team has handled the output in the correct format, we can still go ahead with the current changes.

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.

2 participants