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

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

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

    https://github.com/apache/cloudstack/pull/1497#discussion_r62095734
  
    --- Diff: test/integration/smoke/test_list_ids_parameter.py ---
    @@ -0,0 +1,106 @@
    +# Licensed to the Apache Software Foundation (ASF) under one
    +# or more contributor license agreements.  See the NOTICE file
    +# distributed with this work for additional information
    +# regarding copyright ownership.  The ASF licenses this file
    +# to you under the Apache License, Version 2.0 (the
    +# "License"); you may not use this file except in compliance
    +# with the License.  You may obtain a copy of the License at
    +#
    +#   http://www.apache.org/licenses/LICENSE-2.0
    +#
    +# Unless required by applicable law or agreed to in writing,
    +# software distributed under the License is distributed on an
    +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    +# KIND, either express or implied.  See the License for the
    +# specific language governing permissions and limitations
    +# under the License.
    +""" Tests for API listing methods using 'ids' parameter
    +"""
    +#Import Local Modules
    +from marvin.cloudstackTestCase import cloudstackTestCase
    +from marvin.lib.utils import (cleanup_resources,
    +                              validateList)
    +from marvin.lib.base import (Account,
    +                             Volume,
    +                             DiskOffering)
    +from marvin.lib.common import (get_domain,
    +                                get_zone)
    +from marvin.codes import FAILED, PASS
    +from nose.plugins.attrib import attr
    +#Import System modules
    +import time
    +
    +_multiprocess_shared_ = True
    +class TestListIdsParams(cloudstackTestCase):
    +    
    +    @classmethod
    +    def setUpClass(cls):
    +        testClient = super(TestListIdsParams, cls).getClsTestClient()
    +        cls.apiclient = testClient.getApiClient()
    +        cls.services = testClient.getParsedTestDataConfig()
    +        cls.hypervisor = testClient.getHypervisorInfo()
    +
    +        cls.domain = get_domain(cls.apiclient)
    +        cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
    +
    +        cls.disk_offering = DiskOffering.create(
    +                                    cls.apiclient,
    +                                    cls.services["disk_offering"]
    +                                    )
    +
    +        cls.account = Account.create(
    +                            cls.apiclient,
    +                            cls.services["account"],
    +                            domainid=cls.domain.id
    +                            )
    +
    +        cls.volumes = []
    +        testVolumes = {0: {'diskname': 'TestDiskServ'}, 1: {'diskname': 
'TestDiskServ2'}, 2: {'diskname': 'TestDiskServ3'}}
    +
    +        for k, v in testVolumes.items():
    +            volume = Volume.create(
    +                                   cls.apiclient,
    +                                   v,
    +                                   zoneid=cls.zone.id,
    +                                   account=cls.account.name,
    +                                   domainid=cls.account.domainid,
    +                                   diskofferingid=cls.disk_offering.id
    +                                   )
    +            cls.debug("Created a volume with ID: %s" % volume.id)
    +            cls.volumes.append(volume)
    +
    +        cls._cleanup = [
    +                        cls.disk_offering,
    +                        cls.account
    +                        #,cls.volumes[0],
    --- End diff --
    
    I pushed this commented lines to ask about this, if I add them test fails 
but I don't understand how they are deleted. After test finishes I see volumes 
are deleted


> Add ids parameter to resource listing API calls
> -----------------------------------------------
>
>                 Key: CLOUDSTACK-9351
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9351
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: API
>    Affects Versions: 4.9.0
>            Reporter: Nicolas Vazquez
>             Fix For: 4.9.0
>
>
> h2. General behaviour
> A new parameter is added in each method, its type a list of IDs of the 
> entity, it will be mutually exclusive with id. (Similar to {{id}} and {{ids}} 
> parameters in {{listVirtualMachines}} method)
> h3. API Methods affected
> * +{{listTemplates}}+: new parameter *{{ids}}*, mutually exclusive with {{id}}
> * +{{listVolumes}}+: new parameter *{{ids}}*, mutually exclusive with {{id}}
> * +{{listSnapshots}}+: new parameter *{{ids}}*, mutually exclusive with {{id}}
> * +{{listVMSnapshots}}+: new parameter *{{vmsnapshotids}}*, mutually 
> exclusive with {{vmsnapshotid}}



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

Reply via email to