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.

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

Added EEH test cases for PCI PT NIC devices #6098

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TasmiyaNalatwad
Copy link
Contributor

Added 2 test cases

  1. Inject EEH from host console for pci pt device
  2. Inject EEH from guest console for pci pt device

@TasmiyaNalatwad TasmiyaNalatwad force-pushed the eeh_pci_pt branch 2 times, most recently from 4b1489d to da9d6f6 Compare December 31, 2024 11:26
@TasmiyaNalatwad
Copy link
Contributor Author

Currently eeh from host is supported and it gets into permanent failure after 2 eeh injections, so tested code with the 2 injections , and eeh inside guest is still feature under development, hence it is failing here.

JOB ID     : af4c73f68b4c24a0a193a40c842f6bfcbcac78b2
JOB LOG    : /root/tests/results/job-2024-12-31T05.25-af4c73f/job.log
 (1/2) type_specific.io-github-autotest-libvirt.libvirt_pci_passthrough_eeh.passthrough_eeh_guest.PCI_NIC.Normal_passthrough: STARTED
 (1/2) type_specific.io-github-autotest-libvirt.libvirt_pci_passthrough_eeh.passthrough_eeh_guest.PCI_NIC.Normal_passthrough: ERROR: Shell command failed: 'errinjct ioa-bus-error -f 6 -p U78DA.ND0.YC039H3-P0-C0-T0 -m 0'    (status: 1,    output: 'errinjct: Could not open error injection facility,\nfile "/proc/device-tree/rtas/ibm,errinjct" does not exist\n\n') (42.23 s)
 (2/2) type_specific.io-github-autotest-libvirt.libvirt_pci_passthrough_eeh.passthrough_eeh_host.PCI_NIC.Normal_passthrough: STARTED
 (2/2) type_specific.io-github-autotest-libvirt.libvirt_pci_passthrough_eeh.passthrough_eeh_host.PCI_NIC.Normal_passthrough: FAIL: PCI device ['0048:01:00.0', '0048:01:00.1', '0048:01:00.2', '0048:01:00.3'] recovery failed after 2 EEH (520.54 s)
RESULTS    : PASS 0 | ERROR 1 | FAIL 1 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /root/tests/results/job-2024-12-31T05.25-af4c73f/results.html
JOB TIME   : 798.49 s

Added 2 test cases
1. Inject EEH from host console for pci pt device
2. Inject EEH from guest console for pci pt device

Signed-off-by: Tasmiya Nalatwad <[email protected]>
@TasmiyaNalatwad
Copy link
Contributor Author

The CI Python 3.6 and 3.7 are failing as it is not understanding some of the spellings like My name added as Author and "EEH"
Could someone help me in fixing these spelling errors .

Thank You

libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:12:0: C0401: Wrong spelling of a word 'Tasmiya' in a comment:
# Author: Tasmiya.Nalatwad<[email protected]>
          ^^^^^^^
Did you mean: ''Tasmania''? (wrong-spelling-in-comment)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:12:0: C0401: Wrong spelling of a word 'Nalatwad' in a comment:
# Author: Tasmiya.Nalatwad<[email protected]>
                  ^^^^^^^^
Did you mean: ''Twaddled''? (wrong-spelling-in-comment)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:12:0: C0401: Wrong spelling of a word 'tasmiya' in a comment:
# Author: Tasmiya.Nalatwad<[email protected]>
                           ^^^^^^^
Did you mean: ''tastily''? (wrong-spelling-in-comment)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:[13](https://github.com/autotest/tp-libvirt/actions/runs/12929182565/job/36058035111?pr=6098#step:9:14):0: C0401: Wrong spelling of a word 'EEH' in a comment:
# EEH Test on pci passthrough devices
  ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-comment)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:279:0: C0401: Wrong spelling of a word 'EEH' in a comment:
# Inject EEH error using below command
         ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-comment)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:39:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
    Test EEH functionality on PCI Passthrough device
         ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:51:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        9. Trigger EEH on pci device
                   ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:52:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        10. Check EEH recovery on the device
                  ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:64:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        9. Trigger EEH on pci device
                   ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:65:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        10. Check EEH recovery on the device
                  ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:306:0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        Check if the pci device is recovered successfully after injecting EEH
                                                                          ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)
libvirt/tests/src/passthrough/pci/libvirt_pci_passthrough_eeh.py:3[29](https://github.com/autotest/tp-libvirt/actions/runs/12929182565/job/36058035111?pr=6098#step:9:30):0: C0402: Wrong spelling of a word 'EEH' in a docstring:
        Function to check if EEH is successfully hit
                             ^^^
Did you mean: ''EH' or 'EEL' or 'MEH' or 'EEK''? (wrong-spelling-in-docstring)

@harihare
Copy link
Contributor

@TasmiyaNalatwad can you please fix the failed checks.

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