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

jewel: rados/upgrade test fails with git clone https://github.com/ceph/ceph.git /home/ubuntu/cephtest/clone.client.0 ; cd -- /home/ubuntu/cephtest/clone.client.0 && git checkout jewel #12736

Closed
wants to merge 2 commits into from

Conversation

smithfarm
Copy link
Contributor

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 72d73b8)

Conflicts:
	qa/tasks/workunit.py (master does not "mv qa/workunits")
If we checkout ceph-ci.git, and don't find a branch,
we'll try again from ceph.git. But the checkout will
already exist and the clone will fail, so we'll still
fail to find the branch.

The same can happen if a previous workunit task already
checked out the repo.

Fix by removing the repo before checkout (the first and
second times).  Note that this may break if there are
multiple workunit tasks running in parallel on the same
role.  That is already racy, so if it's happening, we'll
want to switch to using a truly unique clonedir for each
instantiation.

Fixes: http://tracker.ceph.com/issues/18336
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 2a7013c)
@smithfarm smithfarm self-assigned this Jan 2, 2017
@smithfarm smithfarm added this to the jewel milestone Jan 2, 2017
@smithfarm
Copy link
Contributor Author

Closing in favor of #12686 (ganging up multiple related backports to avoid merge conflict)

@smithfarm smithfarm closed this Jan 2, 2017
smithfarm pushed a commit to smithfarm/ceph that referenced this pull request Jan 29, 2017
The boost mt code uses uninitialized memory for extra randomness,
which is a bad idea in general but more importantly makes valgrind
unhappy.  Use /dev/urandom instead.

Unfortunately this introduces a link time dependency.. meh!

Fixes: ceph#12736
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit dbcaa54)

Conflicts:
	ceph.spec.in (trivial resolution)
@jan--f jan--f deleted the wip-18376-jewel branch February 8, 2017 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants