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

jewel: mds: optimize getattr file size #21038

Closed
wants to merge 1 commit into from

Conversation

pdvian
Copy link

@pdvian pdvian commented Mar 26, 2018

@smithfarm smithfarm added the cephfs Ceph File System label Mar 26, 2018
@smithfarm smithfarm added this to the jewel milestone Mar 26, 2018
@smithfarm
Copy link
Contributor

@pdvian Please rebase (merge conflict).

@smithfarm smithfarm added the DNM label Mar 31, 2018
@smithfarm
Copy link
Contributor

@batrick How important is this backport in terms of getting it into jewel?

@batrick
Copy link
Member

batrick commented Apr 6, 2018

It is not essential.

@pdvian
Copy link
Author

pdvian commented Apr 12, 2018

@batrick @smithfarm Shall I rebase to resolve conflicts ? or close this PR and jewel backport tracker ?

@smithfarm
Copy link
Contributor

@pdvian Please rebase. The PR may not make it into 10.2.11, that's all.

@pdvian
Copy link
Author

pdvian commented Apr 16, 2018

@smithfarm sure. Let me rebase and push the changes.

Fixes: http://tracker.ceph.com/issues/22925
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit c822b3e)
@pdvian
Copy link
Author

pdvian commented Apr 16, 2018

@smithfarm Rebased and pushed the changes. Kindly review.

@smithfarm smithfarm removed the DNM label Apr 16, 2018
@theanalyst theanalyst requested a review from batrick May 23, 2018 16:11
@smithfarm
Copy link
Contributor

Jewel is EOL

@smithfarm smithfarm closed this Oct 19, 2018
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
4 participants