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

Review request for cloudstack and Girish Shilamkar.


Repository: cloudstack-git


Description
-------

Added few enhancements to marvin.
Added new module for Logging Facility to marvin.
Added new Init facility to marvin.
Currently, there are multiple ways we are doing logging
Removed few unwanted logging cases.
Removed few command line switch options for logging.
The new way of logging now provides consolidated logging
under one single folder timestamped under the configured
folder path.
Removed parsing configuration from deploydata center
Added parsing,start logging and deploy as part of init
Added new error handling facility to catch unknown exception from
test cases. Currently, lot of scripts are throwing unknown
exceptions, add a handler to plugin to dump them to a file
    
ToDO:
Will do clean up in phase2 for this patch.
Separate deployDatacenter from creating test client.
Clean up configGenerator


Diffs
-----

  setup/dev/advanced.cfg 216314f 
  tools/marvin/marvin/cloudstackConnection.py 23f81fb 
  tools/marvin/marvin/codes.py f409c7c 
  tools/marvin/marvin/configGenerator.py 631e40f 
  tools/marvin/marvin/deployDataCenter.py 3f7eebb 
  tools/marvin/marvin/marvinInit.py PRE-CREATION 
  tools/marvin/marvin/marvinLog.py PRE-CREATION 
  tools/marvin/marvin/marvinPlugin.py 0e52bab 

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


Testing
-------

Tested locally. Ran smoke tests. As well, run deploydatacenter separately to 
check deployement is working fine.


Thanks,

Santhosh Edukulla

Reply via email to