[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9453?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15429967#comment-15429967
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9453:
--------------------------------------------

Github user jburwell commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1639#discussion_r75610946
  
    --- Diff: tools/marvin/marvin/lib/common.py ---
    @@ -334,13 +335,57 @@ def get_template(
             return FAILED
     
         for template in list_templatesout:
    -        if template.isready and template.templatetype == template_type:
    +        if template.isready and template.templatetype == template_type and 
template.name.startswith('CentOS'):
                 return template
         '''
         Return default first template, if no template matched
         '''
         return list_templatesout[0]
     
    +def get_template_macchinina(
    --- End diff --
    
    This function and the ``get_template`` function seem very similar.  The 
only difference appears to be that  ``get_template`` function is checking for 
``CentOS`` and ``get_template_macchinia`` is checking for ``macchinia``.  
Please refactor these functions to remove the duplication.


> Optimizing Marvin
> -----------------
>
>                 Key: CLOUDSTACK-9453
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9453
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: marvin
>    Affects Versions: Future
>            Reporter: Abhinandan Prateek
>            Assignee: Abhinandan Prateek
>             Fix For: Future
>
>
> Currently running all Marvin tests can take upto 4 days. The tests are not 
> optimized for nested cloud setup where most of the test automation runs. 
> There are some simple things that can be done to optimize the runs:
> 1. Have smaller default template: If we install macchinina template by 
> default and use it where there is no specific dependency on OS, then it will 
> result in speeding up many of Marvin tests.
> 2. Most of the tests have template names hard-coded. It will be a good idea 
> to allow some form of configuration so that test writers can use templates 
> that better suit their test scenario.
> 3. Some test timeouts are unnecessary long and a failure can be deducted much 
> early on instead of undergoing several long timeouts.
> 4. Ability to tune service offerings to better suit marvin environments.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to