appsembler / configuration

a simple, but flexible, way for anyone to stand up an instance of the edX platform that is fully configured and ready-to-go
GNU Affero General Public License v3.0
14 stars 13 forks source link

Wow that we have serial runs, execute cd command in first server pass #309

Closed melvinsoft closed 4 years ago

melvinsoft commented 4 years ago

@thraxil I know you changed the delegate_to from 0 to 1 in the following PR.

https://github.com/appsembler/configuration/pull/207/files

but now that conflict with the serial: 1 directive, because on fresh servers, it'll try to run an edxapp command in server 1 while is still empty and clean.

bryanlandia commented 4 years ago

@melvinsoft I see you removed my self-request. It's pretty easy to undestand and I would approve but I agree better to have @thraxil approve.

melvinsoft commented 4 years ago

@thraxil I'm merging this because we really need to test on Staging and now it's down. We can review tomorrow together if we need to came up with a different solutoin.