cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Edukulla" <santhosh.eduku...@citrix.com>
Subject Re: Review Request 16127: Fixed CLOUDSTACK-5411
Date Mon, 09 Dec 2013 11:17:22 GMT

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

(Updated Dec. 9, 2013, 11:17 a.m.)


Review request for cloudstack and Girish Shilamkar.


Changes
-------

Added correct patch


Bugs: CLOUDSTACK-5411
    https://issues.apache.org/jira/browse/CLOUDSTACK-5411


Repository: cloudstack-git


Description
-------

1. Currently, there was an issue with logging formatter under marvin. Proper format messages
are not getting printed. Fixed that.
    
2. There were few references in existing cfg files for earlier used logger node. Removed them
and added new logger node as part of pending clean up.
    
3. Added TC information started, flow and the result accordingly to runlog. This will simplify
to see the test case starting , sequence of steps and final result including timestamps for
test c
    
4. Added few changes to dump the exception throwing tc's to the explicit exception file.


Diffs (updated)
-----

  setup/dev/advancedsg.cfg c6a1f59 
  setup/dev/basic.cfg e91c87a 
  tools/devcloud/devcloud.cfg e209b55 
  tools/marvin/marvin/codes.py 8f0f88d 
  tools/marvin/marvin/config/setup.cfg 216314f 
  tools/marvin/marvin/marvinInit.py d822a2b 
  tools/marvin/marvin/marvinLog.py ca4d600 
  tools/marvin/marvin/marvinPlugin.py f5b7a88 

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


Testing
-------

Tested running basic sanity post changes.


Thanks,

Santhosh Edukulla


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message