tyler.durden
/
icinga2
mirror of https://github.com/Icinga/icinga2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Cli: Fix confusing error messages in repository <type> add 

fixes #7530
This commit is contained in:
Michael Friedrich 2014-10-31 20:27:10 +01:00
parent 6596192f2c
commit b60adade1a
1 changed files with 5 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
lib/cli/nodeupdateconfigcommand.cpp
View File

@ -97,7 +97,7 @@ int NodeUpdateConfigCommand::Run(const boost::program_options::variables_map& vm
Dictionary::Ptr host_services = make_shared<Dictionary>(); Dictionary::Ptr host_services = make_shared<Dictionary>();
Log(LogInformation, "cli") Log(LogInformation, "cli")
<< "Repository for node '" << endpoint << "' does not contain a health check host. Adding host '" << zone << "'."; << "Adding host '" << zone << "' to the repository.";
Dictionary::Ptr host_attrs = make_shared<Dictionary>(); Dictionary::Ptr host_attrs = make_shared<Dictionary>();
host_attrs->Set("__name", zone); host_attrs->Set("__name", zone);
@ -107,10 +107,8 @@ int NodeUpdateConfigCommand::Run(const boost::program_options::variables_map& vm
host_imports->Add("satellite-host"); //default host node template host_imports->Add("satellite-host"); //default host node template
host_attrs->Set("import", host_imports); host_attrs->Set("import", host_imports);
if (!RepositoryUtility::AddObject(zone, "Host", host_attrs)) { if (!RepositoryUtility::AddObject(zone, "Host", host_attrs))
Log(LogCritical, "cli") continue;
<< "Cannot add node host '" << zone << "' to the config repository!\n";
}
ObjectLock olock(repository); ObjectLock olock(repository);
BOOST_FOREACH(const Dictionary::Pair& kv, repository) { BOOST_FOREACH(const Dictionary::Pair& kv, repository) {
@ -165,10 +163,7 @@ int NodeUpdateConfigCommand::Run(const boost::program_options::variables_map& vm
host_imports->Add("satellite-host"); //default host node template host_imports->Add("satellite-host"); //default host node template
host_attrs->Set("import", host_imports); host_attrs->Set("import", host_imports);
if (!RepositoryUtility::AddObject(host, "Host", host_attrs)) { RepositoryUtility::AddObject(host, "Host", host_attrs);
Log(LogCritical, "cli")
<< "Cannot add node host '" << host << "' to the config repository!\n";
}
} }
/* special condition: what if the host was blacklisted before, but the services should be generated? */ /* special condition: what if the host was blacklisted before, but the services should be generated? */
@ -226,11 +221,8 @@ int NodeUpdateConfigCommand::Run(const boost::program_options::variables_map& vm
service_imports->Add("satellite-service"); //default service node template service_imports->Add("satellite-service"); //default service node template
service_attrs->Set("import", service_imports); service_attrs->Set("import", service_imports);
if (!RepositoryUtility::AddObject(service, "Service", service_attrs)) { if (!RepositoryUtility::AddObject(service, "Service", service_attrs))
Log(LogCritical, "cli")
<< "Cannot add node host '" << host << "' to the config repository!\n";
continue; continue;
}
} }
} }