Merge pull request #24418 from Lazin/fix/CORE-8392

[CORE-8392] http: Shutdown connection if shutdown_now was called

src/v/cloud_storage_clients/client_pool.cc

@@ -194,7 +194,11 @@ ss::future<> client_pool::stop() {
 }
 
 void client_pool::shutdown_connections() {
-    vlog(pool_log.info, "Shutting down client pool: {}", _pool.size());
+    vlog(
+      pool_log.info,
+      "Shutting down client pool: {} ({} connections leased)",
+      _pool.size(),
+      _leased.size());
 
     _as.request_abort();
     _cvar.broken();

src/v/http/client.cc

@@ -202,6 +202,11 @@ ss::future<reconnect_result_t> client::get_connected(
             vlog(
               ctxlog.debug,
               "Stopping connect attempts due to shutdown request");
+            if (is_valid()) {
+                // We might have established connection at this point
+                // which has to be closed.
+                shutdown();
+            }
             co_return reconnect_result_t::timed_out;
         }
         current = ss::lowres_clock::now();