Skip to content

Commit 6c8a0a1

Browse files
author
Gaurav Aradhye
committed
Fixing few issues in Multiple IPs test cases
1 parent daadc45 commit 6c8a0a1

2 files changed

Lines changed: 33 additions & 10 deletions

File tree

test/integration/component/test_multiple_ips_per_nic.py

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -143,11 +143,11 @@ def setUpClass(cls):
143143
cls.services["service_offering"]
144144
)
145145
cls._cleanup = [cls.service_offering]
146-
cls.services["shared_network_offering"]["specifyVlan"] = "True"
147-
cls.services["shared_network_offering"]["specifyIpRanges"] = "True"
146+
cls.services["shared_network_offering_all_services"]["specifyVlan"] = "True"
147+
cls.services["shared_network_offering_all_services"]["specifyIpRanges"] = "True"
148148

149149
cls.shared_network_offering = CreateEnabledNetworkOffering(cls.api_client,
150-
cls.services["shared_network_offering"])
150+
cls.services["shared_network_offering_all_services"])
151151
cls._cleanup.append(cls.shared_network_offering)
152152
cls.mode = cls.zone.networktype
153153
if cls.mode == "advanced":
@@ -443,7 +443,7 @@ def test_operations_non_root_admin_api_client(self, value):
443443
try:
444444
ipaddress_1 = NIC.addIp(apiclient, id=virtual_machine.nic[0].id)
445445
except Exception as e:
446-
self.fail("Failed while adding secondary IP to NIC of vm %s" % virtual_machine.id)
446+
self.fail("Failed while adding secondary IP to NIC of vm %s: %s" % (virtual_machine.id, e))
447447

448448
try:
449449
NIC.list(apiclient, virtualmachineid=virtual_machine.id)
@@ -494,10 +494,11 @@ def setUpClass(cls):
494494
cls.services["service_offering"]
495495
)
496496
cls._cleanup = [cls.service_offering]
497-
cls.services["shared_network_offering"]["specifyVlan"] = "True"
498-
cls.services["shared_network_offering"]["specifyIpRanges"] = "True"
497+
cls.services["shared_network_offering_all_services"]["specifyVlan"] = "True"
498+
cls.services["shared_network_offering_all_services"]["specifyIpRanges"] = "True"
499499

500-
cls.shared_network_offering = CreateEnabledNetworkOffering(cls.api_client, cls.services["shared_network_offering"])
500+
cls.shared_network_offering = CreateEnabledNetworkOffering(cls.api_client,
501+
cls.services["shared_network_offering_all_services"])
501502
cls._cleanup.append(cls.shared_network_offering)
502503
cls.mode = cls.zone.networktype
503504
if cls.mode == "advanced":
@@ -868,10 +869,11 @@ def setUpClass(cls):
868869
cls.services["service_offering"]
869870
)
870871
cls._cleanup = [cls.service_offering]
871-
cls.services["shared_network_offering"]["specifyVlan"] = "True"
872-
cls.services["shared_network_offering"]["specifyIpRanges"] = "True"
872+
cls.services["shared_network_offering_all_services"]["specifyVlan"] = "True"
873+
cls.services["shared_network_offering_all_services"]["specifyIpRanges"] = "True"
873874

874-
cls.shared_network_offering = CreateEnabledNetworkOffering(cls.api_client, cls.services["shared_network_offering"])
875+
cls.shared_network_offering = CreateEnabledNetworkOffering(cls.api_client,
876+
cls.services["shared_network_offering_all_services"])
875877
cls._cleanup.append(cls.shared_network_offering)
876878
cls.mode = cls.zone.networktype
877879
if cls.mode == "advanced":

tools/marvin/marvin/config/test_data.py

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -296,6 +296,27 @@
296296
"UserData": "VirtualRouter"
297297
}
298298
},
299+
"shared_network_offering_all_services": {
300+
"name":"shaared network offering with services enabled",
301+
"displaytext": "Shard network offering",
302+
"guestiptype": "Shared",
303+
"supportedservices": "Dhcp,Dns,SourceNat,PortForwarding,Vpn,Firewall,Lb,UserData,StaticNat",
304+
"specifyVlan" : "False",
305+
"specifyIpRanges" : "False",
306+
"traffictype": "GUEST",
307+
"serviceProviderList" : {
308+
"Dhcp": "VirtualRouter",
309+
"Dns": "VirtualRouter",
310+
"UserData": "VirtualRouter",
311+
"SourceNat": "VirtualRouter",
312+
"PortForwarding": "VirtualRouter",
313+
"Vpn": "VirtualRouter",
314+
"Firewall": "VirtualRouter",
315+
"Lb": "VirtualRouter",
316+
"UserData": "VirtualRouter",
317+
"StaticNat": "VirtualRouter"
318+
}
319+
},
299320
"shared_network_offering_sg": {
300321
"name": "MySharedOffering-sg",
301322
"displaytext": "MySharedOffering-sg",

0 commit comments

Comments
 (0)