You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Eugene Chekanskiy <ec...@hortonworks.com> on 2014/03/04 19:12:40 UTC

Review Request 18739: Any artifact install should check if the artifact exists

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

Review request for Ambari and Andrew Onischuk.


Repository: ambari


Description
-------

Added check for package existent before execution of any yum or zypper execution.


Diffs
-----

  ambari-agent/src/main/python/resource_management/core/providers/package/yumrpm.py 4e1d34e 
  ambari-agent/src/main/python/resource_management/core/providers/package/zypper.py d495f83 
  ambari-agent/src/main/python/resource_management/core/shell.py d261043 
  ambari-agent/src/test/python/resource_management/TestPackageResource.py 45c0260 

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


Testing
-------

Tests changed accordingly to changes and additional test cases was added.


Thanks,

Eugene Chekanskiy


Re: Review Request 18739: Any artifact install should check if the artifact exists

Posted by Eugene Chekanskiy <ec...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18739/
-----------------------------------------------------------

(Updated March 4, 2014, 6:35 p.m.)


Review request for Ambari and Andrew Onischuk.


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


Repository: ambari


Description
-------

Added check for package existent before execution of any yum or zypper execution.


Diffs (updated)
-----

  ambari-agent/src/main/python/resource_management/core/providers/package/yumrpm.py 4e1d34e 
  ambari-agent/src/main/python/resource_management/core/providers/package/zypper.py d495f83 
  ambari-agent/src/main/python/resource_management/core/shell.py d261043 
  ambari-agent/src/test/python/resource_management/TestPackageResource.py 45c0260 

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


Testing
-------

Tests changed accordingly to changes and additional test cases was added.


Thanks,

Eugene Chekanskiy


Re: Review Request 18739: Any artifact install should check if the artifact exists

Posted by Eugene Chekanskiy <ec...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18739/
-----------------------------------------------------------

(Updated March 4, 2014, 6:12 p.m.)


Review request for Ambari and Andrew Onischuk.


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


Repository: ambari


Description
-------

Added check for package existent before execution of any yum or zypper execution.


Diffs
-----

  ambari-agent/src/main/python/resource_management/core/providers/package/yumrpm.py 4e1d34e 
  ambari-agent/src/main/python/resource_management/core/providers/package/zypper.py d495f83 
  ambari-agent/src/main/python/resource_management/core/shell.py d261043 
  ambari-agent/src/test/python/resource_management/TestPackageResource.py 45c0260 

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


Testing
-------

Tests changed accordingly to changes and additional test cases was added.


Thanks,

Eugene Chekanskiy