[PHOENIX-6555] Wait for permissions to sync in Permission tests

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.2.0, 5.1.3
• Component/s: core
• Assignee: Istvan Toth
• Reporter: Istvan Toth
• Created: 21/Sep/21 05:46
• Updated: 19/Oct/23 04:33
• Resolved: 22/Oct/21 06:27

Description

We should do something better than adding retries whenever see another failure.

From [e-mail] on a mailing list thread:

In the HBase security unit tests we wait for the zk mediated cache reload to complete before continuing the test. If you don’t do that your test will be flaky. Check that you are doing this.

java.lang.reflect.UndeclaredThrowableException
	at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1747)
	at org.apache.hadoop.hbase.security.User$SecureHadoopUser.runAs(User.java:347)
	at org.apache.phoenix.end2end.BasePermissionsIT.verifyAllowed(BasePermissionsIT.java:787)
	at org.apache.phoenix.end2end.BasePermissionsIT.verifyAllowed(BasePermissionsIT.java:780)
	at org.apache.phoenix.end2end.BasePermissionsIT.lambda$testAutomaticGrantWithIndexAndView$5(BasePermissionsIT.java:1412)
	at org.apache.phoenix.end2end.BasePermissionsIT.retryVerifyOperation(BasePermissionsIT.java:1035)
	at org.apache.phoenix.end2end.BasePermissionsIT.testAutomaticGrantWithIndexAndView(BasePermissionsIT.java:1411)

Issue Links

• relates to: PHOENIX-7077 Fix flaky PermissionNSEnabledWithCustomAccessControllerIT.testSystemTablePermissions test
• links to: GitHub Pull Request #1335
• links to: GitHub Pull Request #1336