@andreaturli pushed 1 commit.
bec0de5 - remove SslSelectChannelConnector import
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1230/files/2b2688e2080f5443d88915c83f840178871d546e..bec0de59ff8e539608778b44432e6db29
@andreaturli pushed 1 commit.
03f1781 downgrade jetty to 9.2.x as it runs on java7
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1230/files/bec0de59ff8e539608778b44432e6db291cc06ba..03f178150c1a84035cd2143ca1f1c2
lgtm
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/222#issuecomment-439082717
Thanks @mcdonnell-john for this. is there a jira issue tracking this problem,
could you maybe improve the description pointing to that jira issue?
Also have you signed http://apache.org/licenses/icla.pdf as per
https://www.apache.org/foundation/how-it-works/legal.html#incoming-code ?
Thanks aga
@nacx I'd like to have your thoughts, particularly if that was something
similar to your suggestion on IRC
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-64663355
You can merge this Pull Request by running:
git pull https://github.com/andreaturli/jclouds fix/sslcontext
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/615
-- Commit Summary --
* inject SSLSocketFactory instead of SSLContext
-- File Cha
Closed #615.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/615#event-199150436
TLSv1 is suggested here
http://www.oracle.com/technetwork/java/javase/documentation/cve-2014-3566-2342133.html
and it is not a docker requirement. It is only a simple way to avoid SSLv3
support, which is one of the protocol supported by default by
`SSLContext.getInstance("TLS")`
Anyway, I reve
Thanks @nacx seems like it is just-in-time collaboration: it will be super
useful for docker, I'm sure! Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/617#issuecomment-64756725
Thanks again @adriancole and @demobox
I've added a couple of comments to explain why Docker requires TLS and how this
PR deals with it (temporarily)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-64760771
> +* @param containerId The id of the container to be unpaused.
> +*/
> + @Named("container:unpause")
> + @POST
> + @Path("/containers/{id}/unpause")
> + void unpause(@PathParam("id") String containerId);
> +
> + /**
> +* Attach to a container
> +*
> +* @param containe
> +* @param containerId The id of the container to be unpaused.
> +*/
> + @Named("container:unpause")
> + @POST
> + @Path("/containers/{id}/unpause")
> + void unpause(@PathParam("id") String containerId);
> +
> + /**
> +* Attach to a container
> +*
> +* @param containe
> +* @param containerId The id of the container to be unpaused.
> +*/
> + @Named("container:unpause")
> + @POST
> + @Path("/containers/{id}/unpause")
> + void unpause(@PathParam("id") String containerId);
> +
> + /**
> +* Attach to a container
> +*
> +* @param containe
Big +1
Thanks @nacx I'll test it asap with Docker!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/619#issuecomment-65272200
I'll probably be able to close that later this week or during weekend.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-66356676
@nacx tests are green, it is rebased to master but if we are happy with that,
probably best to squash them before merging it.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-66914824
hi @danbroudy I think it is still an issue. Basically the current code creates
too many (identical) firewalls if I deploy multiple VMs in the same network. In
fact, if the I ask for 10 nodes with specific inboundPorts, multiple firewalls
with the same rules are created inside the network for the
> + private static X509Certificate getCertificate(String certificate) {
> + try {
> + return (X509Certificate)
> CertificateFactory.getInstance("X.509").generateCertificate(
> + new
> ByteArrayInputStream(certificate.getBytes(Charsets.UTF_8)));
> + } catch (Cer
> private final TrustManager[] trustManager;
> private final Supplier creds;
>
> @Inject
> - SSLContextWithKeysSupplier(Supplier keyStore, @Provider
> Supplier creds, HttpUtils utils,
> - TrustAllCerts trustAllCerts) {
> - this.keyStore = keyStore;
> - this.trustM
> private final TrustManager[] trustManager;
> private final Supplier creds;
>
> @Inject
> - SSLContextWithKeysSupplier(Supplier keyStore, @Provider
> Supplier creds, HttpUtils utils,
> - TrustAllCerts trustAllCerts) {
> - this.keyStore = keyStore;
> - this.trustM
We want jclouds to provide a sensible default with `NewOkHttpClient` but give
to an advanced user the ability to override completely that behavior using
`OkHttpClientSupplier`
You can merge this Pull Request by running:
git pull https://github.com/andreaturli/jclouds fix/okhttp
Or you can vie
Closed #627.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/627#event-208207105
> @@ -67,7 +67,7 @@ public static Attachment create(String hostedService,
> String deployment, String
> @Nullable public abstract String description();
>
> /** The operating system type of the OS image, or null if a data disk. */
> - @Nullable public abstract OSImage.Type os();
> + @
> @@ -95,9 +95,9 @@ public static Attachment create(String hostedService,
> String deployment, String
> */
> @Nullable public abstract String sourceImage();
>
> - public static Disk create(String name, String location, String
> affinityGroup, String description, OSImage.Type os,
> +
> URI mediaLink, Integer logicalSizeInGB, Attachment attachedTo,
> String sourceImage) {
> - return new AutoValue_Disk(name, location, affinityGroup, description,
> os, mediaLink, logicalSizeInGB, attachedTo,
> + return new AutoValue_Disk(name, location, affinityGroup, descrip
> @@ -37,7 +37,7 @@
> public abstract String name();
>
> /** The geo-locations of the image, if the image is not associated with
> an affinity group. */
> - public abstract List locations();
> + @Nullable public abstract String location();
Good spot!
Please make sure that all the f
> @@ -68,7 +68,7 @@ private void checkOSImage(OSImage OSImage) {
> "MediaLink should be an http(s) url" + OSImage);
>}
>
> - assertTrue(locations.containsAll(OSImage.locations()), "Locations not
> in " + locations + " :" + OSImage);
> + // assertTrue(locations.
> @@ -60,8 +60,8 @@ public void testList() {
> }
>
> private void checkDisk(Disk disk) {
> - assertNull(disk.name(), "Name cannot be null for: " + disk);
> - assertNull(disk.os(), "OS cannot be null for: " + disk);
> + assertNotNull(disk.name(), "Name cannot be null for: "
> +
> + ImageBuilder builder = new ImageBuilder()
> +.id(image.label())
> +.name(image.name())
> +.description(image.description())
> +.status(Image.Status.AVAILABLE)
> +.uri(image.mediaLink())
> +.providerId(image.publish
Thanks @hsbhathiya for your PR, very good start!
There are some comments from my side, please try to address them and if you
have troubles, feel free to reach out to us on IRC #jclouds.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/117#issue
> + .description(image.description());
> + }
> + return
> OperatingSystem.builder().family(OsFamily.LINUX).version(version).description(image.description());
> + }
> +
> + private Location createLocation(String input) {
> + return new
> LocationBuilder().id(input)
> +.id(image.label())
> +.name(image.name())
> +.description(image.description())
> +.status(Image.Status.AVAILABLE)
> +.uri(image.mediaLink())
> +.providerId(image.publisherName())
> +.location(createLocation(image
> +.status(Image.Status.AVAILABLE)
> +.uri(image.mediaLink())
> +.providerId(image.publisherName())
> +.location(createLocation(image.location()));
> +
> + OperatingSystem.Builder osBuilder = setOperatingSystem(image);
> + return builder.ope
>
> public class OSImageToImage implements Function {
> + private static final String UNRECOGNIZED = "UNRECOGNIZED";
> + private final JustProvider provider;
> +
> + @Inject
> + public OSImageToImage(JustProvider provider) {
not sure we will need this parameter here, as we'd probably us
> + //openSUSE 13.1 -> 13.1
> + if (label.toUpperCase().contains("OPENSUSE"))
> + return label.replace("openSUSE ", "");
> + //SUSE Linux Enterprise Server 11 SP3 (Premium Image) ->
> 11 SP3(Premium Image)
> + else if (lab
> @@ -68,7 +68,7 @@ private void checkOSImage(OSImage OSImage) {
> "MediaLink should be an http(s) url" + OSImage);
>}
>
> - assertTrue(locations.containsAll(OSImage.locations()), "Locations not
> in " + locations + " :" + OSImage);
> +assertTrue(locations.co
@nacx Any chance to give your thoughts on that?
FYI I've been also experimenting with the latest boot2docker available which
runs docker 1.4.1 and docker API v1.16 and the code needs only a minor
adjustment for Info class as that version is returning a richer object. Other
than that, the other
As rule of thumb, better to open PR at the time, so we can help you more.
do u really think Role is really needed? If yes, where?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/118#issuecomment-67872617
Also, I know about your roadmap and I'd suggest you to not focus on this
translator during Milestone 1.
In fact, the first thing to do IMO is to finalise the `features` API and all
the `domain` objects managed by those API. Once you get those right, it would
be straightforward to finalise the A
> throw propagate(e);
>}
> }
> +
> + private static X509Certificate getCertificate(String certificate) {
> + try {
> + return (X509Certificate)
> CertificateFactory.getInstance("X.509").generateCertificate(
> + new
> ByteArrayInputStream(certif
> + private static X509Certificate getCertificate(String certificate) {
> + try {
> + return (X509Certificate)
> CertificateFactory.getInstance("X.509").generateCertificate(
> + new
> ByteArrayInputStream(certificate.getBytes(Charsets.UTF_8)));
> + } catch (Cer
@aledsage code looks good to me, but the builder is not happy because of some
[checkstyle
violations](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1475/org.apache.jclouds.provider$aws-ec2/violations/)
+1 to PR against master as well, once you address the violations
---
Reply to t
hi @jdaggett do yo think it is time to merge that?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/134#issuecomment-69171874
> @@ -25,6 +25,7 @@ This involves [dockerizing an SSH daemon
> service](https://docs.docker.com/exampl
> // get a context with docker that offers the portable ComputeService api
> ComputeServiceContext context = ContextBuilder.newBuilder("docker")
>.credentials(email, pas
> +api.getVirtualGuestApi();
> +api.getDatacenterApi();
> +api.getSoftwareDescriptionApi();
> +api.getVirtualGuestBlockDeviceTemplateGroupApi();
> +api.getAccountApi();
> +
> +// Be sure to close the context when done
> +computeServiceContext.close();
> +{% endhighlight %}
> +
> +## SoftLayer Objec
@demobox I've addressed your questions. Are we good to merge that now?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/122#issuecomment-69715798
@nacx I think I need to rebase and squash so that you can merge it, yes?
Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-69901231
@nacx I think you are right, better to remove the password as we don't suggest
to use encrypted PEM file
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-69907182
argh! @nacx
Fixing them now
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-69925743
Closed #113.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#event-218832236
You can merge this Pull Request by running:
git pull https://github.com/andreaturli/jclouds-labs docker-1.4.1
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds-labs/pull/120
-- Commit Summary --
* support docker 1.4.1 (API v1.16)
-- File Changes
You can merge this Pull Request by running:
git pull https://github.com/andreaturli/jclouds-labs docker-fix-test
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds-labs/pull/121
-- Commit Summary --
* fix docker unit tests
-- File Changes --
@iocanel lgtm! I've tested and it works great!
thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/25#issuecomment-70151677
Could you please squash those 3 commits into 1, so that I can merge it?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/25#issuecomment-70152193
Merged #25.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/25#event-219672059
> + *
> + * 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 KIND, either express or implied.
> + * See the License for the specific language governing permission
> + * distributed under the License is distributed on an "AS IS" BASIS,
> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + * See the License for the specific language governing permissions and
> + * limitations under the License.
> + */
> +package org.jclouds.azureco
> import com.google.common.collect.ImmutableSet;
>
> @Test(groups = "live", testName = "OSImageApiLiveTest")
> public class OSImageApiLiveTest extends BaseAzureComputeApiLiveTest {
>
> - private ImmutableSet locations;
why did you remove this check from the test?
---
Reply to this email
thanks for that @hsbhathiya!
Btw, it is a bit hard to review this PR all together, can you maybe split it
into multiple PRs.
In fact, the code looks ok, but I'd like to have time to look at the official
Azure documentation to see if the Value objects cover the spec correctly. Just
to give you an
> +
> +/**
> + * @see href="http://msdn.microsoft.com/en-us/library/azure/jj157186.aspx#OSVirtualHardDisk";
> >api
> + */
> +@AutoValue
> +public abstract class OSVirtualHardDisk {
> +
> + /**
> +* Specifies the caching mode of the operating system disk.
> +* This setting impacts the co
> +/**
> + * @see href="http://msdn.microsoft.com/en-us/library/azure/jj157186.aspx#OSVirtualHardDisk";
> >api
> + */
> +@AutoValue
> +public abstract class OSVirtualHardDisk {
> +
> + /**
> +* Specifies the caching mode of the operating system disk.
> +* This setting impacts the consis
> +* in Windows Azure. If you specify the path to an image with this
> element, an associated VHD is created and
> +* you must use the MediaLink element to specify the location in storage
> where the VHD will be located.
> +* If this element is used, SourceImageName is not used.
> +
> @@ -0,0 +1,11 @@
> +http://schemas.microsoft.com/windowsazure";>
> +ReadOnly
> +MyTestImage_1
> +testosimage1-testosimage1-0-20120817095145
> +
> http://blobs/disks/neotysss/MSFT__Win2K8R2SP1-ABCD-en-us-30GB.vhd
> +Ubuntu Server 12.04 LTS
> +linux
> +30
> +
> http:
@hsbhathiya thanks for splitting the PR, it is now way easier to review!
There are just some comments to get the Value class right, according to the
online doc, at least.
Please revisit the impl, and commit the changes.
Once we'll be happy with the PR, we will squash the commits into just one
c
> + *
> + * 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 KIND, either express or implied.
> + * See the License for the specific language governing permission
> +
> +/**
> + * @see href="http://msdn.microsoft.com/en-us/library/azure/jj157186.aspx#OSVirtualHardDisk";
> >api
> + */
> +@AutoValue
> +public abstract class OSVirtualHardDisk {
> +
> + /**
> +* Specifies the caching mode of the operating system disk.
> +* This setting impacts the co
lgtm! thanks @hsbhathiya
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/125#issuecomment-70991125
> +api.getVirtualGuestApi();
> +api.getDatacenterApi();
> +api.getSoftwareDescriptionApi();
> +api.getVirtualGuestBlockDeviceTemplateGroupApi();
> +api.getAccountApi();
> +
> +// Be sure to close the context when done
> +computeServiceContext.close();
> +{% endhighlight %}
> +
> +## SoftLayer Objec
> +* If the disk that is being added is already registered in the
> subscription or the VHD for the disk already exists
> +* in blob storage, this element is ignored. If a VHD file does not exist
> in blob storage, this element defines the
> +* location of the new VHD that is created
> +
> + /**
> +* Required if an existing disk is being used to create a Virtual Machine.
> +* Specifies the name of a new or existing disk
> +*/
> + @Nullable public abstract String diskName();
> +
> + /**
> +* Specifies the Logical Unit Number (LUN) for the data disk. If the
Reverts jclouds/jclouds-labs#125
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs/pull/127
-- Commit Summary --
* Revert "JCLOUDS-664: OSVirtualHardDisk Support With Tests"
-- File Changes --
-- Patch Links --
https://github.com/jclo
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/653
-- Commit Summary --
* JCLOUDS-813: fix list private images
-- File Changes --
M
providers/softlayer/src/main/java/org/jclouds/softlayer/compute/strategy/SoftLayerComputeServic
@nacx does it look good?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/653#issuecomment-71675280
thanks @grkvlt! lgtm +1
merging now
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/654#issuecomment-71797889
Merged as
https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=9e1d1cbfa9e2d77fe8b7e8feeae0bb4222654692
@grkvlt thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/654#issuecomment-71800808
Closed #654.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/654#event-225930035
@nacx Some users get a NPE when `listImages` is called.
This is happening
[here](https://github.com/andreaturli/jclouds/blob/fix/softlayer-list-private-images/providers/softlayer/src/main/java/org/jclouds/softlayer/domain/OperatingSystem.java#L107).
Debugging a little bit, I found out the reason:
correct! it should be already available in 2.0.0-SNAPSHOT available in maven
central, I think.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/654#issuecomment-71862219
Closed #653.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/653#event-226188226
To fix the bug reported at JCLOUDS-813 this PR revisit the strategy to
listImages:
- modify the listImages and getImage strategies in
SoftLayerComputeServiceAdapter: by default, it only consider `stock` images
- add support for VirtualGuestBlockDeviceTemplateGroupApi getObject: this is
need in `
@nacx can you have another look at it? thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#issuecomment-71872105
@nacx I've addressed your comments, sorry I forgot about the tests.
I need to port the expectTests to MWS but not in this PR
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#issuecomment-72004696
@nacx if the builder is happy can backport this to 1.8.x as well?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#issuecomment-72005057
Closed #629.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/629#event-226766595
Great, @nacx! The builder is happy, can I merge it now?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#issuecomment-72011690
Thanks @aledsage
Merged at
https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=ee42fb1a687d3f8de0db9ff870cb7e9ad00d53aa
and
https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=7866612a2eca6e3ee32bc7928e692ec325ac7d30
---
Reply to this email directly or view it on G
Reopened #655.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#event-226802599
Closed #655.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#event-226802574
Great, @nacx! Squashed and rebased but the builder is not happy, because of an
unrelated issue. Closing and re-opening the PR to trigger the builder again
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/655#issuecomment-72020581
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/657
-- Commit Summary --
* Update pom.xml
-- File Changes --
M allcompute/pom.xml (15)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/657.patch
https://github.com/jclou
@jdaggett can you please have a look?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/657#issuecomment-72029281
- Change each individual project in the repo, and the root pom.xml to use
jclouds-project as a parent.
- Replace all occurrences of jclouds.version by project.parent.version.
similarly to jclouds-labs-openstack repo
@nacx can you have a look?
You can view, comment on, or merge this pull request o
@hsbhathiya could you please have a look?
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs/pull/131
-- Commit Summary --
* azurecompute: improvements to the domain objects
-- File Changes --
A
azurecompute/src/main/java/org/jcloud
Merged as
https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=fa8b9e2332e3a8267d201d61b851f9d91d9e2a27
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/666#issuecomment-72895033
Closed #666.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/666#event-230181856
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/668
-- Commit Summary --
* remove cloudservers
-- File Changes --
M allcompute/pom.xml (15)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/668.patch
https://github.com/
@nacx mind merging it?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/131#issuecomment-73908678
Closed #131.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/131#event-234092985
Merged at
https://git1-us-west.apache.org/repos/asf?p=jclouds-labs.git;a=blob;f=azurecompute/src/test/java/org/jclouds/azurecompute/xml/NetworkSecurityGroupHandlerTest.java;hb=a34e62ca
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/131#issuecom
improve DeploymentToNodeMetadata
enhance Deployment value object
add support for get/set NetworkConfiguration
added AzureComputeTemplateOptions to manage networkConfigurations
add support for create/checkAvailable/list storageAccounts
add support for inboundPorts
add more RoleSize
add support for S
301 - 400 of 1680 matches
Mail list logo
