This fixes the directory structure of the Azure ARM provider. It contained a 
directory named `azurecompute.arm` instead of being a hierarchy, which caused 
the javadoc generation to fail.

I've also added the suggestions mentioned in the first pull request:

* Removed the tenant id property from the pom.xml.
* Updated the build status image to point to the jclouds Jenkins.
* Simplified the README to reflect the generic live tests instructions, to show 
how to run all them at once (this is what we expect and what we'll put in our 
CI system).
* Removed the `oauth.endpoint` property from the provider metadata. It can't be 
preconfigured as it requires the tenant id, so we'd better fail fast and report 
the missing property instead of silently try to connect to a wrong endpoint if 
users don't specify the property when creating the context.

@ritazh Comments are very welcome if any of the changes here are not 
appropriate!
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs/pull/254

-- Commit Summary --

  * Azure ARM fixes

-- File Changes --

    M azurecompute-arm/README.md (59)
    M azurecompute-arm/pom.xml (8)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/AzureComputeApi.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/AzureComputeProviderMetadata.java
 (4)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/AzureManagementApiMetadata.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/config/AzureComputeHttpApiModule.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/config/AzureComputeParserModule.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/config/AzureComputeProperties.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/ResourceGroup.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/features/JobApi.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/features/ResourceGroupApi.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/functions/ParseJobStatus.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/functions/URIParser.java
 (0)
    R 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/handlers/AzureComputeErrorHandler.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/AzureComputeProviderMetadataTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/JobApiMockTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/ResourceGroupApiLiveTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/ResourceGroupApiMockTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/functions/URIParserTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/internal/AbstractAzureComputeApiLiveTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/internal/AzureLiveTestUtils.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/internal/BaseAzureComputeApiLiveTest.java
 (0)
    R 
azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/internal/BaseAzureComputeApiMockTest.java
 (24)

-- Patch Links --

https://github.com/jclouds/jclouds-labs/pull/254.patch
https://github.com/jclouds/jclouds-labs/pull/254.diff

---
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/254

Reply via email to