reset.py assumes iptables command is in PATH. about chomper

[aniketpanjwani]

Good point - thanks! One fix is to just change env PATH=$(PATH) to env PATH=$(PATH):/sbin. That should only alter the PATH variable in the context of the sudo command. However, I'm actually not sure now whether env PATH=$(PATH) is needed in the first place, since we have absolute paths to the interpreter and reset.py. It's pretty late here, so I'll think about what to do about this tomorrow, but happy to hear your thoughts too.

[sarg]

Another option is to use python library for interacting with iptables - https://github.com/ldx/python-iptables

btw, iptables could be located in /usr/sbin too

[aniketpanjwani]

Interesting library, but I'd rather not add on another dependency if it's not necessary.

Another idea: use which iptables to figure out where iptables is located on the system, parse out the directory, and make sure it's on path during the sudo command. Specifically, add the following new variable to the top of the Makefile

IPTABLES_PATH = dirname $(which iptables)

and then modify the reset task as follows

reset:
	sudo env PATH=$(PATH):$(IPTABLES_PATH) ${INTERPRETER}3.6m ${CURRENT_DIR}/chomper/reset.py

[sarg]

which works by searching current PATH. I suggest you to add /sbin:/usr/sbin to env and forget about this ticket until someone reports it again.

[aniketpanjwani]

Yeah you're right - which isn't necessary if it's already on PATH. Let's just do what you're suggesting - add /sbin:/usr/sbin to the env.

Could you make a PR? Thanks!