search

ntt-sic / masakari

[UNMAINTAINED]
https://wiki.openstack.org/wiki/Masakari
Apache License 2.0
40 stars 24 forks source link

Source update #15

Closed sampathP closed 8 years ago

sampathP commented 8 years ago

This pull request intend to resolve #14. Summarised Testing log are in: http://paste.openstack.org/show/485796/ Complete log in masakari-controller in: http://paste.openstack.org/show/485797/

toshikazu0314 commented 8 years ago

looks good to me. Minor comment (not required to fix for merge): In the Summarised Testing log, some of Req sender descriptions are incorrect or blank.

muroi commented 8 years ago

LGTM, waiting others to comment.

sampathP commented 8 years ago

Thanks for the review. @toshikazu0314 Sorry for the incorrect/blanck Req sender in Testing log. I will fix the test codes and share it. It wont happen in next tests.