You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Alejandro Fernandez <af...@hortonworks.com> on 2017/07/20 23:15:21 UTC

Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

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

Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.


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


Repository: ambari


Description
-------

Oozie Server restart failed due to this: Unable to copy ext-2.2.zip because it does not exist
during a cross-stack migration from BigInsights to HDP.

In BigInsights 4.2.5, the source path was /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip, but in BigInsights 4.2.0, the source path was /var/lib/oozie/ext-2.2.zip
So will append both to a list and copy the first one found.


Diffs
-----

  ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 142e962 
  ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py 719fb32 
  ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 426c36a 


Diff: https://reviews.apache.org/r/61012/diff/1/


Testing
-------

Python unit tests passed,

----------------------------------------------------------------------
Total run:1161
Total errors:0
Total failures:0
OK


Thanks,

Alejandro Fernandez


Re: Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

Posted by Jonathan Hurley <jh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61012/#review181116
-----------------------------------------------------------


Ship it!




Ship It!

- Jonathan Hurley


On July 20, 2017, 7:15 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61012/
> -----------------------------------------------------------
> 
> (Updated July 20, 2017, 7:15 p.m.)
> 
> 
> Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.
> 
> 
> Bugs: AMBARI-21463
>     https://issues.apache.org/jira/browse/AMBARI-21463
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Oozie Server restart failed due to this: Unable to copy ext-2.2.zip because it does not exist
> during a cross-stack migration from BigInsights to HDP.
> 
> In BigInsights 4.2.5, the source path was /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip, but in BigInsights 4.2.0, the source path was /var/lib/oozie/ext-2.2.zip
> So will append both to a list and copy the first one found.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 142e962 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py 719fb32 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 426c36a 
> 
> 
> Diff: https://reviews.apache.org/r/61012/diff/1/
> 
> 
> Testing
> -------
> 
> Python unit tests passed,
> 
> ----------------------------------------------------------------------
> Total run:1161
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>


Re: Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

Posted by Di Li <di...@ca.ibm.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61012/#review181113
-----------------------------------------------------------


Ship it!




Ship It!

- Di Li


On July 20, 2017, 11:15 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61012/
> -----------------------------------------------------------
> 
> (Updated July 20, 2017, 11:15 p.m.)
> 
> 
> Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.
> 
> 
> Bugs: AMBARI-21463
>     https://issues.apache.org/jira/browse/AMBARI-21463
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Oozie Server restart failed due to this: Unable to copy ext-2.2.zip because it does not exist
> during a cross-stack migration from BigInsights to HDP.
> 
> In BigInsights 4.2.5, the source path was /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip, but in BigInsights 4.2.0, the source path was /var/lib/oozie/ext-2.2.zip
> So will append both to a list and copy the first one found.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 142e962 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py 719fb32 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 426c36a 
> 
> 
> Diff: https://reviews.apache.org/r/61012/diff/1/
> 
> 
> Testing
> -------
> 
> Python unit tests passed,
> 
> ----------------------------------------------------------------------
> Total run:1161
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>


Re: Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61012/#review181082
-----------------------------------------------------------




ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
Lines 287 (patched)
<https://reviews.apache.org/r/61012/#comment256513>

    Earlier the code would log/fail if path did not exist. But now it will skip. So if there is no successful copy should we log that nothing got copied?


- Sumit Mohanty


On July 20, 2017, 11:15 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61012/
> -----------------------------------------------------------
> 
> (Updated July 20, 2017, 11:15 p.m.)
> 
> 
> Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.
> 
> 
> Bugs: AMBARI-21463
>     https://issues.apache.org/jira/browse/AMBARI-21463
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Oozie Server restart failed due to this: Unable to copy ext-2.2.zip because it does not exist
> during a cross-stack migration from BigInsights to HDP.
> 
> In BigInsights 4.2.5, the source path was /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip, but in BigInsights 4.2.0, the source path was /var/lib/oozie/ext-2.2.zip
> So will append both to a list and copy the first one found.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 142e962 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py 719fb32 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 426c36a 
> 
> 
> Diff: https://reviews.apache.org/r/61012/diff/1/
> 
> 
> Testing
> -------
> 
> Python unit tests passed,
> 
> ----------------------------------------------------------------------
> Total run:1161
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>


Re: Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

Posted by Sid Wagle <sw...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61012/#review181084
-----------------------------------------------------------


Ship it!




Ship It!

- Sid Wagle


On July 20, 2017, 11:15 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61012/
> -----------------------------------------------------------
> 
> (Updated July 20, 2017, 11:15 p.m.)
> 
> 
> Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.
> 
> 
> Bugs: AMBARI-21463
>     https://issues.apache.org/jira/browse/AMBARI-21463
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Oozie Server restart failed due to this: Unable to copy ext-2.2.zip because it does not exist
> during a cross-stack migration from BigInsights to HDP.
> 
> In BigInsights 4.2.5, the source path was /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip, but in BigInsights 4.2.0, the source path was /var/lib/oozie/ext-2.2.zip
> So will append both to a list and copy the first one found.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 142e962 
>   ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py 719fb32 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 426c36a 
> 
> 
> Diff: https://reviews.apache.org/r/61012/diff/1/
> 
> 
> Testing
> -------
> 
> Python unit tests passed,
> 
> ----------------------------------------------------------------------
> Total run:1161
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>