diff options
author | innokentii <innokentii@ydb.tech> | 2023-12-05 10:21:02 +0300 |
---|---|---|
committer | innokentii <innokentii@ydb.tech> | 2023-12-05 10:44:25 +0300 |
commit | 147cae18db4087d5cf7aebdf9a0c25fc46292155 (patch) | |
tree | 4200cabd2e0a26fa5bc38ce27b4c6b0e8df8e9c0 | |
parent | 7bf3df81755926d2fce972d4ac113e4c0886d008 (diff) | |
download | ydb-147cae18db4087d5cf7aebdf9a0c25fc46292155.tar.gz |
Fix converter behaviour for multiple tenants
fix
-rw-r--r-- | ydb/library/yaml_config/console_dumper.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/ydb/library/yaml_config/console_dumper.cpp b/ydb/library/yaml_config/console_dumper.cpp index 977be9a34f..5c5550b6e1 100644 --- a/ydb/library/yaml_config/console_dumper.cpp +++ b/ydb/library/yaml_config/console_dumper.cpp @@ -591,7 +591,7 @@ NKikimrConsole::TConfigureRequest DumpYamlConfigRequest(const TString &cItem, co } } - auto prepareActions = [&](auto& result) { + auto prepareActions = [&](auto& result, auto configItemCb) { Y_ABORT_UNLESS(configNode.Type() == NFyaml::ENodeType::Mapping, "Config has to be mapping"); switch (mergeStrategy) { case NKikimrConsole::TConfigItem::MERGE_OVERWRITE_REPEATED: @@ -611,6 +611,7 @@ NKikimrConsole::TConfigureRequest DumpYamlConfigRequest(const TString &cItem, co separateConfig.Root().Map().Append(key, value); auto config = YamlToProto(separateConfig.Root(), true, false); NKikimrConsole::TConfigItem& configItem = *result.AddActions()->MutableAddConfigItem()->MutableConfigItem(); + configItemCb(configItem); configItem.MutableConfig()->CopyFrom(config); configItem.SetCookie(cookie.c_str()); configItem.SetMergeStrategy(mergeStrategy); @@ -619,14 +620,13 @@ NKikimrConsole::TConfigureRequest DumpYamlConfigRequest(const TString &cItem, co // for domain if (tenants.empty()) { - prepareActions(result); + prepareActions(result, [](auto&){}); } for (auto& tenant : tenants) { - prepareActions(result); - for (auto& action : *result.MutableActions()) { - action.MutableAddConfigItem()->MutableConfigItem()->MutableUsageScope()->MutableTenantAndNodeTypeFilter()->SetTenant(tenant); - } + prepareActions(result, [&tenant](auto& configItem) { + configItem.MutableUsageScope()->MutableTenantAndNodeTypeFilter()->SetTenant(tenant); + }); } return result; |