Merge branch '62-standing-issues-with-amd_ping-logging-output' into 'main'

Resolve "Standing issues with `AMD_ping` logging output"

Closes #62

See merge request !45

src/lib/detail/impl.cpp

@@ -166,7 +166,7 @@ ping(const server& srv) {
     LOGGER_INFO("RPC (ADM_{}) => {{}}", __FUNCTION__);
     const auto rpc = endp.call("ADM_ping");
 
-    LOGGER_INFO("RPC (ADM_{}) <= {{ retval: {}}}", __FUNCTION__, ADM_SUCCESS);
+    LOGGER_INFO("RPC (ADM_{}) <= {{retval: {}}}", __FUNCTION__, ADM_SUCCESS);
     return ADM_SUCCESS;
 }
 

src/scord/rpc_handlers.cpp

@@ -43,7 +43,11 @@ ADM_ping(hg_handle_t h) {
 
     [[maybe_unused]] margo_instance_id mid = margo_hg_handle_get_instance(h);
 
-    LOGGER_INFO("PING(noargs)");
+    const auto id = remote_procedure::new_id();
+    LOGGER_INFO("RPC ID {} ({}) => {{}}", id, __FUNCTION__);
+
+    LOGGER_INFO("RPC ID {} ({}) <= {{retval: {}}}", id, __FUNCTION__,
+                ADM_SUCCESS);
 
     ret = margo_destroy(h);
     assert(ret == HG_SUCCESS);