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

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

Github user swill commented on the issue:

    https://github.com/apache/cloudstack/pull/1563
  
    This assumes that people know how to work with the test data and that they 
understand that "this random template needs X format and this other one will 
need Y format".  This is too much to ask without guidance, especially as we try 
to improve the automatability of the testing.  
    
    The regex idea is not a perfect solution, but would allow for us to handle 
more template types with a single entry.  Ideally we would be able to specify 
the expected values for this type of thing in an OS specific way so the tested 
(and verified) values for each OS can live in the test data. It would probably 
need a default that is used if the current OS does not have a match. 
    
    My main problem with this is that it will, by default,  break this test for 
everyone using bubble to automate their testing. Maybe we leave the old default 
test value in place, and people who are testing with other OSs can now override 
manually. 
    
    What do you think? Sorry, I don't mean to be combative here, but this is a 
good opertunity for us to review the way we handle OS specific test data. 


> [Automation ]modifying integration/smoke/test_routers_network_ops.py
> --------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-9389
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9389
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Automation
>            Reporter: prashant kumar mishra
>            Assignee: prashant kumar mishra
>
> Some test cases were failing due to invalid check_string , proposing 
> following modifications  
> 1-Moving check_string to test_data.py  
> 2-Since ping cmd reply is OS dependent ,for  default templates os type  
> CentOS changing check_string  from "3 packets received" to  "3 received" 
> [root@BPKxDmS ~]# ping -c 3 8.8.8.8
> PING 8.8.8.8 (8.8.8.8) 56(84) bytes of data.
> 64 bytes from 8.8.8.8: icmp_seq=1 ttl=52 time=16.9 ms
> 64 bytes from 8.8.8.8: icmp_seq=2 ttl=52 time=16.7 ms
> 64 bytes from 8.8.8.8: icmp_seq=3 ttl=52 time=17.0 ms
> --- 8.8.8.8 ping statistics ---
> 3 packets transmitted, 3 received, 0% packet loss, time 2020ms
> rtt min/avg/max/mdev = 16.720/16.896/17.015/0.196 ms



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

Reply via email to