search

TeemIp / teemip-cable-mgmt

Documents physical connections between devices in TeemIP
GNU General Public License v3.0
5 stars 1 forks source link

Can't create second direct cable connection between two devices. #6

Closed snapet closed 1 month ago

snapet commented 1 month ago

Hi,

Thanks for this plugin we really like it.

As mentioned in the title we think there may be a bug in the logic to add "Direct Network Cable Objects. After connecting a cable between a NAS and a switch I fail to enter/connect a second cable in that order (from NAS to switch). When I select the switch first and then the NAS that works. But we have a third cable running from the NAS to the switch and that's where my luck runs out. There is no way to add a cable to that NAS anymore as we keep getting the error: The object cannot be saved : A cable is already connecting the same network sockets

Best regards, Peter Screenshot Direct Network Connection error

xtophe38 commented 1 month ago

Hello,

Cool to read that you like the extension !

Are you trying to connect your NAS and Switch through the same interfaces or different ones ? Note that, by default, TeemIp considers that only one cable can be attached to a given interface...

snapet commented 1 month ago

Hi again,

Through different interfaces or ports on both NAS and switch. That I wouldn't be able to connect two cables through the same interface makes perfect sense to me!

cheers, peter

On Wed, Oct 2, 2024 at 12:28 PM xtophe38 @.***> wrote:

Hello,

Cool to read that you like the extension !

Are you trying to connect your NAS and Switch through the same interfaces or different ones ? Note that, by default, TeemIp considers that only one cable can be attached to a given interface...

— Reply to this email directly, view it on GitHub https://github.com/TeemIp/teemip-cable-mgmt/issues/6#issuecomment-2388309934, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGD77RY3XRAJG2VUQOCUCJLZZPDDNAVCNFSM6AAAAABPFQ3O6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBYGMYDSOJTGQ . You are receiving this because you authored the thread.Message ID: @.***>

xtophe38 commented 1 month ago

Hello,

I managed to reproduce your issue ! I've corrected it and took the opportunity to review and improve the controls done when cables are created. Correction will be part of next revision of the extensio. In the mean time, you may get the updated files here, from GtHub.

Related commit is this one.

snapet commented 1 month ago

Thank you very much for solving this so quickly!