[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15214084#comment-15214084
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9328:
--------------------------------------------

Github user shwetaag commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1455#discussion_r57562054
  
    --- Diff: test/integration/smoke/test_privategw_acl.py ---
    @@ -287,6 +298,16 @@ def performVPCTests(self, vpc_off, 
restart_with_cleanup = False):
             if not physical_networks:
                 self.fail("No Physical Networks found!")
     
    +        qresultset = self.dbclient.execute(
    +            "select vnet from op_dc_vnet_alloc where physical_network_id=\
    +            (select id from physical_network where uuid='%s' ) and taken 
is NULL;" % physical_networks[0].id
    +        )
    +        self.assertEqual(validateList(qresultset)[0],
    +                         PASS,
    +                         "Invalid sql query response"
    +        )
    +        #vlans = physical_networks[0].vlan.split('-')
    +        vlans = qresultset
             vlans = physical_networks[0].vlan.split('-')
    --- End diff --
    
    this split is not needed for getting correct vlan entry


> Fix vlan issues from test suite test_privategw_acl.py in BVT
> ------------------------------------------------------------
>
>                 Key: CLOUDSTACK-9328
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9328
>             Project: CloudStack
>          Issue Type: Test
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Automation
>            Reporter: Sanjeev N
>            Assignee: Sanjeev N
>
> Fix vlan issues from test suite test_privategw_acl.py in BVT
> Currently the tests reads the vlans from the physical network and takes the 
> first vlan from the list of vlans and uses it. However, this approach might 
> not work when the tests run in parallel. E.g. in CI we run test suits in 
> parallel and the first vlan may not be available and tests would fail if we 
> try to use the vlan which is already in use.
> Made changes to the script so that it will get the free vlan list from the DB 
> and uses the first vlan from the available pool.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to