Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/1197#issuecomment-163967401
  
    We used separate environment, but provided by the same infra.
    
    What I don't get is why the tests are being executed from 
```/data/git/cs1/cloudstack/test/integration/component/test_routers_network_ops.py```,
 whilst I moved them to ```smoke``` directory a couple of days ago.
    
    I'm running the same tests - for KVM - a some changes I'm about to create a 
PR for. Those changes are only tests related, so no code has changed. In 
addition, I have rebased with latest master before creating my branch and the 
result from the same test I get is:
    
    ```
    Test redundant router internals ... === TestName: 
test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
    ok
    ```
    
    I think you are running from the component directory, which does not 
contain the test but might contain the ```pyc``` file. Just a thought. 
    
    I don't see why this PR would break 1 test only. It happened with 2 people, 
but both of them were running tests that might not even exist anymore...?
    
    @ustcweizhou @remibergsma @DaanHoogland I give my LGTM for this. If we find 
out some weird stuff, we just revert it.
    
    Cheers,
    Wilder


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to