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

mgr/dashboard: Evict a CephFS client #28898

Merged
merged 1 commit into from Jul 19, 2019

Conversation

@ricardoasmarques
Copy link
Member

commented Jul 5, 2019

It's now possible to evict (disconnect) a CephFS client, from the UI:

c89bb0fa-5952-45d8-a66d-ad8f01f8adc0

Fixes: https://tracker.ceph.com/issues/24892

Signed-off-by: Ricardo Marques rimarques@suse.com

@ricardoasmarques ricardoasmarques force-pushed the ricardoasmarques:cephfs-client-evict branch 3 times, most recently from 0812d88 to c150016 Jul 5, 2019

@LenzGr LenzGr requested a review from tspmelo Jul 11, 2019

@LenzGr LenzGr added the needs-review label Jul 11, 2019

@tspmelo
Copy link
Contributor

left a comment

Please rebase into master.

@ricardoasmarques ricardoasmarques force-pushed the ricardoasmarques:cephfs-client-evict branch from c150016 to ce3087f Jul 12, 2019

@ricardoasmarques

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2019

@tspmelo Comments addressed, thanks.

@ricardoasmarques ricardoasmarques force-pushed the ricardoasmarques:cephfs-client-evict branch from ce3087f to 3918af5 Jul 12, 2019

@ricardoasmarques ricardoasmarques requested a review from rjfd Jul 16, 2019

@tspmelo
Copy link
Contributor

left a comment

lgtm

@ricardoasmarques ricardoasmarques force-pushed the ricardoasmarques:cephfs-client-evict branch 3 times, most recently from f171bcd to 6fe2025 Jul 17, 2019

mgr/dashboard: Evict a CephFS client
Fixes: https://tracker.ceph.com/issues/24892

Signed-off-by: Ricardo Marques <rimarques@suse.com>

@ricardoasmarques ricardoasmarques force-pushed the ricardoasmarques:cephfs-client-evict branch from 6fe2025 to a74df9b Jul 19, 2019

@ricardoasmarques

This comment has been minimized.

Copy link
Member Author

commented Jul 19, 2019

jenkins retest this please

@rjfd

rjfd approved these changes Jul 19, 2019

Copy link
Contributor

left a comment

lgtm

@rjfd rjfd merged commit 41e7f89 into ceph:master Jul 19, 2019

5 of 6 checks passed

make check (arm64) make check failed
Details
Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
ceph dashboard tests ceph dashboard tests succeeded
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.