----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20114/#review39789 -----------------------------------------------------------
We have to add appropriate lines to ActionMetadata.java for FLUME_SERVICE_CHECK I was able to add FLUME using API calls below curl -i -X POST 'http://localhost:8080/api/v1/clusters/c1/services' --data '[{"ServiceInfo":{"service_name":"FLUME"}}]' curl -i -X POST -H 'X-Requested-By: X-Requested-By' http://localhost:8080/api/v1/clusters/c1/services/FLUME/components/FLUME_HANDLER curl -i -X POST -H 'X-Requested-By: X-Requested-By' http://localhost:8080/api/v1/clusters/c1/hosts/c6406.ambari.apache.org/host_components/FLUME_HANDLER curl -i -X PUT -H 'X-Requested-By: X-Requested-By' -d '{"ServiceInfo": {"state" : "INSTALLED"}}' http://localhost:8080/api/v1/clusters/c1/services/FLUME/ curl -i -X POST -H 'X-Requested-By: X-Requested-By' -d '{"RequestInfo":{"context":"FLUME2 Service Check","command":"FLUME_SERVICE_CHECK"},"Requests/resource_filters":[{"service_name":"FLUME"}]}' http://localhost:8080/api/v1/clusters/c1/requests Service check fails because of where is no security_enabled var in params.py ambari-server/src/main/java/org/apache/ambari/server/Role.java <https://reviews.apache.org/r/20114/#comment72382> public static final Role FLUME_SERVICE_CHECK = valueOf("FLUME_SERVICE_CHECK"); ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume_check.py <https://reviews.apache.org/r/20114/#comment72381> Replace with with FlumeServiceCheck().execute() ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/params.py <https://reviews.apache.org/r/20114/#comment72380> Define security_enabled var. Which is used in flume_check.py - Dmytro Sen On April 8, 2014, 3:19 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20114/ > ----------------------------------------------------------- > > (Updated April 8, 2014, 3:19 p.m.) > > > Review request for Ambari, Andriy Babiichuk, Dmytro Sen, Srimanth Gunturi, > and Sid Wagle. > > > Bugs: AMBARI-5378 > https://issues.apache.org/jira/browse/AMBARI-5378 > > > Repository: ambari > > > Description > ------- > > * Add Flume as a service in 2.0.6 (for 2.0/2.1 stacks) > * Remove the old Flume definition from 1.3.x > * NOTE: This patch is only to get the service work started and unblock the > UI. It doesn't have to be perfect, just pass some tests so we can continue > to work FE and BE in tandem. > > > Diffs > ----- > > ambari-server/src/main/java/org/apache/ambari/server/Role.java f0024a4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java > b81c024 > ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java > 3747319 > ambari-server/src/main/resources/ganglia_properties.json 8a488b6 > ambari-server/src/main/resources/role_command_order.json 84610ca > ambari-server/src/main/resources/scripts/install_flume.sh a6e59e8 > > ambari-server/src/main/resources/stacks/HDP/1.3.0/services/FLUME/configuration/global.xml > f1fa4de > > ambari-server/src/main/resources/stacks/HDP/1.3.0/services/FLUME/metainfo.xml > 87ff4b3 > > ambari-server/src/main/resources/stacks/HDP/1.3.2.GlusterFS/services/FLUME/configuration/global.xml > f1fa4de > > ambari-server/src/main/resources/stacks/HDP/1.3.2.GlusterFS/services/FLUME/metainfo.xml > 185f685 > ambari-server/src/main/resources/stacks/HDP/1.3.2/role_command_order.json > 84610ca > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/FLUME/configuration/global.xml > f1fa4de > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/FLUME/metainfo.xml > bebb54e > ambari-server/src/main/resources/stacks/HDP/1.3.3/role_command_order.json > 84610ca > > ambari-server/src/main/resources/stacks/HDP/2.0.6.GlusterFS/role_command_order.json > 550f885 > ambari-server/src/main/resources/stacks/HDP/2.0.6/role_command_order.json > 4eec50a > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/flume-conf.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/flume-log4j.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/global.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/metainfo.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume.py > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume_check.py > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume_handler.py > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/params.py > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/templates/flume.conf.j2 > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/templates/log4j.properties.j2 > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.0/role_command_order.json > 4eec50a > ambari-server/src/main/resources/stacks/HDP/2.1/role_command_order.json > ded1d23 > > ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java > 9ff575d > > ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProviderTest.java > 4e827ab > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractPropertyProviderTest.java > ff2d0d2 > > ambari-server/src/test/java/org/apache/ambari/server/metadata/RoleGraphTest.java > d8075f2 > ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py PRE-CREATION > ambari-server/src/test/python/stacks/2.0.6/configs/default.json 61dca49 > > ambari-server/src/test/resources/stacks/HDP/1.3.4/services/FLUME/metainfo.xml > bebb54e > ambari-server/src/test/resources/stacks/HDP/2.0.6/role_command_order.json > 550f885 > ambari-server/src/test/resources/stacks/HDP/2.0.7/role_command_order.json > 84610ca > ambari-server/src/test/resources/stacks/HDP/2.0.8/role_command_order.json > 84610ca > ambari-server/src/test/resources/stacks/HDP/2.1.1/role_command_order.json > 84610ca > > Diff: https://reviews.apache.org/r/20114/diff/ > > > Testing > ------- > > Running tests for stack:2.0.6 service:FLUME > test_configure_default (test_flume.TestFlumeHandler) ... ok > test_start_default (test_flume.TestFlumeHandler) ... ok > test_stop_default (test_flume.TestFlumeHandler) ... ok > > ---------------------------------------------------------------------- > Ran 3 tests in 0.038s > > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 14:20.067s > [INFO] Finished at: Mon Apr 07 21:02:19 EDT 2014 > [INFO] Final Memory: 20M/123M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > >
