myRT-contribs icon indicating copy to clipboard operation
myRT-contribs copied to clipboard

Not allowed to add physical ports

Open jpobeda opened this issue 7 years ago • 3 comments

Hi Maik,

When I run snmpgeneric.php version from https://github.com/RackTables/racktables-contribs I'm able to add the physical ports BUT when I try the version from https://github.com/github138/myRT-contribs/tree/develop-0.20.x with the changes we've discussed via email (to work with PHP 5.4) I can't add the ports even if they were discovered by this plugin.

Please, look at the attached screenshots

Version from https://github.com/RackTables/racktables-contribs snmpgeneric_issue _1

Version from https://github.com/github138/myRT-contribs/tree/develop-0.20.x

snmpgeneric_issue _2

This is the file I'm using in case you want to have a look, I followed your advise.

snmpgeneric.zip

You can also see how both version detect SW Version but don't get SW Type. I thought that this plugin would help to avoid adding every "Unknown objectID", am I right?

jpobeda avatar Jun 13 '17 22:06 jpobeda

Hi Javier,

these ports have the "no connector present" flag set. What are these exactly?

You could try it with( Line: 128)

/* create ports without connector */ $sg_create_noconnector_ports = FALSE; # <--- change to TRUE

Regards, Maik

github138 avatar Jun 14 '17 07:06 github138

I can't test it right now but what about the others without "no connector"?

Besides if you look at the first screenshot you'll see that even the ports that have the "no connector" flag are eligible to be created.

As soon as I can I'll test your recommendation.

Regards,

jpobeda avatar Jun 14 '17 08:06 jpobeda

Hey Maik,

I've just tested the latest version of snmpgeneric.php from branch develop-0.20.x and it works well!

The only one thing that still doesn't do is to match the SW Type. It gets the actual SW Version eg.: 12.4(25a) but I thought it would update SW Type by just matching what's on "SW Version". It's not a big issue anyway.

jpobeda avatar Jul 04 '17 00:07 jpobeda