From 11639e6e00850adb2b4d8e91f18da68a48324c25 Mon Sep 17 00:00:00 2001
From: Daniel Milroy <daniel.milroy@gmail.com>
Date: Wed, 8 Jan 2025 17:07:42 -0800
Subject: [PATCH] qmanager: remove no-longer-supported schedutil_free response

Problem: the .free RPC no longer receives a response, and
`schedutil_free_respond()` is a noop.

Remove the call and associated error handling from the qmanager free
callback as it's misleading.
---
 qmanager/modules/qmanager_callbacks.cpp | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/qmanager/modules/qmanager_callbacks.cpp b/qmanager/modules/qmanager_callbacks.cpp
index 56251f835..c344aa0fa 100644
--- a/qmanager/modules/qmanager_callbacks.cpp
+++ b/qmanager/modules/qmanager_callbacks.cpp
@@ -367,10 +367,6 @@ void qmanager_cb_t::jobmanager_free_cb (flux_t *h, const flux_msg_t *msg, const
                         static_cast<intmax_t> (id));
         goto done;
     }
-    if (schedutil_free_respond (ctx->schedutil, msg) < 0) {
-        flux_log_error (h, "%s: schedutil_free_respond", __FUNCTION__);
-        goto done;
-    }
 
 done:
     json_decref (Res);