parent_affecting_logging: a config object affects everything function…

…ally depending on it
Icinga · Aug 15, 2023 · c27c309 · c27c309
1 parent 6257627
commit c27c309
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 6 deletions.
diff --git a/lib/icinga/comment.ti b/lib/icinga/comment.ti
@@ -34,12 +34,12 @@ class Comment : ConfigObject < CommentNameComposer
 	load_after Host;
 	load_after Service;
 
-	[config, no_user_modify, protected, required, navigation(host)] name(Host) host_name {
+	[config, no_user_modify, protected, required, navigation(host), parent_affecting_logging] name(Host) host_name {
 		navigate {{{
 			return Host::GetByName(GetHostName());
 		}}}
 	};
-	[config, no_user_modify, protected, navigation(service)] String service_name {
+	[config, no_user_modify, protected, navigation(service), parent_affecting_logging] String service_name {
 		track {{{
 			if (!oldValue.IsEmpty()) {
 				Service::Ptr service = Service::GetByNamePair(GetHostName(), oldValue);

diff --git a/lib/icinga/dependency.ti b/lib/icinga/dependency.ti
@@ -50,13 +50,13 @@ class Dependency : CustomVarObject < DependencyNameComposer
 		}}}
 	};
 
-	[config, no_user_modify, required, navigation(parent_host)] name(Host) parent_host_name {
+	[config, no_user_modify, required, navigation(parent_host), parent_affecting_logging] name(Host) parent_host_name {
 		navigate {{{
 			return Host::GetByName(GetParentHostName());
 		}}}
 	};
 
-	[config, no_user_modify, navigation(parent_service)] String parent_service_name {
+	[config, no_user_modify, navigation(parent_service), parent_affecting_logging] String parent_service_name {
 		track {{{
 			if (!oldValue.IsEmpty()) {
 				Service::Ptr service = Service::GetByNamePair(GetParentHostName(), oldValue);

diff --git a/lib/icinga/downtime.cpp b/lib/icinga/downtime.cpp
@@ -161,6 +161,17 @@ void Downtime::Stop(bool runtimeRemoved)
 	ObjectImpl<Downtime>::Stop(runtimeRemoved);
 }
 
+void Downtime::GetParentsAffectingLogging(std::vector<ConfigObject::Ptr>& output) const
+{
+	ObjectImpl<Downtime>::GetParentsAffectingLogging(output);
+
+	auto object (ConfigObject::GetObject<ScheduledDowntime>(GetConfigOwner()));
+
+	if (object) {
+		output.emplace_back(std::move(object));
+	}
+}
+
 void Downtime::Pause()
 {
 	if (m_CleanupTimer) {

diff --git a/lib/icinga/downtime.hpp b/lib/icinga/downtime.hpp
@@ -71,6 +71,7 @@ class Downtime final : public ObjectImpl<Downtime>
 protected:
 	void Start(bool runtimeCreated) override;
 	void Stop(bool runtimeRemoved) override;
+	void GetParentsAffectingLogging(std::vector<ConfigObject::Ptr>& output) const override;
 
 	void Pause() override;
 	void Resume() override;

diff --git a/lib/icinga/downtime.ti b/lib/icinga/downtime.ti
@@ -25,12 +25,12 @@ class Downtime : ConfigObject < DowntimeNameComposer
 	load_after Host;
 	load_after Service;
 
-	[config, no_user_modify, required, navigation(host)] name(Host) host_name {
+	[config, no_user_modify, required, navigation(host), parent_affecting_logging] name(Host) host_name {
 		navigate {{{
 			return Host::GetByName(GetHostName());
 		}}}
 	};
-	[config, no_user_modify, navigation(service)] String service_name {
+	[config, no_user_modify, navigation(service), parent_affecting_logging] String service_name {
 		track {{{
 			if (!oldValue.IsEmpty()) {
 				Service::Ptr service = Service::GetByNamePair(GetHostName(), oldValue);