Ports added via SIGHUP may stay link down
mwutzke opened this issue · 1 comments
When a port is added to the Faucet configuration via the SIGHUP process, and this then the port is added to the OpenFlow switch, the port will stay 'down' in Faucet, and no flows will be created, and no traffic will pass.
In order to see this issue, 2 items need to occur:
- a port needs to be added to a running Faucet configuration via a SIGHUP
- the port then needs to be added to an OpenFlow switch (eg
ovs-vsctl add-port ...) that is already connected to a Faucet controller.
If any of these conditions do not exist, the setup seems to work. I suspect anything that results in Faucet processing a 'cold start' will not see the issue.
A quick overview of the reproduction is:
starting condition: Faucet is not running, and the OVS bridge does not exist
# 110
create YAML with 1 port defined (port 1)
start Faucet
# 120
update YAML with a 2nd port defined (port 1 and 2)
SIGHUP Faucet
# 130
create OVS bridge and connect to Faucet
create 2 link's in the UP state
attach these 2 link to the OVS bridge
verify: ping between hosts attached to the 2 links fails
The attached sighup.tgz should be able to reproduce the issue on a system which has an OVS installation, and which has Faucet installed (I used a pip install version for 1.9.53devXX).
Using these scripts, the following should show the issue:
cd sighup
./100-all
The fist ping sequence (140-test) will fail.
With this setup, the interesting sequence is when port2 get added (130-ovs):
INFO DPID 100 (0x64) br100 status did not change: Port 2 up status True reason ADD state 0
INFO DPID 100 (0x64) br100 Port 2 state OFPPS_LIVE reset, ignoring in expectation of port down
The equivalent (working) port1 logs are:
INFO DPID 100 (0x64) br100 status change: Port 1 up status True reason ADD state 0
INFO DPID 100 (0x64) br100 Port 1 (1) up
INFO DPID 100 (0x64) br100 Configuring VLAN vlan100 vid:100 untagged: Port 1,Port 2
I suspect the issue is related to the OFPPS_LIVE log, that is only seen for port2 in this setup.