You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2014/08/28 15:42:00 UTC

Review Request 25150: Need for a separate config file that comprises of cluster specific properties

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25150/
-----------------------------------------------------------

Review request for Ambari and Dmitro Lisnichenko.


Bugs: AMBARI-7056
    https://issues.apache.org/jira/browse/AMBARI-7056


Repository: ambari


Description
-------

Ambari agent presumes that hadoop-env.sh file will be always available and so
puts cluster specific properties like smokeuser and kerberos_domain in hadoop-
env.sh file. These properties are used in shared initialization file for
before-INSTALL and before-ANY hooks.

Ambari should manage a separate file (cluster-env file) that carries cluster
specific properties.

    
    
    
    <configuration>
        <property>
            <name>security_enabled</name>
            <value>false</value>
            <description>Hadoop Security</description>
        </property>
        <property>
            <name>kerberos_domain</name>
            <value>EXAMPLE.COM</value>
            <description>Kerberos realm.</description>
        </property>
        <property>
            <name>ignore_groupsusers_create</name>
            <value>false</value>
            <description>Whether to ignores failures on users and group creation</description>
        </property>
        <property>
            <name>smokeuser</name>
            <value>ambari-qa</value>
            <description>User executing service checks</description>
        </property>
        <property>
            <name>user_group</name>
            <value>hadoop</value>
            <description>Proxy user group.</description>
        </property>
    </configuration>
    
    


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java 261559a 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java 13fff1d 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackLevelConfigurationResourceDefinition.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackVersionResourceDefinition.java d641e4c 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java 56b866d 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java 822199c 
  ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java fb3ddc9 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java bee118e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java c6b2f9f 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationRequest.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackVersionResponse.java 84105fc 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java 71ddc8d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackVersionResourceProvider.java 9cef695 
  ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java b10b4bc 
  ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java de70620 
  ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java cc1a45e 
  ambari-server/src/main/resources/key_properties.json 06ebb61 
  ambari-server/src/main/resources/properties.json 785bd24 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml PRE-CREATION 
  ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java 5596a1e 
  ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java 3a62147 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractResourceProviderTest.java c1f4f5b 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProviderTest.java PRE-CREATION 

Diff: https://reviews.apache.org/r/25150/diff/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk


Re: Review Request 25150: Need for a separate config file that comprises of cluster specific properties

Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25150/#review51776
-----------------------------------------------------------

Ship it!


Ship It!

- Dmitro Lisnichenko


On Aug. 28, 2014, 2:39 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25150/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2014, 2:39 p.m.)
> 
> 
> Review request for Ambari and Dmitro Lisnichenko.
> 
> 
> Bugs: AMBARI-7056
>     https://issues.apache.org/jira/browse/AMBARI-7056
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Ambari agent presumes that hadoop-env.sh file will be always available and so
> puts cluster specific properties like smokeuser and kerberos_domain in hadoop-
> env.sh file. These properties are used in shared initialization file for
> before-INSTALL and before-ANY hooks.
> 
> Ambari should manage a separate file (cluster-env file) that carries cluster
> specific properties.
> 
>     
>     
>     
>     <configuration>
>         <property>
>             <name>security_enabled</name>
>             <value>false</value>
>             <description>Hadoop Security</description>
>         </property>
>         <property>
>             <name>kerberos_domain</name>
>             <value>EXAMPLE.COM</value>
>             <description>Kerberos realm.</description>
>         </property>
>         <property>
>             <name>ignore_groupsusers_create</name>
>             <value>false</value>
>             <description>Whether to ignores failures on users and group creation</description>
>         </property>
>         <property>
>             <name>smokeuser</name>
>             <value>ambari-qa</value>
>             <description>User executing service checks</description>
>         </property>
>         <property>
>             <name>user_group</name>
>             <value>hadoop</value>
>             <description>Proxy user group.</description>
>         </property>
>     </configuration>
>     
>     
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java 261559a 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java 13fff1d 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackLevelConfigurationResourceDefinition.java PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackVersionResourceDefinition.java d641e4c 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java 56b866d 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java 822199c 
>   ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java fb3ddc9 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java bee118e 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java c6b2f9f 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationRequest.java PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/StackVersionResponse.java 84105fc 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java 71ddc8d 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackVersionResourceProvider.java 9cef695 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java b10b4bc 
>   ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java de70620 
>   ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java cc1a45e 
>   ambari-server/src/main/resources/key_properties.json 06ebb61 
>   ambari-server/src/main/resources/properties.json 785bd24 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml PRE-CREATION 
>   ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java 5596a1e 
>   ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java 3a62147 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractResourceProviderTest.java c1f4f5b 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProviderTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/25150/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>


Re: Review Request 25150: Need for a separate config file that comprises of cluster specific properties

Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25150/
-----------------------------------------------------------

(Updated Aug. 28, 2014, 2:39 p.m.)


Review request for Ambari and Dmitro Lisnichenko.


Bugs: AMBARI-7056
    https://issues.apache.org/jira/browse/AMBARI-7056


Repository: ambari


Description
-------

Ambari agent presumes that hadoop-env.sh file will be always available and so
puts cluster specific properties like smokeuser and kerberos_domain in hadoop-
env.sh file. These properties are used in shared initialization file for
before-INSTALL and before-ANY hooks.

Ambari should manage a separate file (cluster-env file) that carries cluster
specific properties.

    
    
    
    <configuration>
        <property>
            <name>security_enabled</name>
            <value>false</value>
            <description>Hadoop Security</description>
        </property>
        <property>
            <name>kerberos_domain</name>
            <value>EXAMPLE.COM</value>
            <description>Kerberos realm.</description>
        </property>
        <property>
            <name>ignore_groupsusers_create</name>
            <value>false</value>
            <description>Whether to ignores failures on users and group creation</description>
        </property>
        <property>
            <name>smokeuser</name>
            <value>ambari-qa</value>
            <description>User executing service checks</description>
        </property>
        <property>
            <name>user_group</name>
            <value>hadoop</value>
            <description>Proxy user group.</description>
        </property>
    </configuration>
    
    


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java 261559a 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java 13fff1d 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackLevelConfigurationResourceDefinition.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackVersionResourceDefinition.java d641e4c 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java 56b866d 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java 822199c 
  ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java fb3ddc9 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java bee118e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java c6b2f9f 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationRequest.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackVersionResponse.java 84105fc 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java 71ddc8d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackVersionResourceProvider.java 9cef695 
  ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java b10b4bc 
  ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java de70620 
  ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java cc1a45e 
  ambari-server/src/main/resources/key_properties.json 06ebb61 
  ambari-server/src/main/resources/properties.json 785bd24 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml PRE-CREATION 
  ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java 5596a1e 
  ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java 3a62147 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractResourceProviderTest.java c1f4f5b 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProviderTest.java PRE-CREATION 

Diff: https://reviews.apache.org/r/25150/diff/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk