-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15715/
-----------------------------------------------------------

Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-5215
    https://issues.apache.org/jira/browse/CLOUDSTACK-5215


Repository: cloudstack-git


Description
-------

The function getFreeVlan was used in 3 files separately and it was not checking 
whether the vlan id is already in use by any other network.

Now checking if it is used by any network, else search for another vlan id, 
also moved this function to common.py and made respective changes across 3 
files.


Diffs
-----

  test/integration/component/test_shared_networks.py 28468c9 
  test/integration/component/test_vpc_vm_life_cycle.py beb5b99 
  test/integration/component/test_vpc_vms_deployment.py c4811ed 
  tools/marvin/marvin/integration/lib/common.py 6c2bb2a 

Diff: https://reviews.apache.org/r/15715/diff/


Testing
-------

Tested is the function works correctly. Ran 2 test cases simultaneously which 
create multiple shared networks.

Log:

Test 1:
==> result.log <==
test_createSharedNetwork_projectSpecific 
(test_shared_networks.TestSharedNetworks)
Test Shared Network with scope project ... skipped 'skip'
test_createSharedNetwork_usedVlan (test_shared_networks.TestSharedNetworks)
Test Shared Network with used vlan 01 ... skipped 'skipped - This is a 
redundant case and also this is causing issue for rest fo the cases '
test_createSharedNetwork_usedVlan2 (test_shared_networks.TestSharedNetworks)
Test Shared Network with used vlan 02 ... skipped 'skip'
test_deployVM_isolatedAndShared (test_shared_networks.TestSharedNetworks)
Test VM deployment in shared and isolated networks ... skipped 'skip'
test_deployVM_multipleSharedNetwork (test_shared_networks.TestSharedNetworks)
Test Vm deployment with multiple shared networks ...
==> client.log <==
2013-11-19 21:40:40,957 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Admin account created: 
7d379404-9bf3-47aa-a4bc
-0fe77e7455eb
2013-11-19 21:40:41,004 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - 2360
2013-11-19 21:40:41,004 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Physical Network found: 
05ef4a90-f9ba-449f-b1b
6-a437e6c4d4dd
2013-11-19 21:40:41,071 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Shared Network offering created: 
d6b52bcc-f7d4
-401a-83d2-6c049854ab76
2013-11-19 21:40:41,266 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Shared Network created: 
bc992430-00f8-42eb-9e2
2-fdfcf3b2e94a
2013-11-19 21:40:41,321 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - 2368
2013-11-19 21:40:41,420 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Network created: 
e661393d-a1bb-4619-adcc-4d7d6
5172e29
2013-11-19 21:42:01,929 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Virtual Machine created: 
44d30c61-07cb-4986-ab
16-96b9506c6061
2013-11-19 21:43:47,519 - DEBUG - test_deployVM_multipleSharedNetwork 
(test_shared_networks.TestSharedNetworks) - Virtual Machine created: 
2d0f4114-a763-43be-90
70-73e3bb74571f

==> result.log <==
ok
test_networkWithsubdomainaccessFalse (test_shared_networks.TestSharedNetworks)
Test shared Network with subdomainaccess=False ... skipped 'skip'
test_networkWithsubdomainaccessTrue (test_shared_networks.TestSharedNetworks)
Test Shared Network with subdomainaccess=True ... skipped 'skip'
test_sharedNetworkOffering_01 (test_shared_networks.TestSharedNetworks)
Test shared network Offering 01 ... skipped 'skip'
test_sharedNetworkOffering_02 (test_shared_networks.TestSharedNetworks)
Test Shared Network Offering 02 ... skipped 'skip'
test_sharedNetworkOffering_03 (test_shared_networks.TestSharedNetworks)
Test Shared Network Offering 03 ... skipped 'skip'

----------------------------------------------------------------------
Ran 13 tests in 472.706s

OK (skipped=12)


Test 2:

 ==> result.log <==
test_createSharedNetwork_All (test_shared_networks.TestSharedNetworks)
Test Shared Network ALL ...
==> client.log <==
2013-11-19 21:34:56,669 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Admin type account created: 
admin-XABU1-JAM9J0
2013-11-19 21:34:56,744 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - User type account created: 
admin-XABU1-YIDA7H
2013-11-19 21:34:56,782 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Physical network found: 
05ef4a90-f9ba-449f-b1b6-a437e
6c4d4dd
2013-11-19 21:34:57,023 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Shared Network offering created: 
575d5258-21eb-4f55-9
697-4fbdad00f0dd
2013-11-19 21:34:57,221 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Shared Network created for scope 
domain: a9874639-d40
c-4b8b-8129-6496bdf4aa07
2013-11-19 21:36:27,711 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Virtual Machine created: 
7181bcb1-f114-4013-8d93-3696
edcf7653
2013-11-19 21:36:32,971 - DEBUG - test_createSharedNetwork_All 
(test_shared_networks.TestSharedNetworks) - Virtual Machine created: 
4562afe4-b2a5-473e-9c9f-4c2a
31be6cef

==> result.log <==
ok
test_createSharedNetwork_accountSpecific 
(test_shared_networks.TestSharedNetworks)
Test Shared Network with scope account ... skipped 'skip'
test_createSharedNetwork_domainSpecific 
(test_shared_networks.TestSharedNetworks)
Test Shared Network with scope domain ... skipped 'skip'
test_createSharedNetwork_projectSpecific 
(test_shared_networks.TestSharedNetworks)
Test Shared Network with scope project ... skipped 'skip'
test_createSharedNetwork_usedVlan (test_shared_networks.TestSharedNetworks)
Test Shared Network with used vlan 01 ... skipped 'skipped - This is a 
redundant case and also this is causing issue for rest fo the cases '
test_createSharedNetwork_usedVlan2 (test_shared_networks.TestSharedNetworks)
Test Shared Network with used vlan 02 ... skipped 'skip'
test_deployVM_isolatedAndShared (test_shared_networks.TestSharedNetworks)
Test VM deployment in shared and isolated networks ... skipped 'skip'
test_deployVM_multipleSharedNetwork (test_shared_networks.TestSharedNetworks)
Test Vm deployment with multiple shared networks ... skipped 'skip'
test_networkWithsubdomainaccessFalse (test_shared_networks.TestSharedNetworks)
Test shared Network with subdomainaccess=False ... skipped 'skip'
test_networkWithsubdomainaccessTrue (test_shared_networks.TestSharedNetworks)
Test Shared Network with subdomainaccess=True ... skipped 'skip'
test_sharedNetworkOffering_01 (test_shared_networks.TestSharedNetworks)
Test shared network Offering 01 ... skipped 'skip'
test_sharedNetworkOffering_02 (test_shared_networks.TestSharedNetworks)
Test Shared Network Offering 02 ... skipped 'skip'
test_sharedNetworkOffering_03 (test_shared_networks.TestSharedNetworks)
Test Shared Network Offering 03 ... skipped 'skip'

----------------------------------------------------------------------
Ran 13 tests in 362.481s

OK (skipped=12)


Thanks,

Gaurav Aradhye

Reply via email to