Skip to content

Commit

Permalink
Remove the restriction that the sys tenant's password cannot be empty (
Browse files Browse the repository at this point in the history
…oceanbase#625)

* Rename Dockerfile to DockerFile

* remove the restriction that the sys tenant's password cannot be empty
  • Loading branch information
Teingi authored Dec 12, 2024
1 parent 4bf8ba5 commit b67cfbb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/common/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -168,8 +168,8 @@ def update_obcluster_nodes(self, config):

ob_cluster = {"db_host": cluster_config.get("db_host"), "db_port": cluster_config.get("db_port"), "tenant_sys": {"user": cluster_config.get("tenant_sys", {}).get("user"), "password": cluster_config.get("tenant_sys", {}).get("password")}}

if not all(ob_cluster.values()) or not all(ob_cluster['tenant_sys'].values()):
raise ValueError("Missing required configuration values in ob_cluster or tenant_sys")
if not ob_cluster["db_host"] or not ob_cluster["db_port"] or not ob_cluster["tenant_sys"]["user"]:
raise ValueError("Missing required configuration values in ob_cluster or tenant_sys (excluding password)")

if config_data.get('obcluster') and config_data.get('obcluster').get('servers') and config_data.get('obcluster').get('servers').get('nodes'):
return
Expand Down

0 comments on commit b67cfbb

Please sign in to comment.