[jira] [Commented] (JCLOUDS-660) Improve portable container ACL support

2015-02-16 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-660?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14323738#comment-14323738 ] Andrew Gaul commented on JCLOUDS-660: - Surveying the major object stores, only Azure

Jenkins build became unstable: jclouds-karaf #1019

2015-02-16 Thread jenkins-no-reply
See

Build failed in Jenkins: jclouds-guava-17-18 » 17.0,OpenJDK 7 (latest) #136

2015-02-16 Thread jenkins-no-reply
See Changes: [Ignasi Barrera] Added support for RunInstances parameter PrivateIpAddress on EC2 -- [...truncated 3733 lines...] Test suite

Re: [jclouds] Added support for RunInstances parameter PrivateIpAddress on EC2 (#670)

2015-02-16 Thread Ignasi Barrera
Pushed to [master](https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=5b4ac5f1) and [1.8.x](https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=c47816d1). Thanks @chemikadze and apologies for the delay! --- Reply to this email directly or view it on GitHub: https

Re: [jclouds] Added support for RunInstances parameter PrivateIpAddress on EC2 (#670)

2015-02-16 Thread Ignasi Barrera
Closed #670. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/670#event-236589868

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
A couple more things to consider that have come to my mind after reading the ComputeService implementation again: * The `templateOptions.inboundPorts` should be considered to open those ports in the target virtual machine. Is this something that can be done per VM, or the "external" ports must

Jenkins build is back to normal : jclouds-guava-17-18 » 17.0,OpenJDK 7 (latest) #135

2015-02-16 Thread jenkins-no-reply
See

[jira] [Closed] (JCLOUDS-818) Contribution - Azure provider in the JClouds Labs

2015-02-16 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera closed JCLOUDS-818. -- Resolution: Fixed > Contribution - Azure provider in the JClouds Labs > ---

Re: [jclouds] Nova Attach Interface Extension API (#679)

2015-02-16 Thread Ignasi Barrera
I'm a bit late to the party :) Just a comment about the missing tests. Also there should be a live test for each new API method, right? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/679#issuecomment-74572327

Re: [jclouds] Nova Attach Interface Extension API (#679)

2015-02-16 Thread Ignasi Barrera
> +.method("GET") > +.endpoint( > + > "https://az-1.region-a.geo-1.compute.hpcloudsvc.com/v2/3456/servers/instance-1/os-interface/ce531f90-199f-48c0-816c-13e38010b442";) > +.addHeader("Accept", > "application/json").addHeader("X-Auth-Token", au

Build failed in Jenkins: jclouds-guava-17-18 » 17.0,OpenJDK 7 (latest) #134

2015-02-16 Thread jenkins-no-reply
See Changes: [Zack Shoylev] Refactors consitency pause in blob tests and more fixes -- [...truncated 11425 lines...] Starting test testFi

Jenkins build is back to normal : jclouds-labs-google #496

2015-02-16 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-labs-google » Apache jclouds labs google #496

2015-02-16 Thread jenkins-no-reply
See

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Eduard Koller
Thank you for the thorough review, Ignasi – much appreciated. We’ll address the issues you flagged and provide an updated PR soon. Thanks, Eduard From: Ignasi Barrera [mailto:notificati...@github.com] Sent: Monday, February 16, 2015 5:28 AM To: jclouds/jclouds-labs Cc: Eduard Koller Subj

Re: [jclouds] Refactors consitency pause in blob tests and more fixes (#686)

2015-02-16 Thread Zack Shoylev
Closed #686. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/686#event-236510645

[jclouds] Refactors consitency pause in blob tests and more fixes (#686)

2015-02-16 Thread Zack Shoylev
You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/686 -- Commit Summary -- * Refactors consitency pause in blob tests and more fixes -- File Changes -- M blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseConta

Build failed in Jenkins: jclouds-guava-17-18 » 17.0,OpenJDK 7 (latest) #133

2015-02-16 Thread jenkins-no-reply
See Changes: [Zack Shoylev] Adds a few more pauses for eventual consistency -- [...truncated 29370 lines...] Starting test testFileGetPar

Build failed in Jenkins: jclouds-labs-google » Apache jclouds labs google #495

2015-02-16 Thread jenkins-no-reply
See -- [...truncated 2594 lines...] [WARNING]

Build failed in Jenkins: jclouds-labs-google #495

2015-02-16 Thread jenkins-no-reply
See -- [...truncated 2725 lines...] Downloaded: http://repo.cloudbees.com/content/repositories/central/log4j/log4j/1.2.17/log4j-1.2.17.pom (22 KB at 1179.7 KB/sec) Downloading: https://reposi

Re: [jclouds] Adds a few more pauses for eventual consistency (#685)

2015-02-16 Thread Zack Shoylev
Closed #685. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/685#event-236467985

Re: [jclouds] Adds a few more pauses for eventual consistency (#685)

2015-02-16 Thread Zack Shoylev
merged --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/685#issuecomment-74547924

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + * 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

Jenkins build is back to normal : jclouds-labs-aws #414

2015-02-16 Thread jenkins-no-reply
See

[jclouds] Adds a few more pauses for eventual consistency (#685)

2015-02-16 Thread Zack Shoylev
You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/685 -- Commit Summary -- * Adds a few more pauses for eventual consistency -- File Changes -- M blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseContainerInte

Jenkins build is back to normal : jclouds-labs-aws » Apache jclouds labs aws #414

2015-02-16 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-labs-aws #414

2015-02-16 Thread jenkins-no-reply
See

[jira] [Resolved] (JCLOUDS-822) C4 instances missing in ec2 compute

2015-02-16 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-822?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera resolved JCLOUDS-822. Resolution: Fixed Fix Version/s: 1.9.0 2.0.0 C4 type is already p

[jira] [Comment Edited] (JCLOUDS-822) C4 instances missing in ec2 compute

2015-02-16 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14323010#comment-14323010 ] Ignasi Barrera edited comment on JCLOUDS-822 at 2/16/15 5:04 PM: --

[jira] [Created] (JCLOUDS-822) C4 instances missing in ec2 compute

2015-02-16 Thread Tomas Fecko (JIRA)
Tomas Fecko created JCLOUDS-822: --- Summary: C4 instances missing in ec2 compute Key: JCLOUDS-822 URL: https://issues.apache.org/jira/browse/JCLOUDS-822 Project: jclouds Issue Type: Bug Affec

Re: [jclouds-labs] JCLOUDS-702: JClouds ProfitBricks provider - Snapshots API (#134)

2015-02-16 Thread Matt Baldwin
@nacx Hi, we've been working on this with RJ. We have the PR passing tests so we're just waiting on a review to merge. Let me know if there are any issues or changes that need to happen. Thanks. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
Thankd @eduardkoller! I've done an initial review focusing on structural things that must be addressed before this can be merged. Before we can merge it, all the comments must be addressed. I've added some general ones that I've mentioned only once to avoid repeating them: * All license header

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> @@ -0,0 +1 @@ > +org.jclouds.azurecompute.AzureComputeProviderMetadata We use Google Auto for service loader discovery. Remove this file and annotate the `AzureComputeProviderMetadata` accordingly. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/p

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + * OS images from the image repository > + * > + * @see http://msdn.microsoft.com/en-us/library/jj157191"; >api > + */ > +public class OSImage { > +public static String LOCATION = "location"; > + > +public static Builder builder() { > +return new ConcreteBuilder(); > +} > + >

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > + bind(new TypeLiteral>() { > + }).to(ProxyForURI.class); > + bind(new TypeLiteral, Map String>>>() { > + }).to(FilterStringsBoundToInjectorByName.class); > +// install(new GoGridParserModule()); > +// super.configure(); > + bind(new TypeLiteral>() { >

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + * limitations under the License. > + */ > +package org.jclouds.azurecompute.domain; > +import com.google.common.base.Objects; > + > +import java.net.URI; > +import java.util.Calendar; > + > +import static com.google.common.base.Preconditions.checkNotNull; > + > +/** > + * OS images from the ima

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +import org.jclouds.json.config.GsonModule; > +import org.jclouds.location.config.LocationModule; > +import org.jclouds.location.suppliers.ImplicitLocationSupplier; > +import org.jclouds.proxy.ProxyForURI; > +import org.jclouds.rest.ApiContext; > +import org.jclouds.rest.ConfiguresRestClient; > +

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +private ComputeServiceAdapter RoleSizeListResponse.RoleSize, OSImage, LocationsListResponse.Location> > adapter; > +private Function transformer; > + > +@Inject > +public LocationSupplier(ComputeServiceAdapter RoleSizeListResponse.RoleSize, OSImage, > +LocationsListR

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + *//*.volumes(ImmutableList. of(new VolumeImpl(25.0f, true, > true)))*//*.build(); > +public static final Hardware A6 = new > HardwareBuilder().ids("A6").ram(28672) > +.processors(ImmutableList.of(new Processor(4, 1.0))) > + *//*.volumes(ImmutableList. of(new Vo

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +String service = > groupNamingConvention.createWithoutPrefix().extractGroup(id); > +String name = service == null ? id : id.substring(service.length() + > 1); > +String deploymentName = > client.getVirtualMachineApi().getDeploymentBySlot(service, > DeploymentSlot.Prod

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > client.getVirtualMachineApi().getDeploymentBySlot(serviceName, > deploymentSlot); > +if (deploymentResponse != null) { > + > virtualMachines.add(deploymentResponseToVirtualMachine.apply(deploymentResponse)); > +} > +} catch

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +HostedServiceGetDetailedResponse serviceResponse = > client.getHostedServiceApi().getHostedServiceWithDeployments(hostedService.getServiceName()); > +if (!serviceResponse.getDeployments().isEmpty()) { > +HostedServiceGetDetailedResponse.Deployment deploym

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +//client.getServerApi().addDeployment(group, > deploymentToParameters.apply(deployment)); > +client.getVirtualMachineApi().addDeployment(group, parameters); > +} else { > +VirtualMachineCreateParameters parameters = new > VirtualMachineCreateParameters()

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +this.groupNamingConvention = namingConvention; > +// this.serverLatestJobCompleted = retry(new > ServerLatestJobCompleted(client.getJobServices()), > +//timeouts.nodeRunning * 9l / 10l); > +// this.serverLatestJobCompletedShort = retry(new > ServerLatestJobComplet

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +parameters.setServiceName(group); > +parameters.setLocation(template.getLocation().getId()); > +client.getHostedServiceApi().createHostedService(parameters); > +} > + > +//required > +ArrayList configlist = new > ArrayList(); > +

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +//timeouts.nodeRunning * 9l / 10l); > +// this.serverLatestJobCompletedShort = retry(new > ServerLatestJobCompleted(client.getJobServices()), > +//timeouts.nodeRunning * 1l / 10l); > +} > + > + > +@Override > +public NodeAndInitialCredentials > createNo

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + if (from.getRole().getRoleSize() != null) { > + > builder.hardware(from(hardwares.get()).firstMatch(HardwarePredicates.idEquals(from.getRole().getRoleSize())).orNull()); > + } > + if (from.getRole().getOSVirtualHardDisk() != null) { > + String imageId = > from.g

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > +return new OperatingSystem(osFamily, osName, osVersion, osArch, > osDescription, is64Bit); > +} > + > +@Override > +public Image apply(OSImage from) { > +ImageBuilder builder = new ImageBuilder(); > +builder.ids(from.getName()); > +builder.name(fr

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +import org.jclouds.azurecompute.AzureComputeApi; > +import org.jclouds.logging.Logger; > + > +import com.google.common.base.Function; > +import com.google.common.collect.FluentIterable; > + > +/** > + * defines the connection between the {@link > org.jclouds.azurecompute.AzureComputeApi} implem

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +this.deploymentResponseToVirtualMachine = > checkNotNull(deploymentResponseToVirtualMachine, > "deploymentResponseToVirtualMachine"); > +this.serviceResponseToVirtualMachine = > checkNotNull(serviceResponseToVirtualMachine, > "serviceDeploymentResponseToDeployment"); > +

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +super(locations, images, sizes, defaultLocation, optionsProvider, > defaultTemplateProvider, getImageStrategy); > +imageChooser = imageChooser(IMAGE_ORDERING); > +} > + > +Function, Image> imageChooser(final > Ordering ordering) { > +return new Function, Image>(

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + VirtualMachineToNodeMetadata(@Memoized Supplier> > images, @Memoized Supplier> hardwares, > +Supplier> locations, > +GroupNamingConvention.Factory > namingConvention, > +Map > deploymentStatusTo

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +Map > deploymentStatusToStatus, Map instanceStatusToStatus) { > + this.nodeNamingConvention = checkNotNull(namingConvention, > "namingConvention").createWithoutPrefix(); > + this.images = checkNotNull(images, "images"); > + this.hardwares = checkN

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +osFamily = OsFamily.fromValue(from.getOSType().toLowerCase()); > +} catch (IllegalArgumentException e) { > +logger.debug("<< didn't match os(%s)", from.getName()); > +} > +boolean is64Bit = true; > +//(from.getName() != null && from

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +package org.jclouds.azurecompute.compute.functions; > + > +import com.google.common.base.Function; > +import > com.microsoft.windowsazure.management.compute.models.VirtualMachineOSImageGetResponse; > +import org.jclouds.azurecompute.domain.OSImage; > +import org.jclouds.logging.Logger; > + > +i

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > + > +import com.google.common.base.Function; > +import > com.microsoft.windowsazure.management.compute.models.DeploymentGetResponse; > +import org.jclouds.azurecompute.domain.VirtualMachine; > +import org.jclouds.logging.Logger; > + > +import javax.annotation.Resource; > +import javax.inject

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > +import com.google.common.base.Function; > +import > com.microsoft.windowsazure.management.compute.models.VirtualMachineOSImageListResponse; > +import org.jclouds.azurecompute.domain.OSImage; > +import org.jclouds.logging.Logger; > + > +import javax.annotation.Resource; > +import javax.injec

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +import org.jclouds.domain.Credentials; > +import org.jclouds.location.Provider; > + > +import javax.inject.Inject; > +import javax.inject.Singleton; > +import java.io.IOException; > + > +@Singleton > +public class ConfigurationSupplier implements Supplier { > +private final Configuration con

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + > +import javax.inject.Inject; > +import javax.inject.Singleton; > +import java.io.IOException; > + > +@Singleton > +public class ConfigurationSupplier implements Supplier { > +private final Configuration configuration; > + > +@Inject > +public ConfigurationSupplier(@Provider Suppli

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> + Optional imageExtension, > Optional securityGroupExtension) { > + super(context, credentialStore, images, hardwareProfiles, locations, > listNodesStrategy, getImageStrategy, > + getNodeMetadataStrategy, runNodesAndAddToSetStrategy, > rebootN

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> @@ -0,0 +1,67 @@ > +/** > + * Copyright (c)Microsoft Open Technologies, Inc. All rights reserved. All headers must be changed to the ASF license header. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/132/files#r24748196

Re: [jclouds-labs] In progress implementation of JClouds for Azure (#132)

2015-02-16 Thread Ignasi Barrera
> +import java.io.Closeable; > +import java.io.IOException; > + > +import org.jclouds.azurecompute.features.*; > + > +import javax.inject.Inject; > + > +public class AzureComputeApi implements Closeable { > +private final VirtualMachineApi virtualMachineApi; > +private final ImageApi imageA