thanks @neykov
--
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-karaf/pull/81#issuecomment-246346027
> @@ -46,9 +44,9 @@ protected void bindErrorHandlers() {
> @Override
> protected void installLocations() {
>super.installLocations();
> - bind(ImplicitLocationSupplier.class).
> - to(OnlyLocationOrFirstRegionOptionallyMatchingRegionId.class).
@nacx I'm having
@andreaturli pushed 1 commit.
aaae684 revert ZONES to REGIONS
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/308/files/961db5c145342971726063e25776e0011ab8ed36..aaae68457526b198a2b30d7f648b4ef0b46ec359
> @@ -46,9 +44,9 @@ protected void bindErrorHandlers() {
> @Override
> protected void installLocations() {
>super.installLocations();
> - bind(ImplicitLocationSupplier.class).
> - to(OnlyLocationOrFirstRegionOptionallyMatchingRegionId.class).
I think
ok thanks, I guess this one confused me
https://github.com/jclouds/jclouds-labs/pull/308#issuecomment-239163128 :)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@nacx good to merge?
--
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-labs/pull/308#issuecomment-246959604
When running in isolation the *LiveTests are working fine, they fail when
running all together from maven. (see
https://gist.github.com/andreaturli/2725723193ee1a8362f175e8e8c76ee8)
You can view, comment on, or merge this pull request online at:
```
Results :
Failed tests:
AzureComputeServiceLiveTest>BaseComputeServiceLiveTest.testCompareSizes:887->BaseComputeServiceLiveTest.doCompareSizes:891
» Provision
@andreaturli pushed 1 commit.
071c533 fix checkstyle
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/317/files/388177aa18ed03c633a3216f7dbc15be2c51..071c533908fc8e908074f0bd6fc6deb75d325782
- oauth.{endpoint.identity, credential} to jclouds.oauth.{endpoint, identity,
credential}
@nacx what do you think? I think it is more homogenous this way
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/1014
-- Commit Summary --
*
For review only
@nacx I'd appreciate if you could have a look at it
Still something wrong
```
java.lang.AssertionError: expected [[user=foo, passwordPresent=true,
privateKeyPresent=false, shouldAuthenticateSudo=true]] but found
[[user=jclouds, passwordPresent=true, privateKeyPresent=false,
@andreaturli pushed 2 commits.
1b958fc fixing createNodeWithGroupEncodedIntoName
358aa82 improvements to solve AzureComputeServiceLiveTest
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@andreaturli pushed 1 commit.
9456ad1 revert VirtualMachineApi
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/323/files/358aa82d9bf70a870c1abe89ca1ae4d5939adf5e..9456ad1468e8c13e3f9935876fbecf00fb29323c
@andreaturli pushed 3 commits.
21403be refactor from Deployment to VirtualMachine
84ed11f fix Cleanup resources
e450fd4 some more fixes
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@andreaturli pushed 1 commit.
8d223b1 remove DeploymentToNodemetadata
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
looks so clean, thx @nacx ! Merge it right away!
--
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-labs/pull/321#issuecomment-248651612
Closed #320.
--
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-labs/pull/320#event-797411872
thx @nacx, merged at
[master](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/28c56e06)
--
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-labs/pull/320#issuecomment-248632037
```
Failed tests:
AzureComputeServiceLiveTest>BaseComputeServiceLiveTest.testCompareSizes:887->BaseComputeServiceLiveTest.doCompareSizes:891
» Provision
@nacx happy to share the live tests results, **but** it would be useful to
merge first https://github.com/jclouds/jclouds-labs/pull/308 as it speed up the
live tests significantely
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on
@andreaturli pushed 3 commits.
79c8f32 add image publishers filter to AzureComputeServiceLiveTest
eefa3bd add parser module to provider
de2d18b fix checkstyle
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
andreaturli commented on this pull request.
> @@ -28,6 +30,7 @@ public static Properties defaultProperties(Properties
> properties) {
properties.put("oauth.credential", "password");
properties.put("oauth.endpoint",
"https://login.microsoftonline.com/oauth2/token;);
andreaturli commented on this pull request.
> @Inject
AzureComputeServiceAdapter(final AzureComputeApi api, final
AzureComputeConstants azureComputeConstants,
- CleanupResources cleanupResources) {
+ CleanupResources
@andreaturli pushed 1 commit.
388177a fix some live tests more
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/317/files/342666fe60a3c1ec24919a99cde90b6d27dcc9ad..388177aa18ed03c633a3216f7dbc15be2c51
New results
```
Failed tests:
AzureComputeServiceLiveTest>BaseComputeServiceLiveTest.testCompareSizes:887->BaseComputeServiceLiveTest.doCompareSizes:891
» Provision
@andreaturli pushed 1 commit.
342666f fix create network card interface
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@andreaturli pushed 2 commits.
12f49f2 add parser module
8f403aa separate the DeploymentToVMDeployment to a function
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@andreaturli pushed 1 commit.
07007d9 refactor logback-test.xml
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/308/files/8f403aa5cc1c711496a670c3fbf3e03eadca969e..07007d994a33b1cc3c62a6294e8760141014758e
- refactor logback-test.xml
- change jsonBall to Value
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs/pull/320
-- Commit Summary --
* separate the DeploymentToVMDeployment to a function
-- File Changes --
M
Closed #308.
--
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-labs/pull/308#event-795734646
merged at
(master)[http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/89cb6f0f]
--
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-labs/pull/308#issuecomment-248317509
@nacx I think we are good to merge this
```
Results :
Failed tests:
AzureComputeServiceLiveTest>BaseComputeServiceLiveTest.testCompareSizes:887->BaseComputeServiceLiveTest.doCompareSizes:891
» Provision
Closed #317.
--
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-labs/pull/317#event-803805178
merged at
[master](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/c458bafd)
--
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-labs/pull/317#issuecomment-249864763
@nacx I think we are good to merge this
```
Results :
Failed tests:
AzureComputeServiceLiveTest>BaseComputeServiceLiveTest.testCompareSizes:887->BaseComputeServiceLiveTest.doCompareSizes:891
» Provision
@andreaturli pushed 1 commit.
1bf9d27 fix VirtualMachineLiveTest
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/317/files/071c533908fc8e908074f0bd6fc6deb75d325782..1bf9d272b61efae7ab5d2b9a18674a43898d73b8
@andreaturli pushed 1 commit.
f8c20ab remove default creds for all images
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
Thx @nacx! Not got to the bottom of it, but if I specify the default creds for
the images, the test that tries to override them fail!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
thanks @nacx but I don't follow completely.
For example, to run tests in `jclouds-labs/azurecompute-arm` I need to specify:
```
mvn clean install -Plive -Dtest.azurecompute-arm.identity="identity"
-Dtest.azurecompute-arm.credential="password"
because I thought it was jclouds's convention, thinking about stuff like
```
public static final String AUTO_ALLOCATE_FLOATING_IPS =
"jclouds.openstack-nova.auto-create-floating-ips";
```
so for simmetry I'd have expected that for oauth things as well, no?
--
You are receiving this because you
Closed #1014.
--
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/pull/1014#event-802329941
thx @nacx for the explanation! I'll close the pr as not needed
--
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/pull/1014#issuecomment-249579672
great! you want to continue on this PR or prefer merging it and work on the
last failing test on a different PR?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
+1
--
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/pull/1015#issuecomment-249585667
I think you are right!
--
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/pull/1027#issuecomment-256376606
/cc @nacx
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/1027
-- Commit Summary --
* fix jclouds/apis/swift retryOnRenew
-- File Changes --
M
common/openstack/src/main/java/org/jclouds/openstack/handlers/RetryOnRenew.java
(7)
@andrewgaul retriers in general shouldn't close the streams if they are not
reading the payload, as
https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/http/internal/BaseHttpCommandExecutorService.java#L144
will close it anyway.
--
You are receiving this because you
rebuild please
--
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/pull/1027#issuecomment-256448373
ok cool. I think I will volunteer for a new 1.9.x release :)
--
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/pull/1027#issuecomment-256953771
this PR is just to double-check everything is alright
--
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/pull/1031#issuecomment-256954646
merged at
[1.9.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/7167b473)
@nacx @zack-shoylev shall we port that to `master` as well?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Closed #1031.
--
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/pull/1031#event-840319356
ok merged at
[master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/9b3e6d91)
--
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/pull/1031#issuecomment-256971205
This port to master the PR #1027 already merged in `1.9.x`
These classes should not close the release the payload as they are not
reading it.
- fix swift
- fix openstack-swift v1 and v2
- fix RetryOnRenewTest for v1 and v2
You can view, comment on, or merge this pull request online at:
/cc @nacx
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/1032
-- Commit Summary --
* [aws-ec2] Add CRUD for VPC
-- File Changes --
M providers/aws-ec2/pom.xml (5)
M
ok merging now on master and backport to 1.9.x
thanks!
--
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/pull/1020#issuecomment-253742999
Merged at
[master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/2c6caa04) and
bacported to
[1.9.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/a843196b).
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Closed #1020.
--
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/pull/1020#event-823673856
Closed #323.
--
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-labs/pull/323#event-823745486
I think this is superseded by #324 so closing it now
--
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-labs/pull/323#issuecomment-253757624
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/324#issuecomment-253772862
thx @nacx
--
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/pull/956#issuecomment-256049449
Didnt know azure has that rate limit, thx @nacx
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/329#issuecomment-255257826
Could you merge it please? Not at my laptop just now. Thanks
--
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-examples/pull/90#issuecomment-263021863
@demobox are you happy with the status of this PR?
--
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-examples/pull/90#issuecomment-263066699
andreaturli commented on this pull request.
> +date: 2016-11-23 12:00:00
+release_notes: true
+permalink: /releasenotes/1.9.3/
+---
+
+1. [Introduction](#intro)
+1. [Credits](#credits)
+1. [Test Results](#test)
+
+## Introduction
+
+You can read the official announcement at [Apache jclouds
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-site/pull/190
-- Commit Summary --
* add release 1.9.3
-- File Changes --
M _includes/navbar.html (1)
M doap_jclouds.rdf (7)
A releasenotes/1.9.3.md (39)
M
andreaturli commented on this pull request.
> @@ -51,6 +51,11 @@
google-compute-engine
${jclouds.version}
+
+ io.cloudsoft.jclouds.api
+ openstack-mitaka-nova
+ 1.9.3-cloudsoft.20160831
+
oh, don't know why it's there, I'll remove it right away
@andreaturli pushed 1 commit.
df0e785 address comments
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/90/files/70f574af20bc8f30d5835d907411fe85ab259945..df0e785adec6ca32582aef2d942190cfb4370fa4
andreaturli commented on this pull request.
> @@ -69,35 +73,47 @@
public static void main(String[] args) throws IOException {
- if (args.length < PARAMETERS)
+ String provider;
+ String identity;
+ String credential;
+ String containerName;
+ String
andreaturli commented on this pull request.
>// note that you can check if a provider is present ahead of time
checkArgument(contains(allKeys, provider), "provider %s not in supported
list: %s", provider, allKeys);
-
- String identity = args[1];
- String credential =
andreaturli commented on this pull request.
> // Create Container
- BlobStore blobStore = context.getBlobStore();
Location location = null;
not sure I follow, looking at the git history I can see this commit
@andreaturli pushed 1 commit.
7bbd5fc addressing more comments
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/90/files/31184ff15c5d8aa6f2b2c7af0f10f17e811c33b0..7bbd5fc5d7f14d7ed12b76cadf37bf9757ff936e
andreaturli commented on this pull request.
> @@ -138,6 +154,10 @@ public static void main(String[] args) throws
> IOException {
}
} finally {
+ // delete cointainer
+ if (blobStore.containerExists(containerName)) {
ok
--
You are receiving this
andreaturli commented on this pull request.
>
// Init
- BlobStoreContext context = ContextBuilder.newBuilder(provider)
- .credentials(identity,
credential)
-
andreaturli commented on this pull request.
> Location location = null;
if (apiMetadata instanceof SwiftApiMetadata) {
location = Iterables.getFirst(blobStore.listAssignableLocations(),
null);
}
- blobStore.createContainerInLocation(location,
andreaturli commented on this pull request.
> @@ -69,35 +73,47 @@
public static void main(String[] args) throws IOException {
- if (args.length < PARAMETERS)
+ String provider;
+ String identity;
+ String credential;
+ String containerName;
+ String
update README.md
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-examples/pull/90
-- Commit Summary --
* update blobstore-basics to use 2.0.0
-- File Changes --
M blobstore-basics/README.md (11)
M blobstore-basics/pom.xml (4)
thanks @demobox and @nacx, merging now!
--
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/190#issuecomment-262821539
Merged #190.
--
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/190#event-871361747
@andreaturli pushed 1 commit.
56cbcbd fix typos in releasenetes for 1.9.3
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
andreaturli commented on this pull request.
> +## Introduction
+
+You can read the official announcement at [Apache jclouds 1.9.3
released](http://markmail.org/thread/qlapnppmfbilje7p). To get started with
jclouds, please see the [jclouds installation guide](/start/install/).
+
+This is a
@demobox can you have another shot at it?
--
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/190#issuecomment-262810954
thanks @nacx, good spot!
--
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/190#issuecomment-262815836
@andreaturli pushed 1 commit.
7b48221 fix typos in releasenotes for 1.9.3
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
thx @googlielmo
--
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/pull/1039#issuecomment-263231284
Closed #1039.
--
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/pull/1039#event-873386517
Backports https://github.com/jclouds/jclouds/pull/1013
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/1040
-- Commit Summary --
* JCLOUDS-1182: Added Seoul and Mumbai regions to AWS
-- File Changes --
M
rebuild please
--
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/pull/1040#issuecomment-261255266
Closed #1040.
--
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/pull/1040#event-862981462
lgtm, thanks @googlielmo
merging now
--
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/pull/1039#issuecomment-261552521
merged at master http://git-wip-us.apache.org/repos/asf/jclouds/commit/5a3cfe5d
--
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/pull/1039#issuecomment-261554233
ok backported to 1.9.x at
https://git1-us-west.apache.org/repos/asf?p=jclouds.git;a=commit;h=1f1aceb8
--
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/pull/1039#issuecomment-261555183
do you like to have that backported to 1.9.x?
--
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/pull/1039#issuecomment-261553584
andreaturli commented on this pull request.
> + * Microsoft Azure Resource Manager
+ * ProfitBricks v3
+ * OneAndOne
+* Added support for new blob store providers:
+ * Backblaze B2
+* Better integration with OSGi and Apache Karaf.
+* Numerous bug fixes and performance improvements.
+
+The
@andreaturli pushed 1 commit.
77ef915 address comments for PR 1032
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1032/files/55c205a2c08959e97ce079f9c2f2d901d57ea524..77ef91522940ee4b60cddea5286e08ba8b1aafbf
andreaturli commented on this pull request.
> + VPC() {}
+
+ public static Builder builder() {
+ return new AutoValue_VPC.Builder();
+ }
+
+ @AutoValue.Builder
+ public abstract static class Builder {
+
+ public abstract Builder id(String id);
+ public abstract Builder
andreaturli commented on this pull request.
> + client = view.unwrapApi(AWSEC2Api.class).getVPCApi().get();
+ }
+
+ @Test
+ public void testCreate() {
+ vpc = client.createVpc(null, "10.0.0.0/16", CreateVpcOptions.NONE);
+ assertNotNull(vpc);
+ }
+
+
andreaturli commented on this pull request.
> + }
+ }
+ }
+
+ @Nullable
+ public abstract String id();
+ @Nullable
+ public abstract State state();
+ @Nullable
+ public abstract String cidrBlock();
+ @Nullable
+ public abstract String dhcpOptionsId();
+
andreaturli commented on this pull request.
> + VPC() {}
+
+ public static Builder builder() {
+ return new AutoValue_VPC.Builder();
+ }
+
+ @AutoValue.Builder
+ public abstract static class Builder {
+
+ public abstract Builder id(String id);
+ public abstract Builder
701 - 800 of 1591 matches
Mail list logo