-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parallel execution of the contrailctl command is not working #205
Comments
what error you see when you run them in parallel? |
From: Harish Kumar Also I already asked for input in those bugs. Siva, please update the details in the bug on how you tested (how to reproduce it), what errors you get (including ansible playbook output), |
Please try to following the steps to reproduce the issue without the Juju charms
|
I just tested it and this is working for me and I always does the same (simultaneously start all containers which actually start contrailctl on all those containers at same time). |
What happens when you issue contrailctl on same containers simultaneously from different shells.. |
hmmm it should succeed unless any conflicting or a command that need lock run by ansible - for example if ansible run multiple apt-get install at same time, one of them may fail/timeout. |
Say for example say I am deploying 3 controllers, 3 analytics node and 3 analyticsdb node in 3 machines (1 controller, 1 analytics and 1 analytics db per machine). In the Juju charm world the ipaddress of the controller,analytics and analyticsdb node itself will be available dynamically when I establish ‘relations’ between them. This means I will dynamically query and get the ipaddress and the write it into the. conf files. This means the configuration to take effect I will have multiple ‘contrailctl config’ sync commands executing simultaneously (this will be for different instances of the same charm as well as multiple instances different charms). It fails in this scenario. Juju provides the facility to scale up and scale down the number of units of a charm however the user wants and so we need this to be fixed for this to work.
The text was updated successfully, but these errors were encountered: