Changes in debian files

- install: kvm-ifup-custom, kvm-ifdown-custom
- default: SNF_NETWORK_LOG, LOGFILE, TTL
- links: kvm-vif-bridge

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'develop' into debian-develop

Add docs

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Minor fix related to logging in dnshook

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Log table's uplink only if exists

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Move things inside get_info()

* Set GANETI_INSTANCE_NAME
* Set TABLE, INDEV

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add kvm-ifdown-custom script

This should do any cleanup needed related to the interface
that is going down.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Rename kvm-vif-bridge to kvm-ifup-custom

Package will create a symlink until this gets renamed in
upstream Ganeti as well.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix ebtables in vif-custom

..and remove any unused rules (comments) related to ebtables

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Make get_info caclulate ebtables chain names

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Introduce get_info helper method

It gets the exported environment and calculates all needed vars for
a specific interface. Use this function early in scripts and as
soon as you extract each interface info in hooks.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Some minor fixes..

- Use NETWORK_SUBNET6 since this is exported by Ganeti and not
  just SUBNET6
- All logging in helper function
- Do not send GARP or delete neighbor proxy if desired vars are not
  set

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Minor changes related to logging

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Small changes in if* scripts

* Factor out GARP and neighbor proxy action from setup_routed*
* Invoke send_garp in kvm-ifup
* Invoke delete_neighbor_proxy in snf-network-hook

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Use arpsend instead of arping

arpsend comes along with ndsend with vzctl package.
We send only one packet in order not to delay ifup script.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Rename dnshook and fix-net

..to snf-network-dnshook and snf-network-hook correspondingly.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Use env var for logging script

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

hooks-log -> snf-network-log

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Factor out common funtions from dnshook

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Bump version to 0.15.8next

Add ifup-extra script

..in install and default.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'develop' into debian-develop

Create ebtables chains with RETURN policy

This fixes inter-node private networks in case of multicast packets.
Multicast packets are forwarded to all interfaces bridged
on the bridge. Lets assume the following:

prv0 bridge with tap0, tap1, and eth1.
From tap0 comes a multicast packet.
We will see:
 forward IN=tap0 OUT=tap1
 forward IN=tap0 OUT=eth1
 input IN=tap0

In FORWARD chain we have:

 -i tap0 -j FROMTAP0
 -o tap0 -j TOTAP0
 -i tap1 -j FROMTAP1
 -0 tap1 -j TOTAP1

So the first packet will match the first rule. FROMTAP0 does not
drop it since the src MAC is correct and ACCEPTs it!

We fix this by changing the policy to RETURN.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix typo in ebtables

If interface has an IP we used to add a rule that does not allow
packets coming from TAP to have a different IP. There was a typo
in if statement and the rule did never apply.

Still, we do NOT want this rule so we fix typo and comment the
ebtables rule.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Do not add ebtables rule for dhcp response

nfdhpcd opens a socket and binds it with tap interface. So dhcp
response will NOT go though the bridge and ebtables rule is
not needed.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Refactor ifup-extra script

1) Look for all kind of tags (interface specific or not):
    some-prefix:1:mail
    some-prefix:snf-nic-12345:mail
    some-prefix:8252fabd-1021-411c-b8f7-ed79ed509bb8:mail
    some-prefix:mail

2) some-prefix must be other than synnefo:network:

3) introduce setup_extra and clean_extra functions.

4) clean_extra must remove all possible rules that setup_extra could
   have previously added.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Setup ebtables only for mac filtered setups

We used to drop DHCP requests for all bridged setups in case of
nfdhcpd. This requires ebtables setup for private vlans as well.

Dropping DHCP requests in case of nfdhcpd is not really needed
because they are dropped in user space
(payload.set_verdict(nfqueue.NF_DROP)) if a binding file exists.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix ebtables

1) Add ebtables in INPUT/OUTPUT chains too
   This is needed because multicast packets
   (e.g. with dst address 01:00:83:xx:xx:xx) do NOT go through forward chain

2) In case of nfdhpcd allow DHCP replies only with src address the
   MAC address of incoming device (e.g. prv0)

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add sample rules in ifup-extra

Here we allows outgoing traffic with destination port 25.

Please note that every rule that is added when the corresponding
tag is found it should always be removed at the beginning of the
script. With other words you should remove stale entries before
adding new.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Support execution of extra script

In case of kvm-ifup, if an extra script (/etc/ganeti/ifup-extra)
exists, execute it before exiting.

An dummy example script is given, that parses instance's tags,
and searches for synnefo:network:<ident>:mail.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Make fix-net use common.sh and correct env vars

Helper functions for getting uplink and eui64 reside in common.sh

New Ganeti provides correct GANETI_NEW_PRIMARY GANETI_OLD_PRIMARY
environment variables. Use them in hooks.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Move fix-net logic from hooks to if-up

In target node we used to run arping and ndsend. Move these
commands to if-up script.

Introduce helper function to get eui64 and uplink.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Use correct env vars in fix-net hook

