[jira] [Commented] (JCLOUDS-1302) wish to support the OSS of the Aliyun

2017-05-18 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1302?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16016818#comment-16016818 ] Andrew Gaul commented on JCLOUDS-1302: -- This will require someone writing a new provider:

[jira] [Created] (JCLOUDS-1302) wish to support the OSS of the Aliyun

2017-05-18 Thread zreal (JIRA)
zreal created JCLOUDS-1302: -- Summary: wish to support the OSS of the Aliyun Key: JCLOUDS-1302 URL: https://issues.apache.org/jira/browse/JCLOUDS-1302 Project: jclouds Issue Type: Wish

[jira] [Commented] (JCLOUDS-1258) S3 China Beijing Region Support

2017-05-18 Thread zreal (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16016788#comment-16016788 ] zreal commented on JCLOUDS-1258: when I change the

[jira] [Commented] (JCLOUDS-1258) S3 China Beijing Region Support

2017-05-18 Thread zreal (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16016769#comment-16016769 ] zreal commented on JCLOUDS-1258: Hi, Michelle Zhang. when I create a bucket using

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Ignasi Barrera
Good! Have you been able to run the live tests for the affected APIs? >By the way how is the @ApiVersion actually injected in places like FormSigner, >for example? I think the value is being taken from AWSEC2ApiMetadata but I >couldn't work out how jclouds configures Guice (or whatever) to

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Geoff Macartney
Updates made. I was able to switch to using `@SinceApiVersion`, very pleased with that. The [key thing](https://github.com/jclouds/jclouds/pull/1102/commits/3bff59d28c5260e18504c6570ad5cb47117a3d48#diff-7d8365e45e83df7988b1084146439d71R158) is that I needed to change the override from a

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Geoff Macartney
@geomacy pushed 3 commits. 3bff59d Use @SinceApiVersion rather than a custom annotation. 110b188 Use Optional rather than returning null. 74e4245 Apply the version check in FormSignerV2 as well as V4 -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Ignasi Barrera
Unit tests may fail because they hardcode the expected version. Live tests will tell us if the change is safe. Let's run them for the classes that use the annotation. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Geoff Macartney
@nacx I can add the annotation at class level. I originally started using `@SinceApiVersion`, and having the annotation at both method and class level, but found that various tests broke, because various APIs are annotated `@SinceApiVersion`.I will take a look at that again, but

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-18 Thread Andrew Gaul
andrewgaul commented on this pull request. > +* @param prefix +* +*/ + private void removeObjectsWithPrefix(String container, String prefix) { + String nextMarker = null; + ObjectApi objectApi = api.getObjectApi(regionId, container); + do { +

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Ignasi Barrera
I like the annotation approach and agree that the flexibility needed in ARM, where they deploy new versions of the APIs quite often (but keeping backward compatibility) might not be needed here. A couple questions: * It would be worth considering the annotation at class level too. Could you

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Andrea Turli
I think we should edit the FormSignerV2 as well -- 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/1102#issuecomment-302394483

Re: [jclouds/jclouds] Add ModifySubnetAttribute (#1102)

2017-05-18 Thread Andrea Turli
andreaturli commented on this pull request. looks really useful as we discussed on IRC. Some minor comments, but @geomacy your impl is even cleaner than expected using the annotation. Thanks! > @@ -101,7 +105,25 @@ this.serviceAndRegion = serviceAndRegion; } - @Override public

[jira] [Commented] (JCLOUDS-1293) AWS ERROR - Unexpected end of file from server

2017-05-18 Thread Stuart Hendren (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16015586#comment-16015586 ] Stuart Hendren commented on JCLOUDS-1293: - An example retry handler that also needs the above

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-18 Thread archupsg03
archupsg03 commented on this pull request. > +* @param prefix +* +*/ + private void removeObjectsWithPrefix(String container, String prefix) { + String nextMarker = null; + ObjectApi objectApi = api.getObjectApi(regionId, container); + do { +