[kudu-CR] security-itest: fix assertion for el6

[Todd Lipcon (Code Review)]

Todd Lipcon has submitted this change and it was merged.

Change subject: security-itest: fix assertion for el6
..


security-itest: fix assertion for el6

On el6, the error message when no Kerberos credentials are available
is a little bit different. This just fixes the assertion.

Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Reviewed-on: http://gerrit.cloudera.org:8080/6157
Reviewed-by: Todd Lipcon 
Tested-by: Todd Lipcon 
---
M src/kudu/integration-tests/security-itest.cc
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Todd Lipcon: Looks good to me, approved; Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/6157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] security-itest: fix assertion for el6

[Todd Lipcon (Code Review)]

Todd Lipcon has posted comments on this change.

Change subject: security-itest: fix assertion for el6
..


Patch Set 2: Verified+1

Flaky test that Alexey's fixing

-- 
To view, visit http://gerrit.cloudera.org:8080/6157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: No

[kudu-CR] security-itest: fix assertion for el6

[Todd Lipcon (Code Review)]

Todd Lipcon has posted comments on this change.

Change subject: security-itest: fix assertion for el6
..


Patch Set 2: Code-Review+2

-- 
To view, visit http://gerrit.cloudera.org:8080/6157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: No

[kudu-CR] security-itest: fix assertion for el6

[Todd Lipcon (Code Review)]

Hello Dan Burkert,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: security-itest: fix assertion for el6
..

security-itest: fix assertion for el6

On el6, the error message when no Kerberos credentials are available
is a little bit different. This just fixes the assertion.

Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
---
M src/kudu/integration-tests/security-itest.cc
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/6157/2
-- 
To view, visit http://gerrit.cloudera.org:8080/6157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] security-itest: fix assertion for el6

[Dan Burkert (Code Review)]

Dan Burkert has posted comments on this change.

Change subject: security-itest: fix assertion for el6
..


Patch Set 1: Code-Review+2

LGTM, but the line's too long.  Feel free to carry over with that fix.

-- 
To view, visit http://gerrit.cloudera.org:8080/6157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2a118580ed67f3ead60980740b6bdbc8dfcb0f3e
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No