On 02/15/2012 03:08 AM, Phantomcircuit wrote:
# libvirtd --version
libvirtd (libvirt) 0.9.8
# which ebtables
/sbin/ebtables
2012-02-15 09:40:59.083+0000: 17675: error :
ebtablesCreateRuleInstance:1943 : internal error cannot create rule
since ebtables tool is missing.
2012-02-15 09:40:59.107+0000: 17675: error : virNetDevGetIndex:656 :
Unable to get index for interface vnet0: No such device
ebtables_cmd_path = virFindFileInPath("ebtables");
I can't figure out why ebtables_cmd_path is NULL when the call to
ebtablesCreateRuleInstance is made.
Does the $PATH in use by libvirtd include /sbin by default on your
system? Not all systems put /sbin on the path.
Hmm, looks like we have a couple bugs:
1. configure.ac sets EBTABLES_PATH, but only adds /usr/sbin, and not
/sbin, to the PATH used to find it, but then defaults to /sbin/ebtables
if the PATH search failed. This is fragile (the fallback value should
be tested for existence before hard-coding it into the executable).
2. src/util/ebtables.c uses EBTABLES_PATH, while
src/nwfilter/nwfilter_ebiptables_driver.c only uses a PATH lookup of the
raw "ebtables". These should be consolidated to use the same program
(that is, if the user did ./configure EBTABLES=/path/to/alternate, we
should prefer that name rather than half-and-half).
Patches welcome.
--
Eric Blake eblake(a)redhat.com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org