8:36 p.m.
Test attached
Ramon Medeiros (2):
Bug fix #318 Kimchi fails creating new network
Add one more case to network tests
src/kimchi/model/networks.py | 14 +++++++-------
tests/test_model.py | 36 ++++++++++++++++++++++++++++++++----
2 files changed, 39 insertions(+), 11 deletions(-)
--
1.8.3.1
8:36 p.m.
New subject: [PATCH 1/2] Bug fix #318 Kimchi fails creating new network
NetworkModel does not have the method _get_network. So, the method
get_network form NetworksModel become a static method and now are
available to be use in src/kimchi/model/networks.py.
Signed-off-by: Ramon Medeiros <ramonn(a)linux.vnet.ibm.com>
---
src/kimchi/model/networks.py | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/kimchi/model/networks.py b/src/kimchi/model/networks.py
index 88b5ece..9f381e3 100644
--- a/src/kimchi/model/networks.py
+++ b/src/kimchi/model/networks.py
@@ -76,7 +76,7 @@ class NetworksModel(object):
# lookup a free network address for nat and isolated automatically
if not netaddr:
for net_name in self.get_list():
- network = self._get_network(net_name)
+ network = NetworksModel.get_network(self.conn.get(), net_name)
xml = network.XMLDesc(0)
subnet = NetworkModel.get_network_from_xml(xml)['subnet']
subnet and net_addrs.append(ipaddr.IPNetwork(subnet))
@@ -158,7 +158,7 @@ class NetworkModel(object):
self.conn = kargs['conn']
def lookup(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
xml = network.XMLDesc(0)
net_dict = self.get_network_from_xml(xml)
subnet = net_dict['subnet']
@@ -209,17 +209,17 @@ class NetworkModel(object):
return xmlutils.xpath_get_text(xml, xpath)
def activate(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
network.create()
def deactivate(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
if self._get_vms_attach_to_a_network(name, "running"):
raise InvalidOperation("KCHNET0018E", {'name': name})
network.destroy()
def delete(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
if network.isActive():
raise InvalidOperation("KCHNET0005E", {'name': name})
if self._get_vms_attach_to_a_network(name):
@@ -228,8 +228,8 @@ class NetworkModel(object):
self._remove_vlan_tagged_bridge(network)
network.undefine()
- def _get_network(self, name):
- conn = self.conn.get()
+ @staticmethod
+ def get_network(conn, name):
try:
return conn.networkLookupByName(name)
except libvirt.libvirtError:
--
1.8.3.1
8:46 p.m.
New subject: [PATCH 1/2] Bug fix #318 Kimchi fails creating new network
On 02/18/2014 03:36 PM, Ramon Medeiros wrote:
NetworkModel does not have the method _get_network. So, the method
get_network form NetworksModel become a static method and now are
available to be use in src/kimchi/model/networks.py.
Signed-off-by: Ramon Medeiros <ramonn(a)linux.vnet.ibm.com>
---
src/kimchi/model/networks.py | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/kimchi/model/networks.py b/src/kimchi/model/networks.py
index 88b5ece..9f381e3 100644
--- a/src/kimchi/model/networks.py
+++ b/src/kimchi/model/networks.py
@@ -76,7 +76,7 @@ class NetworksModel(object):
# lookup a free network address for nat and isolated automatically
if not netaddr:
for net_name in self.get_list():
- network = self._get_network(net_name)
+ network = NetworksModel.get_network(self.conn.get(), net_name)
i
typed incorrectly, this line is:
network = NetworkModel.get_network(self.conn.get(), net_name)
It's NetworkModel, without the s
xml = network.XMLDesc(0)
subnet = NetworkModel.get_network_from_xml(xml)['subnet']
subnet and net_addrs.append(ipaddr.IPNetwork(subnet))
@@ -158,7 +158,7 @@ class NetworkModel(object):
self.conn = kargs['conn']
def lookup(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
xml = network.XMLDesc(0)
net_dict = self.get_network_from_xml(xml)
subnet = net_dict['subnet']
@@ -209,17 +209,17 @@ class NetworkModel(object):
return xmlutils.xpath_get_text(xml, xpath)
def activate(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
network.create()
def deactivate(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
if self._get_vms_attach_to_a_network(name, "running"):
raise InvalidOperation("KCHNET0018E", {'name': name})
network.destroy()
def delete(self, name):
- network = self._get_network(name)
+ network = self.get_network(self.conn.get(), name)
if network.isActive():
raise InvalidOperation("KCHNET0005E", {'name': name})
if self._get_vms_attach_to_a_network(name):
@@ -228,8 +228,8 @@ class NetworkModel(object):
self._remove_vlan_tagged_bridge(network)
network.undefine()
- def _get_network(self, name):
- conn = self.conn.get()
+ @staticmethod
+ def get_network(conn, name):
try:
return conn.networkLookupByName(name)
except libvirt.libvirtError:
--
Ramon Nunes Medeiros
Software Engineer - Linux Technology Center Brazil
IBM Systems & Technology Group
Phone : +55 19 2132 7878
ramonn(a)br.ibm.com
9:39 p.m.
New subject: [PATCH 1/2] Bug fix #318 Kimchi fails creating new network
On 02/18/2014 03:46 PM, Ramon Medeiros wrote:
On 02/18/2014 03:36 PM, Ramon Medeiros wrote:
> NetworkModel does not have the method _get_network. So, the method
> get_network form NetworksModel become a static method and now are
> available to be use in src/kimchi/model/networks.py.
>
> Signed-off-by: Ramon Medeiros <ramonn(a)linux.vnet.ibm.com>
> ---
> src/kimchi/model/networks.py | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/src/kimchi/model/networks.py b/src/kimchi/model/networks.py
> index 88b5ece..9f381e3 100644
> --- a/src/kimchi/model/networks.py
> +++ b/src/kimchi/model/networks.py
> @@ -76,7 +76,7 @@ class NetworksModel(object):
> # lookup a free network address for nat and isolated
> automatically
> if not netaddr:
> for net_name in self.get_list():
> - network = self._get_network(net_name)
> + network = NetworksModel.get_network(self.conn.get(),
> net_name)
i typed incorrectly, this line is:
network = NetworkModel.get_network(self.conn.get(), net_name)
It's NetworkModel, without the s
Patch seems OK. Just need to fix it
> xml = network.XMLDesc(0)
> subnet =
> NetworkModel.get_network_from_xml(xml)['subnet']
> subnet and net_addrs.append(ipaddr.IPNetwork(subnet))
> @@ -158,7 +158,7 @@ class NetworkModel(object):
> self.conn = kargs['conn']
>
> def lookup(self, name):
> - network = self._get_network(name)
> + network = self.get_network(self.conn.get(), name)
> xml = network.XMLDesc(0)
> net_dict = self.get_network_from_xml(xml)
> subnet = net_dict['subnet']
> @@ -209,17 +209,17 @@ class NetworkModel(object):
> return xmlutils.xpath_get_text(xml, xpath)
>
> def activate(self, name):
> - network = self._get_network(name)
> + network = self.get_network(self.conn.get(), name)
> network.create()
>
> def deactivate(self, name):
> - network = self._get_network(name)
> + network = self.get_network(self.conn.get(), name)
> if self._get_vms_attach_to_a_network(name, "running"):
> raise InvalidOperation("KCHNET0018E", {'name':
name})
> network.destroy()
>
> def delete(self, name):
> - network = self._get_network(name)
> + network = self.get_network(self.conn.get(), name)
> if network.isActive():
> raise InvalidOperation("KCHNET0005E", {'name':
name})
> if self._get_vms_attach_to_a_network(name):
> @@ -228,8 +228,8 @@ class NetworkModel(object):
> self._remove_vlan_tagged_bridge(network)
> network.undefine()
>
> - def _get_network(self, name):
> - conn = self.conn.get()
> + @staticmethod
> + def get_network(conn, name):
> try:
> return conn.networkLookupByName(name)
> except libvirt.libvirtError:
8:36 p.m.
New subject: [PATCH 2/2] Add one more case to network tests
Add a test without subnet while creating a network interface, to test
bug https://github.com/kimchi-project/kimchi/issues/318.
Signed-off-by: Ramon Medeiros <ramonn(a)linux.vnet.ibm.com>
---
tests/test_model.py | 36 ++++++++++++++++++++++++++++++++----
1 file changed, 32 insertions(+), 4 deletions(-)
diff --git a/tests/test_model.py b/tests/test_model.py
index 859f7bd..df6b3d0 100644
--- a/tests/test_model.py
+++ b/tests/test_model.py
@@ -537,21 +537,49 @@ class ModelTests(unittest.TestCase):
inst = model.Model('qemu:///system', self.tmp_store)
with RollbackContext() as rollback:
- name = 'test-network'
+
+ # Regression test:
+ # Kimchi fails creating new network #318
+ name = 'test-network-no-subnet'
networks = inst.networks_get_list()
num = len(networks) + 1
args = {'name': name,
'connection': 'nat',
- 'subnet': '127.0.100.0/24'}
+ 'subnet': ''}
inst.networks_create(args)
rollback.prependDefer(inst.network_delete, name)
networks = inst.networks_get_list()
self.assertEquals(num, len(networks))
+ networkinfo = inst.network_lookup(name)
+ self.assertNotEqual(args['subnet'], networkinfo['subnet'])
+ self.assertEqual(args['connection'],
networkinfo['connection'])
+ self.assertEquals('inactive', networkinfo['state'])
+ self.assertEquals([], networkinfo['vms'])
+ self.assertTrue(networkinfo['autostart'])
+ inst.network_activate(name)
+ rollback.prependDefer(inst.network_deactivate, name)
+
+ networkinfo = inst.network_lookup(name)
+ self.assertEquals('active', networkinfo['state'])
+
+ # test network creation with subnet passed
+ name = 'test-network-subnet'
+
+ networks = inst.networks_get_list()
+ num = len(networks) + 1
+ args = {'name': name,
+ 'connection': 'nat',
+ 'subnet': '127.0.0.0/24'}
+ inst.networks_create(args)
+ rollback.prependDefer(inst.network_delete, name)
+
+ networks = inst.networks_get_list()
+ self.assertEquals(num, len(networks))
networkinfo = inst.network_lookup(name)
- self.assertEquals(args['subnet'], networkinfo['subnet'])
+ self.assertEqual(args['subnet'], networkinfo['subnet'])
self.assertEqual(args['connection'],
networkinfo['connection'])
self.assertEquals('inactive', networkinfo['state'])
self.assertEquals([], networkinfo['vms'])
@@ -564,7 +592,7 @@ class ModelTests(unittest.TestCase):
self.assertEquals('active', networkinfo['state'])
networks = inst.networks_get_list()
- self.assertEquals((num - 1), len(networks))
+ self.assertEquals((num - 2), len(networks))
def test_multithreaded_connection(self):
def worker():
--
1.8.3.1
3739
days inactive
3739
days old
5 comments
2 participants
participants (2)
-
Aline Manera -
Ramon Medeiros