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

hammer: common: linking to -lrbd causes process startup times to balloon #8538

Merged
merged 1 commit into from Jun 27, 2016

Conversation

Vicente-Cheng
Copy link
Contributor

Move initialization to the few tests that actually use it.

Fixes: http://tracker.ceph.com/issues/15225
Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
(cherry picked from commit 1c2831a)

  Conflicts:
	src/test/msgr/perf_msgr_client.cc
	src/test/msgr/perf_msgr_server.cc
	src/test/perf_local.cc
            These three files were not introduced on hammer, just remove
@badone
Copy link
Contributor

badone commented Apr 12, 2016

lgtm

1 similar comment
@yuyuyu101
Copy link
Member

lgtm

@Vicente-Cheng
Copy link
Contributor Author

@badone @yuyuyu101
thanks for review : )

@tchaikov tchaikov added this to the hammer milestone Apr 12, 2016
@ghost ghost self-assigned this May 16, 2016
smithfarm added a commit that referenced this pull request Jun 1, 2016
…rtup times to balloon

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

@jdurgin This passed an rbd suite - see http://tracker.ceph.com/issues/15895#note-2 for details. Do you think it's OK to merge?

The master commit is 1c2831a - it does not have a PR but it looks like you merged it, that's why I'm asking you.

smithfarm added a commit that referenced this pull request Jun 27, 2016
…rtup times to balloon

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@jdurgin
Copy link
Member

jdurgin commented Jun 27, 2016

lgtm

@smithfarm smithfarm merged commit dac65d0 into ceph:hammer Jun 27, 2016
@smithfarm smithfarm changed the title hammer: Linking to -lrbd causes process startup times to balloon hammer: common: linking to -lrbd causes process startup times to balloon Aug 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants