Skip to content

Conversation

@m2je
Copy link
Contributor

@m2je m2je commented Mar 31, 2019

Due to this bug https://bugzilla.redhat.com/show_bug.cgi?id=668830
We need to use localhost.localdomain when running these tests on Jenkins (Centos)
but for Mac OS it should be localhost to pass.
The reason is kerberos principals in this tests are looked up from /etc/hosts
and 127.0.0.1 is resolved to localhost.localdomain rather than localhost on Redhat
KDC sees localhost != localhost.localdomain and as the result test fails with authentication error.
It's also important to note these principals are shared between HDFs and HBase in this mini HBase cluster.
Some more reading https://access.redhat.com/solutions/57330

@karanmehta93
Copy link
Member

@m2je Thanks for the patch and apologies about the delay in review.
I see a lot of common code in the test files, can you also refactor the common code to a base test class instead?

Due to this bug https://bugzilla.redhat.com/show_bug.cgi?id=668830
We need to use `localhost.localdomain` as host name when running these tests on Jenkins (Centos)
but for Mac OS it should be `localhost` to pass.
The reason is kerberos principals in this tests are looked up from /etc/hosts
and a reverse DNS lookup of 127.0.0.1 is resolved to `localhost.localdomain` rather than `localhost` on Centos.
KDC sees `localhost` != `localhost.localdomain` and as the result test fails with authentication error.
It's also important to note these principals are shared between HDFs and HBase in this mini HBase cluster.
Some more reading https://access.redhat.com/solutions/57330
@m2je
Copy link
Contributor Author

m2je commented Apr 28, 2019

@karanmehta93 Done with the comments. Please review again.

Copy link
Member

@karanmehta93 karanmehta93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @m2je
One last nit: can you add apache license on the top of your new file? I will commit it then. Thanks.

@m2je
Copy link
Contributor Author

m2je commented May 23, 2019

@karanmehta93 Done.
Thanks

@karanmehta93 karanmehta93 merged commit e3d28be into apache:master May 24, 2019
@m2je m2je deleted the PHOENIX-5221 branch May 24, 2019 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants