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

do all valgrind runs on centos #16046

Merged
merged 1 commit into from Jul 1, 2017

Conversation

Projects
None yet
2 participants
@liewegas
Member

liewegas commented Jun 30, 2017

We are fighting two issues with valgrind on ubuntu (xenial, yakkety,
and z):

http://tracker.ceph.com/issues/18126
http://tracker.ceph.com/issues/20360

Revert this when it is fixed.

Signed-off-by: Sage Weil sage@redhat.com

do all valgrind runs on centos
We are fighting two issues with valgrind on ubuntu (xenial, yakkety,
and z):

	http://tracker.ceph.com/issues/18126
	http://tracker.ceph.com/issues/20360

Revert this when it is fixed.

Signed-off-by: Sage Weil <sage@redhat.com>
@@ -1,3 +1,5 @@
# see http://tracker.ceph.com/issues/20360 and http://tracker.ceph.com/issues/18126
os_type: centos

This comment has been minimized.

@jdurgin

jdurgin Jun 30, 2017

Member

last time this caused issues since it needed 'os_version: 7' as well, or it defaulted to 14.04

@liewegas

This comment has been minimized.

Member

liewegas commented Jun 30, 2017

@liewegas liewegas merged commit aa986f5 into ceph:master Jul 1, 2017

3 of 4 checks passed

default Build finished.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@liewegas liewegas deleted the liewegas:wip-valgrind-on-centos branch Jul 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment