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
vstart.sh: drop .ceph_port and use randomly selected available port #19268
Conversation
while [ true ] | ||
do | ||
CEPH_PORT="$(echo $(( RANDOM % 1000 + 40000 )))" | ||
ss -a -n | egrep ":${CEPH_PORT} .+LISTEN" 1>/dev/null 2>&1 || break |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test is only partially useful since another process that happened to pick the same port could be racing to start daemons after doing this test.
could you prefix the title of your commit message with the subcomponent your are changing ? see https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#3-describe-your-changes. we don't use "cleanup" as the prefix, in general. |
09d6b6f
to
8de3dde
Compare
@tchaikov Like now? |
@shinobu-x i'd suggest |
vstart.sh becomes independent of do_cmake.sh Signed-off-by: Shinobu Kinjo <shinobu@redhat.com>
8de3dde
to
5a3eed7
Compare
Are there any other users of that file? |
@gregsfortytwo What is |
vstart.sh becomes independent of do_cmake.sh
Signed-off-by: Shinobu Kinjo shinobu@redhat.com