kytos-ng / sdntrace

Napp that traces OpenFlow paths in the dataplane
https://kytos-ng.github.io/api/sdntrace.html
GNU Lesser General Public License v3.0
0 stars 5 forks source link

Review and reassess warning messages #41

Open viniarck opened 1 year ago

viniarck commented 1 year ago

Every trace is being logged as warning, should this be kept as warning? Looks like the intention was to signal to network operators that packet-in/outs can impact but if the volume is low should this still be warning? It seems like a debug message.

2023-02-28 12:25:12,444 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Starting Trace Path ID: 30001
2023-02-28 12:25:12,471 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 
2023-02-28 12:25:12,972 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 
2023-02-28 12:25:13,474 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 
2023-02-28 12:25:13,976 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Trace Completed!
jab1982 commented 1 year ago

This was for troubleshooting during the development.  Jeronimo BezerraOn Feb 28, 2023, at 11:12 AM, Vinicius Arcanjo @.***> wrote: Every trace is being logged as warning, should this be kept as warning? Looks like the intention was to signal to network operators that packet-in/outs can impact but if the volume is low should this still be warning? It seems like a debug message. 2023-02-28 12:25:12,444 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Starting Trace Path ID: 30001 2023-02-28 12:25:12,471 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:12,972 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:13,474 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:13,976 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Trace Completed!

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

viniarck commented 1 year ago

This was for troubleshooting during the development.  Jeronimo BezerraOn Feb 28, 2023, at 11:12 AM, Vinicius Arcanjo @.> wrote: Every trace is being logged as warning, should this be kept as warning? Looks like the intention was to signal to network operators that packet-in/outs can impact but if the volume is low should this still be warning? It seems like a debug message. 2023-02-28 12:25:12,444 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Starting Trace Path ID: 30001 2023-02-28 12:25:12,471 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:12,972 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:13,474 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Sending POut to switch: 00:00:00:00:00:00:00:01 and in_port 1 2023-02-28 12:25:13,976 - WARNING [kytos.napps.amlight/sdntrace] (Dummy-127) Trace 30001: Trace Completed! —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.>

Right. Thanks for confirming, Jeronimo. I suppose we can set them as debug then.