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



This is causing unit test failures. Please take a look.

-----------------------------------------------------------------------
Failed tests:
FAIL: test_hook_default_conf_select (test_after_install.TestHookAfterInstall)
----------------------------------------------------------------------
Traceback (most recent call last):
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-common/src/test/python/mock/mock.py",
 line 1199, in patched
    return func(*args, **keywargs)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py",
 line 109, in test_hook_default_conf_select
    sudo = True,)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/utils/RMFTestCase.py",
 line 278, in assertResourceCalled
    self.assertEquals(name, resource.name)
AssertionError: Tuples differ: ('cp', '-R', '-p', '/etc/pig/c... != ('cp', 
'-R', '-p', '/etc/kafka...

First differing element 3:
/etc/pig/conf
/etc/kafka/conf

- ('cp', '-R', '-p', '/etc/pig/conf', '/etc/pig/conf.backup')
?                          ^^^              ^^^

+ ('cp', '-R', '-p', '/etc/kafka/conf', '/etc/kafka/conf.backup')
?                          ^^^^^              ^^^^^


FAIL: test_hook_default_conf_select_suspended 
(test_after_install.TestHookAfterInstall)
----------------------------------------------------------------------
Traceback (most recent call last):
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-common/src/test/python/mock/mock.py",
 line 1199, in patched
    return func(*args, **keywargs)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py",
 line 695, in test_hook_default_conf_select_suspended
    sudo = True,)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/utils/RMFTestCase.py",
 line 278, in assertResourceCalled
    self.assertEquals(name, resource.name)
AssertionError: Tuples differ: ('cp', '-R', '-p', '/etc/pig/c... != ('cp', 
'-R', '-p', '/etc/kafka...

First differing element 3:
/etc/pig/conf
/etc/kafka/conf

- ('cp', '-R', '-p', '/etc/pig/conf', '/etc/pig/conf.backup')
?                          ^^^              ^^^

+ ('cp', '-R', '-p', '/etc/kafka/conf', '/etc/kafka/conf.backup')
?                          ^^^^^              ^^^^^


FAIL: test_hook_default_conf_select_with_error 
(test_after_install.TestHookAfterInstall)
----------------------------------------------------------------------
Traceback (most recent call last):
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-common/src/test/python/mock/mock.py",
 line 1199, in patched
    return func(*args, **keywargs)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py",
 line 366, in test_hook_default_conf_select_with_error
    sudo = True,)
  File 
"/Users/jluniya/release/trunk2/ambari/ambari-server/src/test/python/stacks/utils/RMFTestCase.py",
 line 278, in assertResourceCalled
    self.assertEquals(name, resource.name)
AssertionError: Tuples differ: ('cp', '-R', '-p', '/etc/pig/c... != ('cp', 
'-R', '-p', '/etc/kafka...

First differing element 3:
/etc/pig/conf
/etc/kafka/conf

- ('cp', '-R', '-p', '/etc/pig/conf', '/etc/pig/conf.backup')
?                          ^^^              ^^^

+ ('cp', '-R', '-p', '/etc/kafka/conf', '/etc/kafka/conf.backup')
?                          ^^^^^              ^^^^^


----------------------------------------------------------------------
Total run:1062
Total errors:0
Total failures:3
ERROR
INFO: AMBARI_SERVER_LIB is not set, using default /usr/lib/ambari-server
INFO: Return code from stack upgrade command, retcode = 0
StackAdvisor implementation for stack HDP1, version 2.0.6 was not found
Returning DefaultStackAdvisor implementation
StackAdvisor implementation for stack XYZ, version 1.0.0 was loaded
StackAdvisor implementation for stack XYZ, version 1.0.1 was loaded
Returning XYZ101StackAdvisor implementation
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:15.808s
[INFO] Finished at: Thu Jun 16 09:56:48 PDT 2016
[INFO] Final Memory: 60M/837M
[INFO] ------------------------------------------------------------------------

- Jayush Luniya


On June 15, 2016, 11:06 p.m., Jeff Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47941/
> -----------------------------------------------------------
> 
> (Updated June 15, 2016, 11:06 p.m.)
> 
> 
> Review request for Ambari, Jayush Luniya and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-16920
>     https://issues.apache.org/jira/browse/AMBARI-16920
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> This a followup ticket for add spark2 stack definition. There's serveral 
> issues:
> 1.  Spark2 thrift server can not started due to miss of 
> spark-thrift-fairscheduler.xml
> 2.  Miss of add spark2 cache file in copy_barball.py
> 3.  Miss the role_commnad_order of spark2
> 4.  conf_select is missign for spark2
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/conf_select.py
>  4eb0015 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py
>  286df8d 
>   
> ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/params.py
>  be99edd 
>   
> ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_service.py
>  c2385df 
>   ambari-server/src/main/resources/stacks/HDP/2.5/role_command_order.json 
> f6011b0 
> 
> Diff: https://reviews.apache.org/r/47941/diff/
> 
> 
> Testing
> -------
> 
> Manually verified.
> 
> 
> Thanks,
> 
> Jeff Zhang
> 
>

Reply via email to