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: save projected path into inode_t::stray_prior_path #16150

Merged
merged 1 commit into from Sep 4, 2017

Conversation

Projects
None yet
4 participants
@Vicente-Cheng
Contributor

Vicente-Cheng commented Jul 6, 2017

mds: save projected path into inode_t::stray_prior_path
Otherwise, path string like #10000000000/xxx may get saved into
inode_t::stray_prior_path.

Fixes: http://tracker.ceph.com/issues/20340
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit ba7472c)

Conflicts:
	src/mds/Server.cc
          - remove counter `change_attr` because it does not involved

@Vicente-Cheng Vicente-Cheng changed the title from mds: save projected path into inode_t::stray_prior_path to jewel: mds: save projected path into inode_t::stray_prior_path Jul 6, 2017

@smithfarm smithfarm added this to the jewel milestone Jul 6, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 2, 2017

Contributor

@batrick This PR was included in the latest jewel integration fs run: http://tracker.ceph.com/issues/20613#note-42

There were 4 failures, 3 of which passed on the second try. The one that failed both times was Java-related, and we have ignored these in the past.

OK to merge?

Contributor

smithfarm commented Sep 2, 2017

@batrick This PR was included in the latest jewel integration fs run: http://tracker.ceph.com/issues/20613#note-42

There were 4 failures, 3 of which passed on the second try. The one that failed both times was Java-related, and we have ignored these in the past.

OK to merge?

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 3, 2017

Contributor

This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

Contributor

smithfarm commented Sep 3, 2017

This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

@batrick

This comment has been minimized.

Show comment
Hide comment
@batrick

batrick Sep 4, 2017

Member

Looks good to me.

Member

batrick commented Sep 4, 2017

Looks good to me.

@smithfarm smithfarm merged commit d639aa9 into ceph:jewel Sep 4, 2017

4 checks passed

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
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment