Page History

...

•  Marvin SSH Library Issues : Current SSH library client of marvin, has issues EX: executing command returns error and output as part of same list, create connection with more delay, taking too much time some times, and other ssh issues. Fixed them.
•  Logging issues:  All related logs will be streamlined at one place for  the total run, It will help in debugging failures post run at one place. Now,  for the entire run, and for each test suite, we will see different logs viz., "run\results\exceptions+passfail " log all at one place. So, all logs separated by different category help resolve issues quickly. identify product issues vs test failures.  Each test suite logs will be a different folder with all these logs.
• Moving Services from tests suites: In phase1, we removed services dict from sanity tests and moved out. This will  help in segregating "tests code" from "tests data". Lot of places, services dictionary used as part of test code, makes it unreadable, not maintainable and it makes code a lot unclear, its passed as it is to base libraries and any changes in services dict breaks base libraries. EX: On a given test fixture, if we are to run all test suites for a given template, currently os type is hard coded under all test suites, that means we have to change under all test suites to get it working. This value of ostype was mentioned as part of services dict. Now, as such it is moved out, we can make change at one place and it will effect all test suites, other issues related to it were fixed.
• TestClient is now made a clear interface with marvin framework and test suites. EX: getHypervisorInfo, getzoneInfo, getparsedTestData etc are all accessed by tests using test client. So, any changes to the underlying things will still  be abstracted to test suites and they continue to work.
• There was an interface change for class "CSConnection" under CloudStackConnection.py , earlier management details were passed individually, now they were passed as it as a single structure. scheme variable was removed, this variable earlier signifies protocol(http\https), this can be obtained again from management details structure.
  
  

Phase2 Changes:

Planned to fix below issue\Enhancements.

...