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.

Sign up for GitHub

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

Jump to bottom

gwcli: fix the erasure pool profile's display info #240

Merged
merged 1 commit into from
Aug 12, 2021
Merged

gwcli: fix the erasure pool profile's display info #240

merged 1 commit into from
Aug 12, 2021

Conversation

lxbsz
Copy link
Member

@lxbsz lxbsz commented Jul 23, 2021

Show the pool profile instead of size/min_size.

Signed-off-by: Xiubo Li [email protected]

@lxbsz
Copy link
Member Author

lxbsz commented Jul 23, 2021 

[root@node01 ceph-iscsi]# gwcli ls
o- / ......................................................................................................................... [...]
  o- cluster ......................................................................................................... [Clusters: 1]
  | o- ceph .......................................................................................................... [HEALTH_WARN]
  |   o- pools .......................................................................................................... [Pools: 6]
  |   | o- cephfs.a.data ......................................................... [(x3), Commit: 0.00Y/29826642K (0%), Used: 0.00Y]
  |   | o- cephfs.a.meta ....................................................... [(x3), Commit: 0.00Y/29826642K (0%), Used: 133931b]
  |   | o- datapool ............................................................... [(x3), Commit: 2M/29826642K (0%), Used: 171996b]
  |   | o- device_health_metrics ................................................. [(x3), Commit: 0.00Y/29826642K (0%), Used: 0.00Y]
  |   | o- ecpool ................................................................. [(2+2), Commit: 0.00Y/59653284K (0%), Used: 24K]
  |   | o- rbd .................................................................. [(x3), Commit: 0.00Y/29826642K (0%), Used: 21263b]
  |   o- topology ................................................................................................ [OSDs: 3,MONs: 3]
  o- disks .......................................................................................................... [2M, Disks: 2]


[root@node01 ceph-iscsi]# ceph osd erasure-code-profile get default 
k=2
m=2
plugin=jerasure
technique=reed_sol_van

@lxbsz lxbsz requested a review from idryomov July 23, 2021 00:58
idryomov
idryomov reviewed Jul 23, 2021
View reviewed changes
gwcli/ceph.py Outdated Show resolved Hide resolved
gwcli/ceph.py Outdated Show resolved Hide resolved
gwcli/ceph.py Outdated Show resolved Hide resolved
@lxbsz lxbsz force-pushed the erasure_profile branch from 9d558ff to df415bd Compare July 26, 2021 01:24
@lxbsz lxbsz requested a review from idryomov July 26, 2021 01:25
idryomov
idryomov reviewed Jul 26, 2021
View reviewed changes
gwcli/ceph.py Outdated Show resolved Hide resolved
@lxbsz lxbsz force-pushed the erasure_profile branch from df415bd to 286ba34 Compare July 27, 2021 01:30
idryomov
idryomov reviewed Aug 9, 2021
View reviewed changes
gwcli/ceph.py Outdated Show resolved Hide resolved
gwcli/ceph.py Outdated Show resolved Hide resolved
gwcli/ceph.py Outdated Show resolved Hide resolved
@lxbsz lxbsz requested a review from idryomov August 10, 2021 08:32
idryomov
idryomov reviewed Aug 10, 2021
View reviewed changes
gwcli/ceph.py Outdated Show resolved Hide resolved
@lxbsz
gwcli: fix the erasure pool profile's display info
f9eca6a 
Show the pool profile instead of size/min_size.

Signed-off-by: Xiubo Li <[email protected]>
@lxbsz lxbsz requested a review from idryomov August 10, 2021 23:48
@idryomov
Copy link
Contributor

Have you tested the new version?

@lxbsz
Copy link
Member Author

lxbsz commented Aug 12, 2021

Have you tested the new version?

Yes, I did. It worked for me and show the same info like above.

@idryomov idryomov merged commit 10b7eaa into ceph:master Aug 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@idryomov idryomov idryomov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lxbsz @idryomov