Commit 326fdbeb authored by Marcin Siodelski's avatar Marcin Siodelski Committed by Francis Dupont
Browse files

[#710,!408] Fixed invalid tabulations in test config backends impl.

parent 22aa7d84
......@@ -562,29 +562,29 @@ TestConfigBackendDHCPv4::createUpdateOption4(const db::ServerSelector& server_se
auto tags = server_selector.getTags();
auto not_in_tags = false;
for (auto subnet : subnets_) {
// Get the pool: if it is not here we can directly go to the next subnet.
// Get the pool: if it is not here we can directly go to the next subnet.
auto pool = subnet->getPool(Lease::TYPE_V4, pool_start_address);
if (!pool) {
continue;
}
// Verify the subnet is in all or one of the given servers.
// Verify the subnet is in all or one of the given servers.
if (!subnet->hasAllServerTag()) {
auto in_tags = false;
auto in_tags = false;
for (auto tag : tags) {
if (subnet->hasServerTag(ServerTag(tag))) {
in_tags = true;
in_tags = true;
break;
}
}
if (!in_tags) {
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
continue;
}
}
// Update the option.
// Update the option.
pool->getCfgOption()->del(option->space_name_, option->option_->getType());
pool->getCfgOption()->add(*option, option->space_name_);
......
......@@ -561,29 +561,29 @@ TestConfigBackendDHCPv6::createUpdateOption6(const db::ServerSelector& server_se
auto tags = server_selector.getTags();
auto not_in_tags = false;
for (auto subnet : subnets_) {
// Get the pool: if it is not here we can directly go to the next subnet.
// Get the pool: if it is not here we can directly go to the next subnet.
auto pool = subnet->getPool(Lease::TYPE_NA, pool_start_address);
if (!pool) {
continue;
}
// Verify the subnet is in all or one of the given servers.
// Verify the subnet is in all or one of the given servers.
if (!subnet->hasAllServerTag()) {
auto in_tags = false;
auto in_tags = false;
for (auto tag : tags) {
if (subnet->hasServerTag(ServerTag(tag))) {
in_tags = true;
in_tags = true;
break;
}
}
if (!in_tags) {
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
continue;
}
}
// Update the option.
// Update the option.
pool->getCfgOption()->del(option->space_name_, option->option_->getType());
pool->getCfgOption()->add(*option, option->space_name_);
......@@ -610,15 +610,15 @@ TestConfigBackendDHCPv6::createUpdateOption6(const db::ServerSelector& server_se
auto tags = server_selector.getTags();
auto not_in_tags = false;
for (auto subnet : subnets_) {
// Get the pd pool: if it is not here we can directly go to the next subnet.
// Get the pd pool: if it is not here we can directly go to the next subnet.
auto pdpool = subnet->getPool(Lease::TYPE_PD, pd_pool_prefix);
if (!pdpool) {
continue;
}
// Verify the subnet is in all or one of the given servers.
// Verify the subnet is in all or one of the given servers.
if (!subnet->hasAllServerTag()) {
auto in_tags = false;
auto in_tags = false;
for (auto tag : tags) {
if (subnet->hasServerTag(ServerTag(tag))) {
in_tags = true;
......@@ -626,13 +626,13 @@ TestConfigBackendDHCPv6::createUpdateOption6(const db::ServerSelector& server_se
}
}
if (!in_tags) {
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
// Records the fact a subnet was found but not in a server.
not_in_tags = true;
continue;
}
}
// Update the option.
// Update the option.
pdpool->getCfgOption()->del(option->space_name_, option->option_->getType());
pdpool->getCfgOption()->add(*option, option->space_name_);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment