[IMPALA-5686] Update to Sentry causes build failures

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Impala 2.10.0
• Fix Version/s: Impala 2.10.0
• Component/s: Infrastructure
• Labels: broken-build
• Epic Color: ghx-label-1
• Assignee: Taras Bobrovytsky
• Reporter: Taras Bobrovytsky
• Created: 20/Jul/17 00:51
• Updated: 24/Jul/17 23:36
• Resolved: 24/Jul/17 23:36

Description

In a recent update to Sentry, the property name "sentry.service.client.server.rpc-address" was changed to "sentry.service.client.server.rpc-addresses". This causes the following exception when starting a mini cluster:

17/07/19 22:15:18 ERROR testutil.SentryServicePinger: Error issuing RPC to Sentry Service (attempt 2/30): 
org.apache.impala.common.InternalException: Error creating Sentry Service client: 
	at org.apache.impala.util.SentryPolicyService$SentryServiceClient.createClient(SentryPolicyService.java:96)
	at org.apache.impala.util.SentryPolicyService$SentryServiceClient.<init>(SentryPolicyService.java:67)
	at org.apache.impala.util.SentryPolicyService.listAllRoles(SentryPolicyService.java:391)
	at org.apache.impala.testutil.SentryServicePinger.main(SentryServicePinger.java:75)
Caused by: sentry.org.apache.sentry.core.common.exception.MissingConfigurationException: Property 'sentry.service.client.server.rpc-addresses' is missing in configuration
	at sentry.org.apache.sentry.core.common.transport.SentryPolicyClientTransportConfig.getSentryServerRpcAddress(SentryPolicyClientTransportConfig.java:73)
	at sentry.org.apache.sentry.core.common.transport.SentryTransportPool.<init>(SentryTransportPool.java:103)
	at org.apache.sentry.service.thrift.SentryServiceClientFactory.<init>(SentryServiceClientFactory.java:83)
	at org.apache.sentry.service.thrift.SentryServiceClientFactory.create(SentryServiceClientFactory.java:65)
	at org.apache.impala.util.SentryPolicyService$SentryServiceClient.createClient(SentryPolicyService.java:94)

This is also causing GVO failures, such as https://jenkins.impala.io/job/gerrit-verify-dryrun/886/