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

bug fix: test case for lfn index #4154

Merged
1 commit merged into from Mar 24, 2015
Merged

Conversation

XinzeChi
Copy link
Contributor

tests: TestFlatIndex.cc races with TestLFNIndex.cc
Both use the same PATH and when run in parallel they sometime conflict.

I am not so sure whether is the reason causing the loic-bot fun fail.
Such as #4113, http://paste2.org/pgBxscKB.

Fixes: #11217
Signed-off-by: Xinze Chi xmdxcxz@gmail.com

@loic-bot
Copy link

SUCCESS: the output of run-make-check.sh on centos-7 for 3b5575a is http://paste2.org/HysbsNhF

:octocat: Sent from GH.

@majianpeng
Copy link
Member

@XinzeChi. Now i met the like problem: for cp it can't find file. I don't know the gtest. Are you mean 2 test of the same test-case paralleled run?

@XinzeChi
Copy link
Contributor Author

yes, i think so.

@ghost ghost added bug-fix core labels Mar 24, 2015
@ghost ghost self-assigned this Mar 24, 2015
@ghost
Copy link

ghost commented Mar 24, 2015

Nice catch @XinzeChi ! The better fix is to make it so TestFlatIndex.cc and TestLFNIndex.cc do not use the same PATH . Could you change this fix according please ?

@ghost
Copy link

ghost commented Mar 24, 2015

I created http://tracker.ceph.com/issues/11217, could you please add Fixes: #11217 in the commit message ?

@majianpeng
Copy link
Member

I see, thanks loci.

@XinzeChi XinzeChi force-pushed the wip-test-lfn-index branch 3 times, most recently from 4cf5298 to 87cd3d6 Compare March 24, 2015 09:59
@loic-bot
Copy link

FAIL: the output of run-make-check.sh on centos-7 for a507a8ea31189ace07a5de1c1f7c1d0dac6d8793 is http://paste2.org/nfwzxt0Y

:octocat: Sent from GH.

@loic-bot
Copy link

FAIL: the output of run-make-check.sh on centos-7 for f91ccfabed5883c45f7bca7a8288ce29f5bdfd4d is http://paste2.org/HA4KLOYV

:octocat: Sent from GH.

tests: TestFlatIndex.cc races with TestLFNIndex.cc
Both use the same PATH and when run in parallel they sometime conflict.

Fixes: ceph#11217
Signed-off-by: Xinze Chi <xmdxcxz@gmail.com>
@loic-bot
Copy link

SUCCESS: the output of run-make-check.sh on centos-7 for f418846 is http://paste2.org/F8IKMkgG

:octocat: Sent from GH.

@XinzeChi
Copy link
Contributor Author

@dachary , Please review. Thanks.

ghost pushed a commit that referenced this pull request Mar 24, 2015
bug fix: test case for lfn index

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit 257ba2b into ceph:master Mar 24, 2015
@ghost ghost added the tests label Mar 24, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants