Made tvm url optional (ydb-platform#1591)

* Made tvm url optional * Python codestyle fix * Changed validation accordingly
domwst · Feb 7, 2024 · 4fba45b · 4fba45b
1 parent fce7307
commit 4fba45b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/ydb/core/protos/config.proto b/ydb/core/protos/config.proto
@@ -1607,7 +1607,7 @@ message TTracingConfig {
     message TBackendConfig {
         message TAuthConfig {
             message TTvmAuth {
-                required string Url = 1;
+                optional string Url = 1;
 
                 required uint32 SelfTvmId = 2;
                 required uint32 TracingTvmId = 3;

diff --git a/ydb/tools/cfg/static.py b/ydb/tools/cfg/static.py
@@ -1133,7 +1133,6 @@ def get_sampling_scope(sampling):
             sampling_scope_pb.MaxBurst = sampling['max_burst']
             return sampling_scope_pb
 
-
         def get_external_throttling(throttling):
             throttling_scope_pb = config_pb2.TTracingConfig.TExternalThrottlingScope()
             selectors = throttling.get("scope")

diff --git a/ydb/tools/cfg/validation.py b/ydb/tools/cfg/validation.py
@@ -153,7 +153,7 @@
                                 secret_file=dict(type="string"),
                                 secret_environment_variable=dict(type="string"),
                             ),
-                            required=["url", "self_tvm_id", "tracing_tvm_id"],
+                            required=["self_tvm_id", "tracing_tvm_id"],
                         )
                     ),
                     required=["tvm"],