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

Review request for Ambari, Balázs Bence Sári, Dmytro Grinenko, Laszlo Puskas, 
Sumit Mohanty, and Sebastian Toader.


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


Repository: ambari


Description
-------

* Moved log out of the loop.
 * Fixed typo ("was" -> "were")


Diffs
-----

  ambari-agent/src/main/python/ambari_agent/Hardware.py 
0d431a389ee492fc8557ae04b56a6c178d5a8fdd 

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


Testing
-------

Verified that only the message with all ignored mounts is logged.  (Note: 
/vagrant is really mounted twice.)

```
INFO 2016-12-21 16:41:03,424 Hardware.py:168 - Some mount points were ignored: 
/, /dev/shm, /boot, /vagrant, /vagrant
```

Ran unit tests:

```
$ mvn -am -pl ambari-agent clean test
...
Ran 451 tests in 15.070s

OK
...
[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 
approved: 151 licence.
...
[INFO] Ambari Agent ...................................... SUCCESS [17.537s]
```


Thanks,

Attila Doroszlai

Reply via email to