You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Andrew Gaul <no...@github.com> on 2014/09/05 03:03:31 UTC

[jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

This will allow use of Azure after August 2015:

http://blogs.msdn.com/b/windowsazurestorage/archive/2014/08/05/microsoft-azure-storage-service-version-removal.aspx
You can merge this Pull Request by running:

  git pull https://github.com/andrewgaul/jclouds azure-version

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/508

-- Commit Summary --

  * JCLOUDS-706: Upgrade Azure API to 2012-02-12

-- File Changes --

    M providers/azureblob/pom.xml (2)
    M providers/azureblob/src/main/java/org/jclouds/azureblob/AzureBlobApiMetadata.java (2)
    M providers/azureblob/src/main/java/org/jclouds/azureblob/AzureBlobAsyncClient.java (2)
    M providers/azureblob/src/test/java/org/jclouds/azureblob/AzureBlobAsyncClientTest.java (38)
    M providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java (12)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/508.patch
https://github.com/jclouds/jclouds/pull/508.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Gaul <no...@github.com>.
Successfully ran integration tests.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54566376

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Gaul <no...@github.com>.
Closed #508.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#event-162027454

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1146](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1146/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54573733

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Phillips <no...@github.com>.
>  I will improve the SkipException condition to address this.

Thanks! Sorry, blanked for a second there that integration tests _are_ run against the actual target, indeed ;-)

+1, looks good to me!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54692803

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Gaul <no...@github.com>.
As stated above, integration tests successfully passed.

As for the failure, Cloudbees does not seem to run on a filesystem with xattr support.  I will improve the `SkipException` condition to address this.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54692667

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1596](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1596/) UNSTABLE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54575667

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #57](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/57/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54566458

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Gaul <no...@github.com>.
Pushed to master as 975c1325bcf980ef4da1de00d1d5843a1d788399.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54694817

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Phillips <no...@github.com>.
> jclouds » jclouds #1596 UNSTABLE

This looks like a [real failure](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/org.apache.jclouds.api$filesystem/1596/testReport/org.jclouds.filesystem.strategy.internal/FilesystemStorageStrategyImplTest/testOverwriteBlobMetadata/), but unrelated to this PR.

Were you able to run any live tests, @andrewgaul? Code changes themselves look good to me...

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54692325

Re: [jclouds] JCLOUDS-706: Upgrade Azure API to 2012-02-12 (#508)

Posted by Andrew Gaul <no...@github.com>.
Addressing filesystem test failure in #510.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/508#issuecomment-54694011