You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by jasminSPC <no...@github.com> on 2015/02/26 13:34:51 UTC

[jclouds-labs] Profitbricks ipblocks api (#140)

Here is Profitbricks IpBlocks feature.
[Live tests](https://gist.github.com/jasminSPC/ae25152f013337119e45)
NOTE: Since Nic feature is still pending merge. In IpBlock I've used hard coded value for nicId (See [link](https://github.com/StackPointCloud/jclouds-labs/blob/profitbricks-ipblocks-api/profitbricks/src/test/java/org/jclouds/profitbricks/features/IpBlockApiLiveTest.java#L31)).
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs/pull/140

-- Commit Summary --

  * Added IpBlock and PubliIp classes
  * EOL
  * Added parsers for PublicIps
  * IpBlockResponseHandlerTest
  * IpBlockListResponseHandlerTest
  * Minor tweaks
  * testGetAllIpBlock
  * Style fix
  * Fixed Response handlers
  * Framework for ReservePublicIpBlock
  * IPBlock mock tests done
  * Live tests passed
  * EOL fix
  * EOL fix
  * Null check for ips list

-- File Changes --

    M profitbricks/src/main/java/org/jclouds/profitbricks/ProfitBricksApi.java (4)
    A profitbricks/src/main/java/org/jclouds/profitbricks/domain/IpBlock.java (125)
    A profitbricks/src/main/java/org/jclouds/profitbricks/features/IpBlockApi.java (81)
    M profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/BaseProfitBricksResponseHandler.java (4)
    A profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/ipblock/BaseIpBlockResponseHandler.java (70)
    A profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/ipblock/IpBlockListResponseHandler.java (62)
    A profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/ipblock/IpBlockResponseHandler.java (64)
    A profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/publicip/BasePublicIpResponseHandler.java (40)
    A profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/publicip/PublicIpListResponseHandler.java (51)
    A profitbricks/src/test/java/org/jclouds/profitbricks/features/IpBlockApiLiveTest.java (79)
    A profitbricks/src/test/java/org/jclouds/profitbricks/features/IpBlockApiMockTest.java (160)
    A profitbricks/src/test/java/org/jclouds/profitbricks/http/parser/ipblock/IpBlockListResponseHandlerTest.java (77)
    A profitbricks/src/test/java/org/jclouds/profitbricks/http/parser/ipblock/IpBlockResponseHandlerTest.java (60)
    A profitbricks/src/test/resources/ipblock/ipblock-addtonic.xml (12)
    A profitbricks/src/test/resources/ipblock/ipblock-release.xml (10)
    A profitbricks/src/test/resources/ipblock/ipblock-removefromnic.xml (12)
    A profitbricks/src/test/resources/ipblock/ipblock-reserve.xml (13)
    A profitbricks/src/test/resources/ipblock/ipblock.xml (18)
    A profitbricks/src/test/resources/ipblock/ipblocks.xml (30)

-- Patch Links --

https://github.com/jclouds/jclouds-labs/pull/140.patch
https://github.com/jclouds/jclouds-labs/pull/140.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/140

Re: [jclouds-labs] Profitbricks ipblocks api (#140)

Posted by Ignasi Barrera <no...@github.com>.
>NOTE: Since Nic feature is still pending merge. In IpBlock I've used hard coded value for nicId (See link).

That PR is now merged. Can you rebase on to of it and apply the required changes to complete the PR?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/140#issuecomment-76192394

Re: [jclouds-labs] Profitbricks ipblocks api (#140)

Posted by jasminSPC <no...@github.com>.
I messed this one up. I'm closing it

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/140#issuecomment-76275921