Send netdisco-users mailing list submissions to
[email protected]
To subscribe or unsubscribe via the World Wide Web, visit
https://lists.sourceforge.net/lists/listinfo/netdisco-users
or, via email, send a message with subject or body 'help' to
[email protected]
You can reach the person managing the list at
[email protected]
When replying, please edit your Subject line so it is more specific
than "Re: Contents of netdisco-users digest..."
Today's Topics:
1. Re: Prevent display of up='notPresent' ports in CiscoSB
device port listings (Oliver Gorwits)
2. Re: Prevent display of up='notPresent' ports in CiscoSB
device port listings (Nick Nauwelaerts)
--- Begin Message ---
Hi,
On Thu, 6 Dec 2018 at 20:35, Nic Bernstein <[email protected]> wrote:
> BTW: I totally agree with your comment about not adding/discovering these
> phantom ports, but even though I wrote the darn SNMP/Info/Layer2/CiscoSB.pm
> module, I haven't the foggiest idea how to tell it not to collect such
> monsters.
>
I think a good place to skip these, if you really never want to hear about
them, would be here:
https://github.com/netdisco/netdisco/blob/master/lib/App/Netdisco/Worker/Plugin/Discover/Properties.pm#L164
This code is part of Discover, and has access to the port "up" status and
can check for notPresent in $i_up->{$entry}. I might have time for a patch,
but not right now, so patches welcome :-). Take care to check that $i_up->{
$entry} is defined before doing string equality, of course.
HTH,
regards,
oliver.
--- End Message ---
--- Begin Message ---
about filtering in device overview:
correct, ofcourse only tested with "not present" selected. it seems the diff
made "not present" just another port_state which will be or'ed together in:
$set = $set->search({-or => \@combi});
after reviewing & testing some options, it seems there are a few options.
* making it the same like "admin disabled" (where deselecting no longer shows
it). difference is that ports cant be up + admin disabled. these ports can be
up + notpresent & down + notpresent
* making "not present" a box to tick, so that $set = $set->search({-or =>
\@combi}); gets modified to with something like "and up != "notPresent"
about not discovering:
i'm kinda mixed about this from a correctness point of view (even is ciscosb's
operating almost does nothing correct). this being that, at least for port
channel interfaces, they actually show up in the running config:
interface Port-channel1
shutdown
no snmp trap link-status
and i can find no way to delete them (most likely because the web ui for these
devices has port channels hardcoded). if up & not connected (admin up, port
down) or when in admin shutdown (admin down, port down) they show "notPresent"
in both cases. we dont actually have any that are up/up, so no clue what other
options there are. but since they are in the config, imo it makes sense to also
have them in the ports table.
both have pros & cons, which one is preferred & i'll see if i can whip up a
working patch.
// nick
From: Oliver Gorwits [mailto:[email protected]]
Sent: Thursday, December 6, 2018 22:34
To: [email protected]
Cc: Nick Nauwelaerts <[email protected]>; [email protected]
Subject: Re: [Netdisco] Prevent display of up='notPresent' ports in CiscoSB
device port listings
Hi,
On Thu, 6 Dec 2018 at 20:35, Nic Bernstein
<[email protected]<mailto:[email protected]>> wrote:
BTW: I totally agree with your comment about not adding/discovering these
phantom ports, but even though I wrote the darn SNMP/Info/Layer2/CiscoSB.pm
module, I haven't the foggiest idea how to tell it not to collect such monsters.
I think a good place to skip these, if you really never want to hear about
them, would be here:
https://github.com/netdisco/netdisco/blob/master/lib/App/Netdisco/Worker/Plugin/Discover/Properties.pm#L164
This code is part of Discover, and has access to the port "up" status and can
check for notPresent in $i_up->{$entry}. I might have time for a patch, but not
right now, so patches welcome :-). Take care to check that $i_up->{$entry} is
defined before doing string equality, of course.
HTH,
regards,
oliver.
________________________________
Volg Aquafin op Facebook<https://www.facebook.com/AquafinNV> |
Twitter<https://twitter.com/aquafinnv> |
YouTube<http://www.youtube.com/channel/UCk_4P5BJ-MtEEDCkCsR_KqQ?feature=mhee> |
LinkedIN<http://www.linkedin.com/company/aquafin/products>
In het kader van de uitoefening van onze taken verzamelen we bij Aquafin
persoonsgegevens. Hoe we omgaan met deze gegevens en wat de rechten van de
betrokkenen zijn, kan je nalezen in onze privacy
policy<https://www.aquafin.be/nl-be/privacy-policy>.
P Denk aan het milieu. Druk deze mail niet onnodig af.
--- End Message ---
_______________________________________________
Netdisco mailing list - Digest Mode
[email protected]
https://lists.sourceforge.net/lists/listinfo/netdisco-users
