Skip to content

nfs_corrupt:fix findmnt default src issue#4357

Merged
XueqiangWei merged 1 commit intoautotest:masterfrom
qingwangrh:4241_fix_nfs_findmnt_dst
Aug 20, 2025
Merged

nfs_corrupt:fix findmnt default src issue#4357
XueqiangWei merged 1 commit intoautotest:masterfrom
qingwangrh:4241_fix_nfs_findmnt_dst

Conversation

@qingwangrh
Copy link
Copy Markdown
Contributor

The is_mounted search should look for the mount information with the mount destination.

ID:4241

@qingwangrh qingwangrh force-pushed the 4241_fix_nfs_findmnt_dst branch 2 times, most recently from eb3e282 to e88b943 Compare August 11, 2025 10:25
The is_mounted search should look for the mount information with the mount destination.

Signed-off-by: qingwangrh <qinwang@redhat.com>
@qingwangrh qingwangrh force-pushed the 4241_fix_nfs_findmnt_dst branch from e88b943 to 73de2ca Compare August 11, 2025 10:26
@qingwangrh
Copy link
Copy Markdown
Contributor Author

python3 ConfigTest.py --guestname=RHEL.10.1 --testcase=nfs_corrupt.with_qcow2_format --firmware=default_bios --clone=no
(1/1) Host_RHEL.m10.u1.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.1.x86_64.io-github-autotest-qemu.nfs_corrupt.with_qcow2_format.q35: STARTED
(1/1) Host_RHEL.m10.u1.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.1.x86_64.io-github-autotest-qemu.nfs_corrupt.with_qcow2_format.q35: PASS (79.00 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

@qingwangrh
Copy link
Copy Markdown
Contributor Author

Depends on avocado-framework/avocado-vt#4199

@qingwangrh
Copy link
Copy Markdown
Contributor Author

@XueqiangWei @hellohellenmao @fbq815 Please help to review, thanks.

Copy link
Copy Markdown
Contributor

@XueqiangWei XueqiangWei left a comment

Choose a reason for hiding this comment

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

LGTM,
Acked-by: xuwei@redhat.com

@hellohellenmao
Copy link
Copy Markdown
Contributor

I will give it a try after the patch of avocado-framework/avocado-vt#4199 merged(seems there is still some discussions on that patch now). Thanks

Copy link
Copy Markdown
Contributor

@fbq815 fbq815 left a comment

Choose a reason for hiding this comment

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

LGTM:
On RHEL.10 newest release:
(1/3)qemu.nfs_corrupt.with_qcow2_format.s390-virtio: PASS (65.49 s)
(2/3) qemu.nfs_corrupt.with_raw_format.s390-virtio: PASS (63.27 s)
(3/3) qemu.nfs_corrupt.with_luks_format.s390-virtio: STARTED
nfs_corrupt.with_luks_format.s390-virtio: PASS (68.28 s)

On RHEL.9 newest release:
(1/3)qemu.nfs_corrupt.with_qcow2_format.s390-virtio: PASS (69.03 s)
(2/3) qemu.nfs_corrupt.with_raw_format.s390-virtio: PASS (63.29 s)
(3/3) qemu.nfs_corrupt.with_luks_format.s390-virtio: PASS (67.95 s)

@XueqiangWei
Copy link
Copy Markdown
Contributor

@hellohellenmao The dependent pullreq has been merged, could you help review it again? thanks.

@hellohellenmao
Copy link
Copy Markdown
Contributor

(1/1) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.1.x86_64.io-github-autotest-qemu.nfs_corrupt.with_qcow2_format.q35: PASS (90.92 s)

@hellohellenmao
Copy link
Copy Markdown
Contributor

LGTM

@XueqiangWei XueqiangWei merged commit eb6499c into autotest:master Aug 20, 2025
7 checks passed
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