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

Test:bencher wrong test margin casuses writes over object_size #6677

Merged
merged 2 commits into from Dec 3, 2015

Conversation

jack-changtao
Copy link

Signed-off-by: Tao Chang changtao@hihuron.com

@jack-changtao jack-changtao changed the title test:bencher.h wrong test margin casuses writes over object_size Test:bencher wrong test margin casuses writes over object_size Nov 23, 2015
@tchaikov tchaikov added the tests label Nov 23, 2015
@tchaikov
Copy link
Contributor

@jack-changtao the change looks good to me. but the Signed-off line does not look right:

Signed-off-by:changtao changtao@hihuron.com

you might want to change it to

Signed-off-by: Chang Tao <changtao@hihuron.com>

if "Chang" is your first name.

@jack-changtao
Copy link
Author

@tchaikov
I changed the comment! I don't know how to change the commit message. Thanks

@tchaikov
Copy link
Contributor

@jack-changtao use
git commit --amend
to edit your commit message. and
git push -f to update your remote repo

and you need a space after the colon:

Signed-off-by: Tao Chang <changtao@hihuron.com>

@jack-changtao
Copy link
Author

@tchaikov
Thanks very much!

Signed-off-by: Tao Chang <changtao@hihuron.com>
Signed-off-by: Tao Chang <changtao@hihuron.com>
liewegas added a commit that referenced this pull request Dec 3, 2015
rados: bench: fix off-by-one to avoid writing past object_size

Reviewed-by: Kefu Chai <kchai@redhat.com>
@liewegas liewegas merged commit c80945f into ceph:master Dec 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants