Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/1277#issuecomment-170483420
@wilderrodrigues My questions and comments are very valid. If you don't see
any value in them too bad. Looks like you don't want to understand the point I
am trying to make. Here is some more data if you are really interested. I also
don't have time for the useless comments you are making.
There is a set of VR tests which validates network gc (check out
test/integration/component/test_routers.py). I ran it on latest master (without
any PRs) using a simulator setup and it worked as expected.
nosetests-2.7 --with-marvin --marvin-config=setup/dev/advanced.cfg
--with-xunit --xunit-file=out.xml
test/integration/component/test_routers.py:TestRouterServices -a
tags=advanced,required_hardware=false --zone=Sandbox-simulator
--hypervisor=simulator
Test advanced zone router services ... === TestName:
test_01_AdvancedZoneRouterServices | Status : SUCCESS ===
ok
Test network garbage collection ... === TestName:
test_02_NetworkGarbageCollection | Status : SUCCESS ===
ok
Test router start on VM deploy ... === TestName:
test_03_RouterStartOnVmDeploy | Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 3 tests in 150.061s
OK
---
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 [email protected] or file a JIRA ticket
with INFRA.
---