After fixing HooksEnv and HooksNodes in Ganeti migration opcode
to include both source and target nodes, use them correctly in
fix-net hook.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

return in case expected env vars are not set

In case of IPv6 only setup, IP environment variable should not be
set. Still if we have a routed setup routed_setup_ipv4() gets invoked.

This means that the following command will be invoked:

ip route replace proto static dev tap3 table public

This replaces default route!!

With this patch fixes routed_setup* functions to return early
in case expected environment vars are not set.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove unused dsnhook.conf file

The relevant settings have been moved in /etc/default/snf-network.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

runlocked forgotten ebtables command

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix links and dirs

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Changes in debian files to include dnshook

* Modify MAC_MASK so that it complies with synnefo implementation.
* Add dns network default tag. This tag triggers dnshook.
* Add public network default tag. This is currently not used.
* Add defaults related to nsupdate.
* Install dnshook in /usr/bin and create links under /etc/ganeti/hooks

Private networks across different Ganeti backends have the
same mac prefix; the first 28 bits. The next 4 correspond to the
backend-id. Currently we have limitation of 16 backends. This
will be removed once MAC generation is done in Synnefo and not
in Ganeti side.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Some fixes/changes in dnshook

* Do not exit if something fails
* Split update_* for IPv4/IPv6
* Add hooks-log at crucial points
* Fix typos

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'develop' into debian-develop

Practically rewrite dnshook

Make it more readable.

Introduce specific methods for getting reverse dns info of an IPv4
or eui64.

Parse every nic info and check for DNS_FLAG network flag

Do not explicitly pass zone statement to nsupdate. Let it determine
the correct zone to update based on the rest of the input.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add GRNET's specific dnshook

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Bump version to 0.15.1next

Support firewalls based on NINC index, uuid, names

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Bump version to 0.15.0next

Use the script name when logging

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Update debian files

* Depend on iputils-arping, vzctl
* Rename clear-proxy-ndp to fix-net

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'develop' into debian-develop

Rename clear-proxy-ndp to fix-net

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Refactor clear-proxy-ndp post-migrate hook

* Search all NICs for ip-less routed setup and not only NIC0
* On source node clear proxy ndp
* On target node arping and ndsend

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Debian files changes, needed for runlocked script

Merge branch 'develop' into debian-develop

Serialize the excecution of ebtables processes

ebtables cannot handle multiple userspace ebtables processes running
concurrently. This could lead to failures while setting up or cleaning
up ebtables for VM networks.

ebtables latest release (included in Debian Wheezy) supports ebtables
command seriliazation with the --concurrent cmdline option (which uses
flock).

In order to make sure that snf-network works with older versions of
ebtables, we wrap the ebtables commands around a Python helper script
which serializes the excecution of any processes on a host machine by
waiting until successfully binding on a TCPv4 port on localhost (for
more information see the runlocked script).

Bump version to 0.14.1next

Use devflow

* add devflow.conf
* add base version file

Bump new upstream version

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'master' into debian

Mods to work with debian's xen

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add vif-custom script and split kvm-vif-bridge

Put functions in /usr/lib/snf-network/common.sh

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add custom.sh library under /usr/lib/snf-network

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Bump new version

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Merge branch 'master' into debian

Fix a bug in clear-proxy-ndp and add logging

Signed-off-by: Dimitris Aragriorgs <dimara@grnet.gr>

Add hooks-log script

Signed-off-by: Dimitris Aragriorgs <dimara@grnet.gr>

Merge branch 'master' into debian

Clear existing binding before adding new

This removes the enrty from nfdhcpd runtime (due to inotify).

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Use tag types in clear-proxy-ndp

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Use tag types in kvm-vif-bridge

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add tags types in default file

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix dependencies

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Minor changes: remove vlan files

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove vlan files

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove clear-tap file

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove clear-tap file

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove /etc/snf-network dir

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Edit changelog

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add interfaces example

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix snf-network.examples

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

README refactor

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Minimize README

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove useless files

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Refactor debian tree for minor snf-network package

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Refactor snf-network

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix a bug in generic mode

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix a minor bug in the generic case

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix a bug in snf-network-build-node-infra

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add grnet.conf and generic.conf

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Change conf files and vars

Insteed of infra.conf introduce generic.conf and grnet.conf.
Change PUBLIC_* var into NODE_*.

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add exit 0 to hooks

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add exit 0 in kvm-vif-bridge

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Remove conflicts/provides for grnet-vnode-tool

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix a minor bug

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Replaces nfdhcpd (for mac2eui64)

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Ready for testing

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add PUBLIC_LINK

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Handle correctly the PUBLIC_BRIDGE

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add SETUP conf var

Might be either GRNET or GENERIC

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Add dependecy >=0.7-1 for nfdhcpd

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix kvm-vif-bridge

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>

Fix prv-net-helper bug

Signed-off-by: Dimitris Aragiorgis <dimara@grnet.gr>