You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jclouds.apache.org by Hyeonu Jeong <no...@github.com> on 2013/12/12 07:31:02 UTC

[jclouds] Fix CloudStack EgressFirewall request parameter name (#228)

createEgressFirewall needs networkid instead of ipaddressid:
    http://download.cloud.com/releases/3.0.6/api_3.0.6/root_admin/createEgressFirewallRule.html
You can merge this Pull Request by running:

  git pull https://github.com/hyeonu/jclouds bugfix-cloudstack-egressfirewall

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/228

-- Commit Summary --

  * Fix CloudStack EgressFirewall request parameter name

-- File Changes --

    M apis/cloudstack/src/main/java/org/jclouds/cloudstack/features/FirewallApi.java (11)
    A apis/cloudstack/src/main/java/org/jclouds/cloudstack/options/ListEgressFirewallRulesOptions.java (179)
    M apis/cloudstack/src/test/java/org/jclouds/cloudstack/features/FirewallApiExpectTest.java (6)
    M apis/cloudstack/src/test/java/org/jclouds/cloudstack/features/FirewallApiLiveTest.java (4)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/228.patch
https://github.com/jclouds/jclouds/pull/228.diff

Re: [jclouds] Fix CloudStack EgressFirewall request parameter name (#228)

Posted by CloudBees pull request builder plugin <no...@github.com>.

[jclouds-java-7-pull-requests #928](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/928/) SUCCESS
This pull request looks good

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

Re: [jclouds] Fix CloudStack EgressFirewall request parameter name (#228)

Posted by CloudBees pull request builder plugin <no...@github.com>.

[jclouds-pull-requests #466](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/466/) UNSTABLE
Looks like there's a problem with this pull request

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

Re: [jclouds] Fix CloudStack EgressFirewall request parameter name (#228)

Posted by Andrew Phillips <no...@github.com>.

See comments at #229.

> jclouds-pull-requests #466 UNSTABLE

Looks like a [timeout](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/org.apache.jclouds.common$azure-common/466/testReport/junit/org.jclouds.azure.storage.filters/SharedKeyLiteAuthenticationTest/testIdempotent/) unrelated to this change.

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

Re: [jclouds] Fix CloudStack EgressFirewall request parameter name (#228)

Posted by BuildHive <no...@github.com>.

[jclouds » jclouds #701](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/701/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

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