yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-10-27 00:40:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(tableau): prevents warning in case of site admin creator role (#12175)

Browse Source
This commit is contained in:
Sergio Gómez Villamor 2024-12-19 20:26:58 +01:00 committed by GitHub
parent 45ace13fe2
commit acb76cd97c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py
View File

@ -645,7 +645,7 @@ def report_user_role(report: TableauSourceReport, server: Server) -> None:
# the site-role might be different on another site
logged_in_user: UserInfo = UserInfo.from_server(server=server)
if not logged_in_user.is_site_administrator_explorer():
if not logged_in_user.has_site_administrator_explorer_privileges():
report.warning(
title=title,
message=message,

4
metadata-ingestion/src/datahub/ingestion/source/tableau/tableau_constant.py
View File

@ -82,4 +82,6 @@ PROJECT = "Project"
SITE = "Site"
IS_UNSUPPORTED_CUSTOM_SQL = "isUnsupportedCustomSql"
SITE_PERMISSION = "sitePermission"
SITE_ROLE = "SiteAdministratorExplorer"
ROLE_SITE_ADMIN_EXPLORER = "SiteAdministratorExplorer"
ROLE_SITE_ADMIN_CREATOR = "SiteAdministratorCreator"
ROLE_SERVER_ADMIN = "ServerAdministrator"

8
metadata-ingestion/src/datahub/ingestion/source/tableau/tableau_server_wrapper.py
View File

@ -11,8 +11,12 @@ class UserInfo:
site_role: str
site_id: str
def is_site_administrator_explorer(self):
return self.site_role == c.SITE_ROLE
def has_site_administrator_explorer_privileges(self):
return self.site_role in [
c.ROLE_SITE_ADMIN_EXPLORER,
c.ROLE_SITE_ADMIN_CREATOR,
c.ROLE_SERVER_ADMIN,
]
@staticmethod
def from_server(server: Server) -> "UserInfo":

2
metadata-ingestion/src/datahub/ingestion/source/tableau/tableau_validation.py
View File

@ -28,7 +28,7 @@ def check_user_role(
try:
# TODO: Add check for `Enable Derived Permissions`
if not logged_in_user.is_site_administrator_explorer():
if not logged_in_user.has_site_administrator_explorer_privileges():
capability_dict[c.SITE_PERMISSION] = CapabilityReport(
capable=False,
failure_reason=f"{failure_reason} Their current role is {logged_in_user.site_role}.",