From 7d21b700f5f1437d932d37a17e96fe2097ef9aba Mon Sep 17 00:00:00 2001
From: Ali Mohammed <ali.mohammed@hpe.com>
Date: Tue, 7 Feb 2023 14:21:12 +0100
Subject: [PATCH] always match dispose events - similar to declare
---
maestro/subscription_registry.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/maestro/subscription_registry.c b/maestro/subscription_registry.c
index 9ffb9f85..bd19618b 100644
--- a/maestro/subscription_registry.c
+++ b/maestro/subscription_registry.c
@@ -1631,11 +1631,12 @@ mstro_subscription_selector_check(struct mstro_subscription_ *s,
mstro_status status = MSTRO_NOMATCH;
- if(ev->payload_case==MSTRO__POOL__EVENT__PAYLOAD_DECLARE) {
+ if((ev->payload_case==MSTRO__POOL__EVENT__PAYLOAD_DECLARE) || (ev->payload_case==MSTRO__POOL__EVENT__PAYLOAD_DISPOSE)){
/* declare is special: no CDOID known yet, no attribuites, only
* string name. Efficient subscriptions will subscribe to
* DECLARE_ACK instead. */
- WARN("DECLARE event always matches -- FIXME: only name could be checked (and we don't)\n");
+ /**After dispose, we do not have an entry in the app registry for this cdo. */
+ WARN("DECLARE and DISPOSE events always matches -- FIXME: only name could be checked (and we don't)\n");
return MSTRO_OK;
} else {
/* we always need the CDO id. Unfortunately it's slightly buried in the EV */
--
GitLab