On 10/13/2017 05:44 PM, John Ferlan wrote:
On 10/13/2017 07:32 PM, Michal Privoznik wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1497396
In 0d3d020ba6c4f I've added capability to accept MAC addresses for the API too. However, the implementation was faulty. It needs to lookup the corresponding interface in the domain definition and pass the ifname instead of MAC address.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- src/libxl/libxl_driver.c | 2 +- src/lxc/lxc_driver.c | 2 +- src/openvz/openvz_driver.c | 2 +- src/qemu/qemu_driver.c | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-)
face-palm - so much for my assumptions
Yeah, I facepalmed too when noticing this. Fortunately, the patch is pretty simple.
Reviewed-by: John Ferlan <jferlan@redhat.com>
Pushed. Thanks. Michal