Hello Dan Burkert,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/5046

to review the following change.

Change subject: sasl: disable error messages assertions on el6
......................................................................

sasl: disable error messages assertions on el6

MIT krb5 on RHEL 6 has a bug which means that many of the error messages
from GSSAPI don't get exposed in a reasonable way. So, assertions in
sasl_rpc-test have been failing.

I couldn't find any way to workaround the actual bug and get better
error messages, so this patch just disables those assertions on krb5
1.10 or earlier.

I used a hack to determine the version of Kerberos since spending the
effort to learn enough cmake to do so didn't seem worth the effort for a
test-local purpose.

Change-Id: I1e780e2060dceb322bbbbb772d06b68271ddf675
---
M src/kudu/rpc/sasl_rpc-test.cc
M src/kudu/rpc/sasl_server.cc
2 files changed, 34 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/5046/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1e780e2060dceb322bbbbb772d06b68271ddf675
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>

Reply via email to