jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject [jclouds] JCLOUDS-717: Join Enterprise and OpenSource Chef (#874)
Date Fri, 30 Oct 2015 00:09:05 GMT
https://issues.apache.org/jira/browse/JCLOUDS-717

Joins the `enterprisechef` and `chef` providers. Since Chef 12, the whole Chef product was
open sourced and the hosted and on premise versions are exactly the same, removing the need
for having two different providers.

This PR joins the enterprise chef specific apis in the chef provider. Those APIs are marked
optional and jclouds will automatically return them or not automatically depending on the
version of the target Chef Server (Chef versions prior to Chef 12 won't have those APIs).

The `enterprisechef` provider is now removed, and the module and test structure has been simplified
as there is no longer inheritance between different provider classes.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * JCLOUDS-717: Join Enterprise and OpenSource Chef

-- File Changes --

    M apis/chef/src/main/clojure/org/jclouds/chef.clj (4)
    M apis/chef/src/main/java/org/jclouds/chef/ChefApi.java (11)
    M apis/chef/src/main/java/org/jclouds/chef/ChefApiMetadata.java (8)
    R apis/chef/src/main/java/org/jclouds/chef/binders/BindGroupToUpdateRequestJsonPayload.java
(4)
    R apis/chef/src/main/java/org/jclouds/chef/binders/GroupName.java (4)
    D apis/chef/src/main/java/org/jclouds/chef/config/BaseChefHttpApiModule.java (204)
    M apis/chef/src/main/java/org/jclouds/chef/config/ChefHttpApiModule.java (180)
    R apis/chef/src/main/java/org/jclouds/chef/domain/Group.java (2)
    R apis/chef/src/main/java/org/jclouds/chef/domain/User.java (2)
    R apis/chef/src/main/java/org/jclouds/chef/features/OrganizationApi.java (14)
    M apis/chef/src/main/java/org/jclouds/chef/filters/SignedHeaderAuth.java (2)
    M apis/chef/src/main/java/org/jclouds/chef/options/CreateClientOptions.java (2)
    M apis/chef/src/test/java/org/jclouds/chef/BaseChefApiExpectTest.java (1)
    A apis/chef/src/test/java/org/jclouds/chef/ChefApiDelegationTest.java (55)
    M apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java (3)
    M apis/chef/src/test/java/org/jclouds/chef/ChefApiLiveTest.java (522)
    R apis/chef/src/test/java/org/jclouds/chef/binders/BindGroupToUpdateRequestJsonPayloadTest.java
(4)
    R apis/chef/src/test/java/org/jclouds/chef/binders/GroupNameTest.java (4)
    R apis/chef/src/test/java/org/jclouds/chef/features/OrganizationApiExpectTest.java (88)
    R apis/chef/src/test/java/org/jclouds/chef/features/OrganizationApiLiveTest.java (68)
    M apis/chef/src/test/java/org/jclouds/chef/functions/ParseClientFromJsonTest.java (2)
    D apis/chef/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java (542)
    M apis/chef/src/test/java/org/jclouds/chef/internal/BaseChefLiveTest.java (16)
    D apis/chef/src/test/java/org/jclouds/chef/internal/BaseStubbedOhaiLiveTest.java (60)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/CleanupStaleNodesAndClientsImplLiveTest.java
(3)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/CreateNodeAndPopulateAutomaticAttributesImplLiveTest.java
(3)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/CreateNodeAndPopulateAutomaticAttributesImplTest.java
(6)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/DeleteAllApisAndNodesInListImplLiveTest.java
(3)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/ListCookbookVersionsInEnvironmentImplLiveTest.java
(7)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/ListNodesImplLiveTest.java
(13)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/ListNodesInEnvironmentImplLiveTest.java
(13)
    M apis/chef/src/test/java/org/jclouds/chef/strategy/internal/UpdateAutomaticAttributesOnNodeImplLiveTest.java
(3)
    R apis/chef/src/test/resources/group-update.json (0)
    R apis/chef/src/test/resources/group.json (0)
    R apis/chef/src/test/resources/groups.json (0)
    R apis/chef/src/test/resources/user.json (2)
    D providers/enterprisechef/pom.xml (114)
    D providers/enterprisechef/src/main/java/org/jclouds/enterprisechef/EnterpriseChefApiMetadata.java
(81)
    D providers/enterprisechef/src/main/java/org/jclouds/enterprisechef/EnterpriseChefProviderMetadata.java
(79)
    D providers/enterprisechef/src/main/java/org/jclouds/enterprisechef/config/EnterpriseChefHttpApiModule.java
(36)
    D providers/enterprisechef/src/test/java/org/jclouds/enterprisechef/EnterpriseChefProviderMetadataTest.java
(31)
    M providers/pom.xml (1)

-- Patch Links --

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

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

Mime
View raw message