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

hammer: mds: fix stray purging in 'stripe_count > 1' case #8042

Merged
merged 1 commit into from Aug 3, 2016
Merged

hammer: mds: fix stray purging in 'stripe_count > 1' case #8042

merged 1 commit into from Aug 3, 2016

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented Mar 11, 2016

@ukernel ukernel added bug-fix cephfs Ceph File System labels Mar 11, 2016
@ukernel ukernel added this to the hammer milestone Mar 11, 2016
@ghost ghost changed the title mds: fix stray purging in 'stripe_count > 1' case DNM: mds: fix stray purging in 'stripe_count > 1' case Mar 11, 2016
@ghost ghost changed the title DNM: mds: fix stray purging in 'stripe_count > 1' case DNM: hammer: mds: fix stray purging in 'stripe_count > 1' case Mar 11, 2016
@ghost ghost self-assigned this Mar 11, 2016
@smithfarm
Copy link
Contributor

@dachary Hammer backport issue created: http://tracker.ceph.com/issues/15056

@ghost ghost changed the title DNM: hammer: mds: fix stray purging in 'stripe_count > 1' case hammer: mds: fix stray purging in 'stripe_count > 1' case Mar 22, 2016
@ghost ghost changed the title hammer: mds: fix stray purging in 'stripe_count > 1' case DNM: hammer: mds: fix stray purging in 'stripe_count > 1' case Mar 22, 2016
@ghost
Copy link

ghost commented Mar 22, 2016

@ukernel could you please git cherry-pick -x 99ec183 instead so that it has a reference to the commit that was merged in master ?

1 similar comment
@ghost
Copy link

ghost commented Mar 29, 2016

@ukernel could you please git cherry-pick -x 99ec183 instead so that it has a reference to the commit that was merged in master ?

@ukernel
Copy link
Contributor Author

ukernel commented Mar 30, 2016

Sorry for the delay.

Correspond code was modified a lot and was moved to different file. But I can add "cherry picked from ..." to the commit message

@ukernel
mds: fix stray purging in 'stripe_count > 1' case
3ecdedd 
Fixes: ceph#15050
Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit 99ec183)
@ghost ghost changed the title DNM: hammer: mds: fix stray purging in 'stripe_count > 1' case hammer: mds: fix stray purging in 'stripe_count > 1' case Mar 30, 2016
ghost pushed a commit that referenced this pull request Apr 5, 2016
Merge pull request #8042: hammer: mds: fix stray purging in 'stripe_c…
9c4181c 
…ount > 1' case

Reviewed-by: Loic Dachary <ldachary@redhat.com>
smithfarm added a commit that referenced this pull request Jun 1, 2016
@smithfarm
Merge pull request #8042: hammer: mds: fix stray purging in 'stripe_c…
f38f206 
…ount > 1' case

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jun 27, 2016
@smithfarm
Merge pull request #8042: hammer: mds: fix stray purging in 'stripe_c…
2a5d029 
…ount > 1' case

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jul 18, 2016
@smithfarm
Merge pull request #8042: hammer: mds: fix stray purging in 'stripe_c…
d130c55 
…ount > 1' case

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jul 24, 2016
@smithfarm
Merge pull request #8042: hammer: mds: fix stray purging in 'stripe_c…
f52323e 
…ount > 1' case

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
Copy link
Contributor

@gregsfortytwo This PR passed a cephfs integration run with two failures. Both failures are reproducible in both hammer-backports and hammer itself. One failure is due to http://tracker.ceph.com/issues/11258 ("cephfs-java ftruncate unit test failure") which is fixed in master, but the hammer backport is non-trivial. The second failure is probably an issue with some of the tests. For details, see http://tracker.ceph.com/issues/15895#note-21

Do you think this PR can be merged?

@gregsfortytwo
Copy link
Member

Those test failures don't look like a problem to me.

@smithfarm
Copy link
Contributor

@jcsp OK to merge?

@jcsp jcsp merged commit d34ac2a into ceph:hammer Aug 3, 2016
@ukernel ukernel deleted the hammer-15050 branch January 12, 2017 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug-fix cephfs Ceph File System
Projects
None yet
Milestone
hammer
4 participants
@ukernel @smithfarm @gregsfortytwo @jcsp