Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/908#issuecomment-145441458
@remibergsma
Hi,
I see that currently we invoke the main function of configure.py every time
we call the update_config when a new config to vr is pushed.The main function
tries to process all the components again, even the once which are not
modified. IMHO this is not a good way to update vr config. We should only
configure things which are changed newly. I think we need to fix this as it is
not scalable and redoing the whole config again might result in restarting one
or many services like you said is happening with conntrackd. However i think we
can fix the conntrackd issue temporarily by copying the conntrackd to some non
standard location and run conntrackd using this config file.Currently the
config file is copied to a standard location in /etc, doing this will make sure
that the conntrackd will not be there when this script runs for the first time.
I do not know of any tests for conntrackd case and for the default routes
case in the isolated networks. I think we need to test this manually.
---
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.
---
