Say 'name' instead of 'identity' in log and status
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
Merge branch 'master' into 130_launch_without_ip

Conflicts:
	weave