LGTM thank you for the updates.
https://codereview.appspot.com/172380043/diff/1/quickstart/app.py File quickstart/app.py (right):
https://codereview.appspot.com/172380043/diff/1/quickstart/app.py#newcode190 quickstart/app.py:190: candidates = jenv.get_value(env_name, 'state-servers') do you know how this list gets updated from Juju? In an HA situation does juju rewrite the jenv file?
https://codereview.appspot.com/172380043/diff/1/quickstart/manage.py File quickstart/manage.py (right):
https://codereview.appspot.com/172380043/diff/1/quickstart/manage.py#newcode520 quickstart/manage.py:520: already_bootstrapped = app.bootstrap( On 2014/11/12 14:48:25, bac wrote: > What is the scenario for api_url being None but it is already bootstrapped? > Someone deleted the jenv file out from under juju?
That has happened and there's been bugs/work around the scenario. It's a good one to watch for.
https://codereview.appspot.com/172380043/
« Back to merge proposal
LGTM thank you for the updates.
https:/ /codereview. appspot. com/172380043/ diff/1/ quickstart/ app.py
File quickstart/app.py (right):
https:/ /codereview. appspot. com/172380043/ diff/1/ quickstart/ app.py# newcode190 app.py: 190: candidates = jenv.get_ value(env_ name,
quickstart/
'state-servers')
do you know how this list gets updated from Juju? In an HA situation
does juju rewrite the jenv file?
https:/ /codereview. appspot. com/172380043/ diff/1/ quickstart/ manage. py manage. py (right):
File quickstart/
https:/ /codereview. appspot. com/172380043/ diff/1/ quickstart/ manage. py#newcode520 manage. py:520: already_ bootstrapped = app.bootstrap(
quickstart/
On 2014/11/12 14:48:25, bac wrote:
> What is the scenario for api_url being None but it is already
bootstrapped?
> Someone deleted the jenv file out from under juju?
That has happened and there's been bugs/work around the scenario. It's a
good one to watch for.
https:/ /codereview. appspot. com/172380043/