Closed #198.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#event-366096053
Reopened #198.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#event-366096075
Closed #198.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#event-366052273
I have merged the jclouds prereq PR.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#issuecomment-125198739
Reopened #198.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#event-366052286
Closed #821.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#event-366029827
merged, backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#issuecomment-125191314
Closed #819.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/819#event-366014737
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/819#issuecomment-125185167
want to run it through some tests, but
looks good. If it works ok, we might have to refactor some other mock tests to
use it as well.
From: arvindn05
Sent: Thursday, July 23, 2015 3:51 AM
To: jclouds/jclouds
Cc: Zack Shoylev
Subject: Re: [jclouds] add os-h
> +import org.jclouds.http.HttpRequest;
> +import org.jclouds.http.HttpResponse;
> +import org.jclouds.openstack.nova.v2_0.NovaApi;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.Hypervisor;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.HypervisorDetails;
> +import or
> +import org.jclouds.http.HttpRequest;
> +import org.jclouds.http.HttpResponse;
> +import org.jclouds.openstack.nova.v2_0.NovaApi;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.Hypervisor;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.HypervisorDetails;
> +import or
@andrewgaul I might not be able to test until next week, but will give it a try.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/820#issuecomment-123568819
> +import org.jclouds.http.HttpRequest;
> +import org.jclouds.http.HttpResponse;
> +import org.jclouds.openstack.nova.v2_0.NovaApi;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.Hypervisor;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.HypervisorDetails;
> +import or
Looks good to me, but might not be able to merge it until Saturday.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#issuecomment-123564906
Reopened #821.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#event-361738700
Closed #821.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#event-361738691
Wow, that's a lot of work there! Thank you for contributing. I added some
notes. To see how these notes are implemented in other places, check our
support for openstack-heat here:
https://github.com/jclouds/jclouds-labs-openstack/tree/master/openstack-heat/src/main/java/org/jclouds/openstack/hea
> +
> +import org.jclouds.date.DateService;
> +import org.jclouds.date.internal.SimpleDateFormatDateService;
> +import org.jclouds.http.HttpResponse;
> +import org.jclouds.openstack.manila.v1.domain.Share;
> +import org.jclouds.openstack.manila.v1.options.CreateShareOptions;
> +import org.jclouds.r
> + @Nullable
> + ShareNetwork get(@PathParam("id") String shareNetworkId);
> +
> + /**
> +* Creates a new ShareNetwork
> +*
> +* @param shareNetworkId Id of the ShareNetwork
> +* @param options See CreateShareNetworkOptions
> +* @return The new ShareNetwork
> +*/
> +
> + */
> +@SkipEncoding({'/', '='})
> +@RequestFilters(AuthenticateRequest.class)
> +@Consumes(MediaType.APPLICATION_JSON)
> +@Path("/share-networks")
> +public interface ShareNetworkApi {
> + /**
> +* Returns a summary list of ShareNetworks.
> +*
> +* @return The list of ShareNetwork
> + */
> +@SkipEncoding({'/', '='})
> +@RequestFilters(AuthenticateRequest.class)
> +@Consumes(MediaType.APPLICATION_JSON)
> +@Path("/share-networks")
> +public interface ShareNetworkApi {
> + /**
> +* Returns a summary list of ShareNetworks.
> +*
> +* @return The list of ShareNetwork
> +import java.util.Date;
> +import java.util.Map;
> +
> +import javax.inject.Named;
> +
> +import org.jclouds.javax.annotation.Nullable;
> +
> +import com.google.common.base.CaseFormat;
> +import com.google.common.base.Objects;
> +import com.google.common.base.Objects.ToStringHelper;
> +import com
Note: Failing because it depends on the corresponding jclouds change for
service type.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/198#issuecomment-123489121
Closed #821.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#event-361664233
Reopened #821.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/821#event-361664261
> +import org.jclouds.http.HttpRequest;
> +import org.jclouds.http.HttpResponse;
> +import org.jclouds.openstack.nova.v2_0.NovaApi;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.Hypervisor;
> +import org.jclouds.openstack.nova.v2_0.domain.regionscoped.HypervisorDetails;
> +import or
> +package org.jclouds.openstack.nova.v2_0.domain.regionscoped;
> +
> +import com.google.auto.value.AutoValue;
> +import org.jclouds.json.SerializedNames;
> +
> +/**
> + * Hypervisor which shows id, name and other details
> + */
> +@AutoValue
> +public abstract class HypervisorDetails {
> +
> + p
Zack Shoylev created JCLOUDS-965:
Summary: Ensure jclouds retries properly when writing the request
is interrupted
Key: JCLOUDS-965
URL: https://issues.apache.org/jira/browse/JCLOUDS-965
Project
Closed #815.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#event-357808807
merged, backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#issuecomment-122041635
[
https://issues.apache.org/jira/browse/JCLOUDS-963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629839#comment-14629839
]
Zack Shoylev commented on JCLOUDS-963:
--
I suspect fairly normal network prob
Going to try rebuilding...
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#issuecomment-121796608
Closed #815.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#event-357070225
Reopened #815.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#event-357070227
Can we add some of your first comment in this PR as in-code docs for the domain
classes?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/815#issuecomment-121768474
> @@ -34,7 +36,7 @@
> @Test(groups = "unit", testName = "AvailabilityZoneApiExpectTest")
> public class AvailabilityZoneApiExpectTest extends BaseNovaApiExpectTest {
>
> - public void testAvailabilityZonesList() throws Exception {
> + public void testListAvailabilityZones() throws Exception
> + * See the License for the specific language governing permissions and
> + * limitations under the License.
> + */
> +package org.jclouds.openstack.nova.v2_0.domain.regionscoped;
> +
> +import com.google.common.base.Objects;
> +import com.google.common.collect.ImmutableMap;
> +import com.google.
> + * See the License for the specific language governing permissions and
> + * limitations under the License.
> + */
> +package org.jclouds.openstack.nova.v2_0.domain.regionscoped;
> +
> +import com.google.common.base.Objects;
> +import com.google.common.collect.ImmutableMap;
> +import com.google.
Closed #813.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/813#event-355952792
merged, backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/813#issuecomment-121434718
Also note
https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/floating_ip_pools.py
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/813#issuecomment-121431532
[
https://issues.apache.org/jira/browse/JCLOUDS-700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev resolved JCLOUDS-700.
--
Resolution: Fixed
Fix Version/s: (was: 2.0.0)
1.9.1
> OpenSt
Closed #806.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/806#event-355939619
merged, backported. Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/806#issuecomment-121431327
Fixes broken namespace for the ip pool extension
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/813
-- Commit Summary --
* Adds missng floating-ip-pool expect test
-- File Changes --
M
apis/openstack-nova/src/main/java/org/jclou
[
https://issues.apache.org/jira/browse/JCLOUDS-700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14627297#comment-14627297
]
Zack Shoylev commented on JCLOUDS-700:
--
https://github.com/jclouds/jclouds/pull
[
https://issues.apache.org/jira/browse/JCLOUDS-333?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev resolved JCLOUDS-333.
--
Resolution: Fixed
> Glance failed to parse Image details which has microseconds in Date for
Hm, no, that seems to have messed it up again.
What I do: In my PR branch, I rebase the last 2 (or more) commits:
git rebase -i HEAD~2
Then I force push to the github branch I used for the pull request:
git push -f
---
Reply to this email directly or view it on GitHub:
https://github.com/jclou
Closed #809.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/809#event-355476344
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/809#issuecomment-121281052
Thanks for explaining! In this case, this should be good to merge - rebase to a
single commit.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/806#issuecomment-121258189
This will have to be backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/809#issuecomment-121143386
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/809
-- Commit Summary --
* Updates live tests and template builder settings for compute
-- File Changes --
M
compute/src/main/java/org/jclouds/compute/reference/ComputeServiceConst
[
https://issues.apache.org/jira/browse/JCLOUDS-907?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev updated JCLOUDS-907:
-
Fix Version/s: (was: 1.9.1)
2.0.0
> Extra space in endpoint cau
[
https://issues.apache.org/jira/browse/JCLOUDS-907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14625659#comment-14625659
]
Zack Shoylev commented on JCLOUDS-907:
--
It won't make it for 1.9.1; mostl
Ok. Thanks, ran it and looks good, but I have some questions:
It seems you need to deprecate, because you are using a new AvailabilityZone
class. Why not just update the old availability zone so you don't have to
deprecate it? It seems the new one is just adding stuff, so it should be
backwards
> @@ -34,11 +33,21 @@ public void testListAvailabilityZones() throws Exception {
>
>Optional availabilityZoneApi =
> api.getAvailabilityZoneApi("RegionOne");
>if (availabilityZoneApi.isPresent()) {
> - FluentIterable zones =
> availabilityZoneApi.get().list();
> +
> @@ -34,11 +33,21 @@ public void testListAvailabilityZones() throws Exception {
>
>Optional availabilityZoneApi =
> api.getAvailabilityZoneApi("RegionOne");
>if (availabilityZoneApi.isPresent()) {
> - FluentIterable zones =
> availabilityZoneApi.get().list();
> +
Closed #806.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/806#event-353012009
Thank you for contributing!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/197#issuecomment-120174005
Closed #804.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/804#event-352428392
merged but not backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/804#issuecomment-120173734
Rebuild, please.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/804#issuecomment-120168172
> @@ -141,20 +142,17 @@ public void testCopyObject() throws Exception {
> // test exception thrown on bad source name
> try {
> destApi.copy(destinationObject, badSource, sourceObjectName);
> -fail("Expected CopyObjectException");
Ah, right. Thanks, I w
Closed #197.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/197#event-352274058
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/197#issuecomment-120128167
Thanks, I will merge and backport in a bit.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/195#issuecomment-120109048
@andrewgaul ^
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/804#issuecomment-120083348
[
https://issues.apache.org/jira/browse/JCLOUDS-957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14620869#comment-14620869
]
Zack Shoylev commented on JCLOUDS-957:
--
https://github.com/jclouds/jclouds/pull
https://issues.apache.org/jira/browse/JCLOUDS-957
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/804#issuecomment-120075502
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/804
-- Commit Summary --
* Removes CopyObjectException in favor of jclouds-standard blobstore
exceptions
-- File Changes --
D
apis/openstack-swift/src/main/java/org/jclouds/openst
Hmm commits seem messed up. If you want to, you can make a new one. Just make
sure to reference it here. Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/195#issuecomment-119784737
Closed #803.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/803#event-351392236
merged
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/803#issuecomment-119776774
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/803
-- Commit Summary --
* Fixes a small test bug on windows
-- File Changes --
M
apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java
(2)
-- Patch L
Rebase to a single commit
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/195#issuecomment-119757953
[
https://issues.apache.org/jira/browse/JCLOUDS-680?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev resolved JCLOUDS-680.
--
Resolution: Fixed
> OpenStack Swift: Improve Object update behav
[
https://issues.apache.org/jira/browse/JCLOUDS-680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14619580#comment-14619580
]
Zack Shoylev commented on JCLOUDS-680:
--
I think it's better now.
> Ope
[
https://issues.apache.org/jira/browse/JCLOUDS-957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14618823#comment-14618823
]
Zack Shoylev commented on JCLOUDS-957:
--
I'm looking
> openstack-sw
squashed, merged, backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/196#issuecomment-118152081
Closed #196.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/196#event-346826522
> + */
> + public abstract Builder adminStateUp(Boolean adminStateUp);
> +
> + /**
> + *
> + * @param firewallPolicyId The firewall policy uuid that this firewall
> is associated with. This firewall implements the rules contained in the
> + *
That is strange. Sure, but maybe have a look and see it's not something else
that's going wrong.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/196#issuecomment-118012210
Also I think the getSomeName() getters were fine. Auto works fine with
getName()
and name(String name)/getName() in the builder, for example.
https://github.com/jclouds/jclouds-labs-openstack/blob/master/openstack-heat/src/main/java/org/jclouds/openstack/heat/v1/options/CreateStack.java
---
Reply
Another thing I overlooked - openstack-heat has the good up-to-date examples
that I recently refactored.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/196#issuecomment-118010952
Zack Shoylev created JCLOUDS-954:
Summary: Upgrade trove to current openstack features
Key: JCLOUDS-954
URL: https://issues.apache.org/jira/browse/JCLOUDS-954
Project: jclouds
Issue Type
> +public abstract class CreateFirewallPolicy {
> +
> + @Nullable public abstract String getTenantId();
> + @Nullable public abstract String getName();
> + @Nullable public abstract String getDescription();
> + public abstract boolean isShared();
> + @Nullable public abstract List getFire
> + @Nullable public abstract String getStatus();
> + @Nullable public abstract String getFirewallPolicyId();
> +
> + @SerializedNames({"id", "tenant_id", "name", "description",
> "admin_state_up", "status", "firewall_policy_id"})
> + public static Firewall create(String id, String tenantI
> +public abstract class CreateFirewallPolicy {
> +
> + @Nullable public abstract String getTenantId();
> + @Nullable public abstract String getName();
> + @Nullable public abstract String getDescription();
> + public abstract boolean isShared();
> + @Nullable public abstract List getFire
> +
> +/**
> + * A firewall policy
> + *
> + * @see + *
> href="http://docs.openstack.org/admin-guide-cloud/content/fwaas_api_abstractions.html";>api
> doc
> + */
> +@AutoValue
> +public abstract class FirewallPolicy {
> +
> + @Nullable
> + public abstract String getId();
> + @Nullabl
> + @SelectJson("firewall")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + Firewall get(@PathParam("id") String id);
> +
> + /**
> +* Create a new firewall
> +*
> +* @param firewall Options for creating a firewall
> +* @return the newly created firewall
> +
Thanks @andreaturli ; looking pretty good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/196#issuecomment-117638687
> +
> + @Named("firewall:insertFirewallRuleToPolicy")
> + @PUT
> + @Path("/firewall_policies/{id}/insert_rule")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + FirewallPolicy insertFirewallRuleToPolicy(@PathParam("id") String
> policyId, @WrapWith("firewall_rule_id") String
> +import org.jclouds.javax.annotation.Nullable;
> +import org.jclouds.json.SerializedNames;
> +
> +import com.google.auto.value.AutoValue;
> +import com.google.common.collect.ImmutableList;
> +
> +/**
> + * A firewall policy
> + *
> + * @see + *
> href="http://docs.openstack.org/admin-guide
> + @SelectJson("firewall")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + Firewall get(@PathParam("id") String id);
> +
> + /**
> +* Create a new firewall
> +*
> +* @param firewall Options for creating a firewall
> +* @return the newly created firewall
> +
> +
> +/**
> + * A firewall policy
> + *
> + * @see + *
> href="http://docs.openstack.org/admin-guide-cloud/content/fwaas_api_abstractions.html";>api
> doc
> + */
> +@AutoValue
> +public abstract class FirewallPolicy {
> +
> + @Nullable
> + public abstract String getId();
> + @Nullabl
> + * @see + *
> href="http://docs.openstack.org/admin-guide-cloud/content/fwaas_api_abstractions.html";>api
> doc
> + */
> +@AutoValue
> +public abstract class FirewallPolicy {
> +
> + @Nullable
> + public abstract String getId();
> + @Nullable public abstract String getTenantId();
>
> +
> +/**
> + * A Firewall
> + *
> + * @see + *
> href="http://docs.openstack.org/admin-guide-cloud/content/fwaas_api_abstractions.html";>api
> + * doc
> + */
> +@AutoValue
> +public abstract class Firewall {
> +
> + @Nullable public abstract String getId();
> + @Nullable public ab
> @@ -47,6 +48,10 @@ public void testCreateUpdateAndDeleteSecurityGroup() {
> try {
> sgApi = api.getSecurityGroupApi(region).get();
>
> +for (Subnet subnet :
> api.getSubnetApi(region).list().concat().toList()) {
> + System.out.println("subnet: "
501 - 600 of 1837 matches
Mail list logo
