-
Notifications
You must be signed in to change notification settings - Fork 6k
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
qa: use kclient xattr to lookup client id #42081
Conversation
qa/tasks/cephfs/kernel_mount.py
Outdated
@@ -202,18 +180,54 @@ def read_debug_file(self, filename): | |||
return None | |||
raise | |||
|
|||
def _get_global_id(self): | |||
try: | |||
p = self.run_shell_payload(f"getfattr --only-values -n ceph.client_id .", stdout=StringIO()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you not call getxattr() directly from python? I'd think that would be possible, and it would be more efficient.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're probably thinking of the python programs that call getxattr for us. Those are transmitted to the "client" host and then executed. It would probably be less efficient than calling /bin/getfattr
.
https://pulpito.ceph.com/pdonnell-2021-06-29_14:48:22-fs-master-distro-basic-smithi/ will test more widely in a future qa run |
jenkins test make check |
* refs/pull/42081/head: qa: use kclient xattr to lookup client id qa: refactor reading debug file code qa: get mount id before failing fs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Otherwise getxattr will hang. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
No need to invoke Python to read a file! Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* refs/pull/42081/head: qa: use kclient xattr to lookup client id qa: refactor reading debug file code qa: get mount id before failing fs
* refs/pull/42081/head: qa: use kclient xattr to lookup client id qa: refactor reading debug file code qa: get mount id before failing fs
Added relatively recently but in RHEL 8.4. Fixes: https://tracker.ceph.com/issues/51183 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
https://tracker.ceph.com/projects/cephfs/wiki/Master#2021-July-01 with small follow-on fix in test branch. |
No description provided.