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

Fix OSError when checking the mount status #3963

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

Conversation

cliping
Copy link
Contributor

@cliping cliping commented Jul 29, 2024

Wrong nfs server cleanup order causes OSError. So update code.

Wrong nfs server cleanup order causes OSError. So update code.

Signed-off-by: lcheng <[email protected]>
@cliping cliping changed the title Debug OSError Fix OSError when checking the mount status Sep 2, 2024
@cliping
Copy link
Contributor Author

cliping commented Sep 2, 2024

Before:

2024-08-30 22:27:40,658 process L0470 DEBUG| [stdout] nfsd /proc/fs/nfsd nfsd rw,relatime 0 0
2024-08-30 22:27:40,658 process L0470 DEBUG| [stdout] 127.0.0.1:/var/lib/avocado/data/avocado-vt/images /var/lib/libvirt/migrate nfs4 rw,relatime,vers=4.2,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,timeo=600,retrans=2,sec=sys,clientaddr=127.0.0.1,local_lock=none,addr=127.0.0.1 0 0
2024-08-30 22:27:40,658 output L0702 DEBUG| [stderr] Exception in thread
2024-08-30 22:27:40,658 output L0702 DEBUG| [stderr] cat /proc/mounts-stdout
2024-08-30 22:27:40,659 output L0702 DEBUG| [stderr] :
2024-08-30 22:27:40,659 output L0702 DEBUG| [stderr] Traceback (most recent call last):
2024-08-30 22:27:40,659 output L0702 DEBUG| [stderr] File "/usr/lib64/python3.9/threading.py", line 980, in _bootstrap_inner
2024-08-30 22:27:40,660 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,660 output L0702 DEBUG| [stderr] self.run()
2024-08-30 22:27:40,660 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,660 output L0702 DEBUG| [stderr] File "/usr/lib64/python3.9/threading.py", line 917, in run
2024-08-30 22:27:40,661 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,661 output L0702 DEBUG| [stderr] self._target(*self._args, **self._kwargs)
2024-08-30 22:27:40,661 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,661 output L0702 DEBUG| [stderr] File "/var/ci/libvirt-ci/runtest/avocado-vt/avocado/avocado/utils/process.py", line 488, in _drainer
2024-08-30 22:27:40,662 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,662 output L0702 DEBUG| [stderr] tmp = os.read(self.fd, 8192)
2024-08-30 22:27:40,662 output L0702 DEBUG| [stderr]
2024-08-30 22:27:40,662 output L0702 DEBUG| [stderr] OSError
2024-08-30 22:27:40,663 output L0702 DEBUG| [stderr] :
2024-08-30 22:27:40,663 output L0702 DEBUG| [stderr] [Errno 9] Bad file descriptor
2024-08-30 22:27:40,663 output L0702 DEBUG| [stderr]

After:
(1/1) type_specific.io-github-autotest-libvirt.virsh.live_migration.listen_address.all_ipaddrs.ipv4.migrateuri_ipv6_addr.non_p2p_live.without_postcopy: PASS (112.43 s)

@cliping cliping marked this pull request as draft September 19, 2024 10:34
@cliping cliping marked this pull request as ready for review October 11, 2024 03:54
@cliping cliping marked this pull request as draft October 23, 2024 01:11
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