]> xenbits.xensource.com Git - xen-guest-agent.git/commitdiff
netlink: remove redundant traces
authorYann Dirson <yann.dirson@vates.fr>
Tue, 21 Nov 2023 12:31:14 +0000 (13:31 +0100)
committerYann Dirson <yann.dirson@vates.fr>
Thu, 30 Nov 2023 10:40:29 +0000 (11:40 +0100)
* separately logging header+nlas just duplicates with less info the
  full message log
* the full message itself is already available from netlink crate, as now
  described in the README

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>
src/collector_net_netlink.rs

index 5bda2093b4ab3f19c936d2b6bc11f0328314d2ca..396cfb0c71323695a2d51c3b8ff465ded6eb5c70 100644 (file)
@@ -64,7 +64,6 @@ impl NetworkSource {
         let mut nl_response = self.handle.request(nl_msg, SocketAddr::new(0, 0))?;
         // Handle response
         while let Some(packet) = nl_response.next().await {
-            log::trace!("<<< {:?}", packet);
             if let NetlinkMessage{payload: NetlinkPayload::InnerMessage(msg), ..} = packet {
                 events.push(netevent_from_rtnetlink(&msg)?);
             }
@@ -81,7 +80,6 @@ impl NetworkSource {
         let mut nl_response = self.handle.request(nl_msg, SocketAddr::new(0, 0))?;
         // Handle response
         while let Some(packet) = nl_response.next().await {
-            log::trace!("<<< {:?}", packet);
             if let NetlinkMessage{payload: NetlinkPayload::InnerMessage(msg), ..} = packet {
                 events.push(netevent_from_rtnetlink(&msg)?);
             }
@@ -93,7 +91,6 @@ impl NetworkSource {
     pub fn stream(&mut self) -> impl Stream<Item = io::Result<NetEvent>> + '_ {
         try_stream! {
             while let Some((message, _)) = self.messages.next().await {
-                log::trace!("rtnetlink change message - {message:?}");
                 if let NetlinkMessage{payload: NetlinkPayload::InnerMessage(msg), ..} = message {
                     yield netevent_from_rtnetlink(&msg)?;
                 }
@@ -131,7 +128,6 @@ fn netevent_from_rtnetlink(nl_msg: &RtnlMessage) -> io::Result<NetEvent> {
 
 fn nl_linkmessage_decode(msg: &LinkMessage) -> io::Result<(NetInterface, String)> {
     let LinkMessage{header, nlas, ..} = msg;
-    log::trace!("{header:?} {nlas:?}");
 
     // extract fields of interest
     let mut address_bytes: Option<&Vec<u8>> = None;
@@ -158,7 +154,6 @@ fn nl_linkmessage_decode(msg: &LinkMessage) -> io::Result<(NetInterface, String)
 
 fn nl_addressmessage_decode(msg: &AddressMessage) -> io::Result<(NetInterface, IpAddr)> {
     let AddressMessage{header, nlas, ..} = msg;
-    log::trace!("{header:?} {nlas:?}");
 
     // extract fields of interest
     let mut address_bytes: Option<&Vec<u8>> = None;