You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Attila Doroszlai <ad...@hortonworks.com> on 2017/07/12 09:31:15 UTC

Review Request 60797: AMBARI-21446. Force-remove package does not work on CentOS 6 and SuSE 11

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

Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Sumit Mohanty, and Sid Wagle.


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


Repository: ambari


Description
-------

Command without path works fine.


Diffs
-----

  ambari-agent/src/test/python/resource_management/TestPackageResource.py bc1bfeb9b0d2c4a552919ab55c41836ef6ad11dd 
  ambari-common/src/main/python/resource_management/core/providers/package/yumrpm.py 064b504a4551de8c260705458febbd828416d499 
  ambari-common/src/main/python/resource_management/core/providers/package/zypper.py c1aab6034c60a4fc6321b58eb1c895ec238dbb20 


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


Testing
-------

Manually tested on CentOS 6.4 and CentOS 7.3.

Unit tests:

```
$ mvn -am -pl ambari-agent clean test
...
Ran 456 tests in 27.326s

OK
...
[INFO] BUILD SUCCESS
```


Thanks,

Attila Doroszlai


Re: Review Request 60797: AMBARI-21446. Force-remove package does not work on CentOS 6 and SuSE 11

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


Ship it!




Ship It!

- Jonathan Hurley


On July 12, 2017, 5:31 a.m., Attila Doroszlai wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60797/
> -----------------------------------------------------------
> 
> (Updated July 12, 2017, 5:31 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-21446
>     https://issues.apache.org/jira/browse/AMBARI-21446
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Command without path works fine.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/test/python/resource_management/TestPackageResource.py bc1bfeb9b0d2c4a552919ab55c41836ef6ad11dd 
>   ambari-common/src/main/python/resource_management/core/providers/package/yumrpm.py 064b504a4551de8c260705458febbd828416d499 
>   ambari-common/src/main/python/resource_management/core/providers/package/zypper.py c1aab6034c60a4fc6321b58eb1c895ec238dbb20 
> 
> 
> Diff: https://reviews.apache.org/r/60797/diff/1/
> 
> 
> Testing
> -------
> 
> Manually tested on CentOS 6.4 and CentOS 7.3.
> 
> Unit tests:
> 
> ```
> $ mvn -am -pl ambari-agent clean test
> ...
> Ran 456 tests in 27.326s
> 
> OK
> ...
> [INFO] BUILD SUCCESS
> ```
> 
> 
> Thanks,
> 
> Attila Doroszlai
> 
>


Re: Review Request 60797: AMBARI-21446. Force-remove package does not work on CentOS 6 and SuSE 11

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60797/#review180310
-----------------------------------------------------------


Ship it!




Ship It!

- Alejandro Fernandez


On July 12, 2017, 9:31 a.m., Attila Doroszlai wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60797/
> -----------------------------------------------------------
> 
> (Updated July 12, 2017, 9:31 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-21446
>     https://issues.apache.org/jira/browse/AMBARI-21446
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Command without path works fine.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/test/python/resource_management/TestPackageResource.py bc1bfeb9b0d2c4a552919ab55c41836ef6ad11dd 
>   ambari-common/src/main/python/resource_management/core/providers/package/yumrpm.py 064b504a4551de8c260705458febbd828416d499 
>   ambari-common/src/main/python/resource_management/core/providers/package/zypper.py c1aab6034c60a4fc6321b58eb1c895ec238dbb20 
> 
> 
> Diff: https://reviews.apache.org/r/60797/diff/1/
> 
> 
> Testing
> -------
> 
> Manually tested on CentOS 6.4 and CentOS 7.3.
> 
> Unit tests:
> 
> ```
> $ mvn -am -pl ambari-agent clean test
> ...
> Ran 456 tests in 27.326s
> 
> OK
> ...
> [INFO] BUILD SUCCESS
> ```
> 
> 
> Thanks,
> 
> Attila Doroszlai
> 
>