You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Ashadeepa <gi...@git.apache.org> on 2017/02/22 07:04:17 UTC

[GitHub] cloudstack pull request #1959: CLOUDSTACK-9786:API reference guide entry for...

GitHub user Ashadeepa opened a pull request:

    https://github.com/apache/cloudstack/pull/1959

    CLOUDSTACK-9786:API reference guide entry for associateIpAddress needs additional information

    Going through the code & implementation, it seems like either of the parameters are not required while accessing the API : associateIpAddress.
    There are 3 cases for which this api works. 1) networkId 2) vpcId 3) zoneId. Either of these can be provided to achieve the same functionality. If neither of them is provided, there is an error text shown.
    E.g.
    [root@CCP ~]# curl -s 'http://10.66.43.37:8096/client/api?command=associateIpAddress&listall=true' | xmllint --format - -o
    <?xml version="1.0" encoding="UTF-8"?>
    <associateipaddressresponse cloud-stack-version="4.5.1.0">
    <errorcode>431</errorcode>
    <cserrorcode>4350</cserrorcode>
    <errortext>Unable to figure out zone to assign ip to. Please specify either zoneId, or networkId, or vpcId in the call</errortext>
    </associateipaddressresponse>
    Modify the API reference guide entry with this detail in the "description"

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Accelerite/cloudstack CLOUDSTACK-9786

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1959.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1959
    
----
commit 030d34dca89621965afa2043a78a165a21adc26e
Author: Ashadeepa Debnath <as...@accelerite.com>
Date:   2017-02-21T11:29:02Z

    CLOUDSTACK-9786:API reference guide entry for associateIpAddress needs a fix

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1959: CLOUDSTACK-9786:API reference guide entry for associ...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1959
  
    LGTM, code review.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---