wlanslovenija / nodewatcher

A modular open networks growing platform.
http://nodewatcher.net/
Other
63 stars 20 forks source link

Add Edgerouter X device descriptor #39

Closed robimarko closed 7 years ago

robimarko commented 7 years ago

Pull request to add Ubiquti Edgerouter X device descriptor. This time properly made in separate branch.

robimarko commented 7 years ago

@kostko Does Nodewatcher include CPU port in ports variable?

kostko commented 7 years ago

@kostko Does Nodewatcher include CPU port in ports variable?

No, the ports must include the CPU port identifier, otherwise this is an error. This is why you are getting the following exception:

ImproperlyConfigured: Switch descriptor 'sw0' refers to an invalid CPU port '6'!

You configured ports=5 on the switch, but the switch as you defined it actually has 7 ports (e.g., 0, 1, 2, 3, 4, 5, 6), not just 5. So it should be ports=7.

robimarko commented 7 years ago

I get I know. I did not know that it has to include CPU port as well.

According to Lede source and actuall running Lede on ERX port 5 is not being used. And port 6 is CPU.

Ok,will edit config to 7 ports. Have you checked out Unifi PR#38?

kostko commented 7 years ago

According to Lede source and actuall running Lede on ERX port 5 is not being used.

You do not need to specify sequential ports, instead of ports=7 you can say ports=[0, 1, 2, 3, 4, 6] and then port 5 will be invalid.

robimarko commented 7 years ago

Thanks,that makes sense. Also thanks for patience

robimarko commented 7 years ago

Should be fixed now

robimarko commented 7 years ago

Just figured out that I had wrong device profile for making defined. Now it is per LEDE profile. @kostko @mitar Any chance of merging?

robimarko commented 7 years ago

I have squashed previous commits into one.

robimarko commented 7 years ago

@kostko Is ramips_mt7621 builder ready?

kostko commented 7 years ago

@kostko Is ramips_mt7621 builder ready?

Yes.

robimarko commented 7 years ago

Sure, that would be great.