Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Jayush Luniya

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

Ship it!


Ship It!

- Jayush Luniya


On Sept. 11, 2015, 5:40 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38310/
> ---
> 
> (Updated Sept. 11, 2015, 5:40 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-13077
> https://issues.apache.org/jira/browse/AMBARI-13077
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> -
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/service_check.py
>  e633dcb 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_service_check.py 
> bb0ce90 
> 
> Diff: https://reviews.apache.org/r/38310/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Andrew Onischuk

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

Review request for Ambari and Mahadev Konar.


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


Repository: ambari


Description
---

Cluster:  \- please stop when not needed - label
ambari-heavyrolling-upg-deb7-9475-split5  
Artifacts: 

There may be some config issues this is new test. So please report if there
are some.


Diffs
-

  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/service_check.py
 e633dcb 
  ambari-server/src/test/python/stacks/2.1/FALCON/test_service_check.py bb0ce90 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Mahadev Konar

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

Ship it!


Ship It!

- Mahadev Konar


On Sept. 11, 2015, 5:40 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38310/
> ---
> 
> (Updated Sept. 11, 2015, 5:40 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-13077
> https://issues.apache.org/jira/browse/AMBARI-13077
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> -
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/service_check.py
>  e633dcb 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_service_check.py 
> bb0ce90 
> 
> Diff: https://reviews.apache.org/r/38310/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Mahadev Konar

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


Please make sure HDP 2.2 works fine with these changes.

- Mahadev Konar


On Sept. 11, 2015, 5:40 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38310/
> ---
> 
> (Updated Sept. 11, 2015, 5:40 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-13077
> https://issues.apache.org/jira/browse/AMBARI-13077
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> -
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/service_check.py
>  e633dcb 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_service_check.py 
> bb0ce90 
> 
> Diff: https://reviews.apache.org/r/38310/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Andrew Onischuk

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

(Updated Sept. 11, 2015, 6:44 p.m.)


Review request for Ambari and Mahadev Konar.


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


Repository: ambari


Description
---

-


Diffs (updated)
-

  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
 38c806b 
  
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
 793a44b 
  
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
 2cee73e 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_client.py d2fa8ab 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py c4ff3dc 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py 
59f8a8f 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py 
1711f69 
  ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_client.py 4fcfb33 
  ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 3341511 
  ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py 17c6137 
  ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py 11570ed 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Andrew Onischuk

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


Uploaded a new patch, because I was told that some other SC failed. Included 
that into this one.
Tested on 2.2 and 2.3

- Andrew Onischuk


On Sept. 11, 2015, 6:44 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38310/
> ---
> 
> (Updated Sept. 11, 2015, 6:44 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-13077
> https://issues.apache.org/jira/browse/AMBARI-13077
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> -
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
>  38c806b 
>   
> ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
>  793a44b 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
>  2cee73e 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_client.py 
> d2fa8ab 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py 
> c4ff3dc 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py 
> 59f8a8f 
>   
> ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py 
> 1711f69 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_client.py 
> 4fcfb33 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 
> 3341511 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py 
> 17c6137 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py 
> 11570ed 
> 
> Diff: https://reviews.apache.org/r/38310/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Andrew Onischuk

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


Also found some commoness between those failure. Seems it was only missing 
permissions for -env.sh files. Seems like they wasn't packaged with the 
package. That's why when they were created with umask 027 with Ambari, the 
issue happened.

- Andrew Onischuk


On Sept. 11, 2015, 6:44 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38310/
> ---
> 
> (Updated Sept. 11, 2015, 6:44 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-13077
> https://issues.apache.org/jira/browse/AMBARI-13077
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> -
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
>  38c806b 
>   
> ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
>  793a44b 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
>  2cee73e 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_client.py 
> d2fa8ab 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py 
> c4ff3dc 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py 
> 59f8a8f 
>   
> ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py 
> 1711f69 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_client.py 
> 4fcfb33 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 
> 3341511 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py 
> 17c6137 
>   ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py 
> 11570ed 
> 
> Diff: https://reviews.apache.org/r/38310/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 38310: RU: Debian7 fails with multiple service check issues

2015-09-11 Thread Andrew Onischuk

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

(Updated Sept. 11, 2015, 5:40 p.m.)


Review request for Ambari and Mahadev Konar.


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


Repository: ambari


Description (updated)
---

-


Diffs
-

  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/service_check.py
 e633dcb 
  ambari-server/src/test/python/stacks/2.1/FALCON/test_service_check.py bb0ce90 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk