remove misleading comment...
attach() and detach() are never invoked directly,
plus it's really not that hard to figure out what they do.
make with_container_netns take the container as the first arg
and invoke the subsequent command.
This is closer to typical 'with' semantics.
refactor: don't extract param to var and shift unless we have to
refactor: it's not an IPADDR but a CIDR
refactor: remove add_nat_rule
it did too little to be worthwhile
refactor: simplify 'attach'
Move the interface into the container network NS asap, which
simplifies error handling.
refactor: simplify multicast route addition in 'attach'
Merge pull request #152 from rade/weave_script_tidy

various simplifications and tidying up in weave script
remove explicit multicast detection
We don't need. Multicast addresses never appear as sources and hence
never end up in the peer_cache. And the router always broadcasts
frames with destination addresses that aren't in the peer cache.
Merge pull request #153 from rade/91_simplify_multicast_detection

remove explicit multicast detection

Closes #91