Re: Review Request 48169: Ensure smokeuser HDFS folder exists before running MR, YARN, PIG, OOZIE service checks

2016-06-02 Thread Alejandro Fernandez 

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


Ship it!




Ship It!

- Alejandro Fernandez


On June 2, 2016, 2:38 p.m., Sandor Magyari wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48169/
> ---
> 
> (Updated June 2, 2016, 2:38 p.m.)
> 
> 
> Review request for Ambari, Laszlo Puskas, Oliver Szabo, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16956
> https://issues.apache.org/jira/browse/AMBARI-16956
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Modify service check scripts for YARN, MR, PIG, OOZIE, MAHOUT to ensure that 
> before running actual service checks using '/user/{smokeuser}' HDFS folder 
> (YARN, MR, PIG, OOZIE, MAHOUT) create this resource if doesn't already exists.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/params.py
>  60ebc42 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/service_check.py
>  dbb8079 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  b76bc89 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/service_check.py
>  8d14836 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params_linux.py
>  b7f798c 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py
>  395295f 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapred_service_check.py
>  d450701 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  150fedf 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
>  c802c41 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_service_check.py 
> 84ecf76 
>   ambari-server/src/test/python/stacks/2.0.6/PIG/test_pig_service_check.py 
> 100d78a 
>   
> ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_service_check.py
>  5f97efe 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_service_check.py 
> 320092d 
>   ambari-server/src/test/python/stacks/2.2/PIG/test_pig_service_check.py 
> 906c627 
>   
> ambari-server/src/test/python/stacks/2.3/MAHOUT/test_mahout_service_check.py 
> 7a3f010 
> 
> Diff: https://reviews.apache.org/r/48169/diff/
> 
> 
> Testing
> ---
> 
> Tested running service checks after removing Hdfs folder /uysers/ambari-qa.
> 
> --
> Ran 261 tests in 6.702s
> 
> OK
> --
> Total run:1052
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Sandor Magyari
> 
>

Re: Review Request 48169: Ensure smokeuser HDFS folder exists before running MR, YARN, PIG, OOZIE service checks

2016-06-02 Thread Oliver Szabo 

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


Ship it!




Ship It!

- Oliver Szabo


On June 2, 2016, 2:38 p.m., Sandor Magyari wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48169/
> ---
> 
> (Updated June 2, 2016, 2:38 p.m.)
> 
> 
> Review request for Ambari, Laszlo Puskas, Oliver Szabo, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16956
> https://issues.apache.org/jira/browse/AMBARI-16956
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Modify service check scripts for YARN, MR, PIG, OOZIE, MAHOUT to ensure that 
> before running actual service checks using '/user/{smokeuser}' HDFS folder 
> (YARN, MR, PIG, OOZIE, MAHOUT) create this resource if doesn't already exists.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/params.py
>  60ebc42 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/service_check.py
>  dbb8079 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  b76bc89 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/service_check.py
>  8d14836 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params_linux.py
>  b7f798c 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py
>  395295f 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapred_service_check.py
>  d450701 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  150fedf 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
>  c802c41 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_service_check.py 
> 84ecf76 
>   ambari-server/src/test/python/stacks/2.0.6/PIG/test_pig_service_check.py 
> 100d78a 
>   
> ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_service_check.py
>  5f97efe 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_service_check.py 
> 320092d 
>   ambari-server/src/test/python/stacks/2.2/PIG/test_pig_service_check.py 
> 906c627 
>   
> ambari-server/src/test/python/stacks/2.3/MAHOUT/test_mahout_service_check.py 
> 7a3f010 
> 
> Diff: https://reviews.apache.org/r/48169/diff/
> 
> 
> Testing
> ---
> 
> Tested running service checks after removing Hdfs folder /uysers/ambari-qa.
> 
> --
> Ran 261 tests in 6.702s
> 
> OK
> --
> Total run:1052
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Sandor Magyari
> 
>

Re: Review Request 48169: Ensure smokeuser HDFS folder exists before running MR, YARN, PIG, OOZIE service checks

2016-06-02 Thread Sebastian Toader 

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


Ship it!




Ship It!

- Sebastian Toader


On June 2, 2016, 4:38 p.m., Sandor Magyari wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48169/
> ---
> 
> (Updated June 2, 2016, 4:38 p.m.)
> 
> 
> Review request for Ambari, Laszlo Puskas, Oliver Szabo, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16956
> https://issues.apache.org/jira/browse/AMBARI-16956
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Modify service check scripts for YARN, MR, PIG, OOZIE, MAHOUT to ensure that 
> before running actual service checks using '/user/{smokeuser}' HDFS folder 
> (YARN, MR, PIG, OOZIE, MAHOUT) create this resource if doesn't already exists.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/params.py
>  60ebc42 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/service_check.py
>  dbb8079 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  b76bc89 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/service_check.py
>  8d14836 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params_linux.py
>  b7f798c 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py
>  395295f 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapred_service_check.py
>  d450701 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  150fedf 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
>  c802c41 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_service_check.py 
> 84ecf76 
>   ambari-server/src/test/python/stacks/2.0.6/PIG/test_pig_service_check.py 
> 100d78a 
>   
> ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_service_check.py
>  5f97efe 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_service_check.py 
> 320092d 
>   ambari-server/src/test/python/stacks/2.2/PIG/test_pig_service_check.py 
> 906c627 
>   
> ambari-server/src/test/python/stacks/2.3/MAHOUT/test_mahout_service_check.py 
> 7a3f010 
> 
> Diff: https://reviews.apache.org/r/48169/diff/
> 
> 
> Testing
> ---
> 
> Tested running service checks after removing Hdfs folder /uysers/ambari-qa.
> 
> --
> Ran 261 tests in 6.702s
> 
> OK
> --
> Total run:1052
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Sandor Magyari
> 
>

Re: Review Request 48169: Ensure smokeuser HDFS folder exists before running MR, YARN, PIG, OOZIE service checks

2016-06-02 Thread Laszlo Puskas 

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


Ship it!




Ship It!

- Laszlo Puskas


On June 2, 2016, 2:38 p.m., Sandor Magyari wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48169/
> ---
> 
> (Updated June 2, 2016, 2:38 p.m.)
> 
> 
> Review request for Ambari, Laszlo Puskas, Oliver Szabo, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16956
> https://issues.apache.org/jira/browse/AMBARI-16956
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Modify service check scripts for YARN, MR, PIG, OOZIE, MAHOUT to ensure that 
> before running actual service checks using '/user/{smokeuser}' HDFS folder 
> (YARN, MR, PIG, OOZIE, MAHOUT) create this resource if doesn't already exists.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/params.py
>  60ebc42 
>   
> ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/service_check.py
>  dbb8079 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  b76bc89 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/service_check.py
>  8d14836 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params_linux.py
>  b7f798c 
>   
> ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py
>  395295f 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapred_service_check.py
>  d450701 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  150fedf 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
>  c802c41 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_service_check.py 
> 84ecf76 
>   ambari-server/src/test/python/stacks/2.0.6/PIG/test_pig_service_check.py 
> 100d78a 
>   
> ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_service_check.py
>  5f97efe 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_service_check.py 
> 320092d 
>   ambari-server/src/test/python/stacks/2.2/PIG/test_pig_service_check.py 
> 906c627 
>   
> ambari-server/src/test/python/stacks/2.3/MAHOUT/test_mahout_service_check.py 
> 7a3f010 
> 
> Diff: https://reviews.apache.org/r/48169/diff/
> 
> 
> Testing
> ---
> 
> Tested running service checks after removing Hdfs folder /uysers/ambari-qa.
> 
> --
> Ran 261 tests in 6.702s
> 
> OK
> --
> Total run:1052
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Sandor Magyari
> 
>