Skip to content
Snippets Groups Projects
Commit 9a2123a7 authored by Alex Reimers's avatar Alex Reimers
Browse files

Merge pull request #332 from Sovietaced/master

Fixed inconsistent switch representations in log messages
parents b9bba4a0 760376f4
No related branches found
No related tags found
No related merge requests found
......@@ -554,7 +554,7 @@ IFloodlightModule, IInfoProvider, IHAListener {
if (ofpPort == null) {
if (log.isTraceEnabled()) {
log.trace("Null physical port. sw={}, port={}", sw, port);
log.trace("Null physical port. sw={}, port={}", HexString.toHexString(sw), port);
}
return;
}
......@@ -572,7 +572,7 @@ IFloodlightModule, IInfoProvider, IHAListener {
if (log.isTraceEnabled()) {
log.trace("Sending LLDP packet out of swich: {}, port: {}",
sw, port);
HexString.toHexString(sw), port);
}
// using "nearest customer bridge" MAC address for broadest possible propagation
......@@ -836,7 +836,7 @@ IFloodlightModule, IInfoProvider, IHAListener {
if (!remoteSwitch.portEnabled(remotePort)) {
if (log.isTraceEnabled()) {
log.trace("Ignoring link with disabled source port: switch {} port {}", remoteSwitch, remotePort);
log.trace("Ignoring link with disabled source port: switch {} port {}", remoteSwitch.getStringId(), remotePort);
}
return Command.STOP;
}
......@@ -844,13 +844,13 @@ IFloodlightModule, IInfoProvider, IHAListener {
remotePort))) {
if (log.isTraceEnabled()) {
log.trace("Ignoring link with suppressed src port: switch {} port {}",
remoteSwitch, remotePort);
remoteSwitch.getStringId(), remotePort);
}
return Command.STOP;
}
if (!iofSwitch.portEnabled(pi.getInPort())) {
if (log.isTraceEnabled()) {
log.trace("Ignoring link with disabled dest port: switch {} port {}", sw, pi.getInPort());
log.trace("Ignoring link with disabled dest port: switch {} port {}", HexString.toHexString(sw), pi.getInPort());
}
return Command.STOP;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment