Ian Main
2009-Jan-23 19:19 UTC
[Ovirt-devel] [PATCH server] Fix bridge usage after update.
This patch fixes the usage of the ovirtbr after it was changed.
Unfortunately it's still hardcoded but that's the way it is for now.
Signed-off-by: Ian Main <imain at redhat.com>
---
src/task-omatic/taskomatic.rb | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/task-omatic/taskomatic.rb b/src/task-omatic/taskomatic.rb
index 19bd4f0..34bff51 100755
--- a/src/task-omatic/taskomatic.rb
+++ b/src/task-omatic/taskomatic.rb
@@ -283,7 +283,7 @@ class TaskOmatic
# FIXME: get rid of the hardcoded bridge
xml = create_vm_xml(db_vm.description, db_vm.uuid, db_vm.memory_allocated,
db_vm.memory_used, db_vm.num_vcpus_allocated, db_vm.boot_device,
- db_vm.vnic_mac_addr, "ovirtbr0", storagedevs)
+ db_vm.vnic_mac_addr, "ovirtbreth0", storagedevs)
result = node.domainDefineXML(xml.to_s)
raise "Error defining virtual machine: #{result.text}" unless
result.status == 0
--
1.6.0.4
Alan Pevec
2009-Jan-23 23:32 UTC
[Ovirt-devel] Re: [PATCH server] Fix bridge usage after update.
Ian Main wrote:> This patch fixes the usage of the ovirtbr after it was changed. > Unfortunately it's still hardcoded but that's the way it is for now.please put it as a constant at the top of the file> - db_vm.vnic_mac_addr, "ovirtbr0", storagedevs) > + db_vm.vnic_mac_addr, "ovirtbreth0", storagedevs)there are incoming patches to rename it again to breth0 ...