aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorinnokentii <innokentii@ydb.tech>2023-12-05 10:21:02 +0300
committerinnokentii <innokentii@ydb.tech>2023-12-05 10:44:25 +0300
commit147cae18db4087d5cf7aebdf9a0c25fc46292155 (patch)
tree4200cabd2e0a26fa5bc38ce27b4c6b0e8df8e9c0
parent7bf3df81755926d2fce972d4ac113e4c0886d008 (diff)
downloadydb-147cae18db4087d5cf7aebdf9a0c25fc46292155.tar.gz
Fix converter behaviour for multiple tenants
fix
-rw-r--r--ydb/library/yaml_config/console_dumper.cpp12
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;