Skip to content

Commit

Permalink
do not expose YDB_MON_PORT by default (#14050)
Browse files Browse the repository at this point in the history
  • Loading branch information
adameat authored Jan 31, 2025
1 parent 5953a41 commit 79d9940
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion ydb/public/tools/lib/cmds/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,10 @@ def enable_tls():
return os.getenv('YDB_GRPC_ENABLE_TLS') == 'true'


def report_monitoring_info():
return os.getenv('YDB_REPORT_MONITORING_INFO') == 'true'


def generic_connector_config():
endpoint = os.getenv("FQ_CONNECTOR_ENDPOINT")
if not endpoint:
Expand Down Expand Up @@ -410,7 +414,8 @@ def deploy(arguments):
recipe.write_endpoint(endpoint)
recipe.write_database(cluster.domain_name)
recipe.write_connection_string(("grpcs://" if enable_tls() else "grpc://") + endpoint + "?database=/" + cluster.domain_name)
recipe.write_mon_port(mon_port)
if report_monitoring_info():
recipe.write_mon_port(mon_port)
if enable_tls():
recipe.write_certificates_path(configuration.grpc_tls_ca.decode("utf-8"))
return endpoint, database
Expand Down

0 comments on commit 79d9940

Please sign in to comment.