Skip to content

reef: cephfs-shell: prints warning, hangs and aborts when launched#58088

Merged
lxbsz merged 1 commit intoceph:reeffrom
rishabh-d-dave:wip-66502-reef
Jul 4, 2024
Merged

reef: cephfs-shell: prints warning, hangs and aborts when launched#58088
lxbsz merged 1 commit intoceph:reeffrom
rishabh-d-dave:wip-66502-reef

Conversation

@rishabh-d-dave
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/66502


backport of #55711
parent tracker: https://tracker.ceph.com/issues/64538

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

Currently, cephfs-shell prints warnings, hangs and aborts when launched.
This occurs because Python module "pkg_resources" has been deprecated.
We use that module only checking version of other Python modules used in
cephfs-shell. Use "Version" from "packaging.version" instead.

Fixes: https://tracker.ceph.com/issues/64538
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 68b5766)
@rishabh-d-dave rishabh-d-dave added this to the reef milestone Jun 17, 2024
@rishabh-d-dave rishabh-d-dave added the cephfs Ceph File System label Jun 17, 2024
@rishabh-d-dave rishabh-d-dave requested a review from a team June 17, 2024 12:09
@joscollin joscollin added the wip-jcollin-testing-reef1 Assigned for review label Jun 25, 2024
@joscollin
Copy link
Member

This PR is under test in https://tracker.ceph.com/issues/66685.

Copy link
Member

@lxbsz lxbsz left a comment

Choose a reason for hiding this comment

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

The qa test failures are not related, please see https://tracker.ceph.com/issues/66685#note-3.

@lxbsz lxbsz merged commit 091453d into ceph:reef Jul 4, 2024
@rishabh-d-dave rishabh-d-dave deleted the wip-66502-reef branch July 4, 2024 18:20
@joscollin joscollin removed the wip-jcollin-testing-reef1 Assigned for review label Jul 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants