Blockade destroy should quit with error if not executed with root privileges

worstcase / blockade

Docker-based utility for testing network failures and partitions in distributed applications

http://blockade.readthedocs.org

Apache License 2.0

906 stars 72 forks source link

Blockade destroy should quit with error if not executed with root privileges #26

Closed szborows closed 7 years ago

szborows commented 8 years ago

Currently the execution happily passes straight into iptables setup and throws meaningless exception from subprocess.check_output - OSError File does not exist

labisso commented 7 years ago

With the (imminent) 0.3.0 release, root privileges are no longer required. Iptables and tc commands are themselves executed in docker containers. So the user just needs the ability to call docker.

It's still not the most beautiful error if the user doesn't have Docker privileges, so I'm going to keep this issue open and add a friendlier error in a future release.