Guard calls that modify the CLB with try/except.
This commit is contained in:
parent
f97c3cb8b6
commit
b76dff4411
1 changed files with 12 additions and 3 deletions
|
@ -154,12 +154,18 @@ def cloud_load_balancer_ssl(module, loadbalancer, state, enabled, private_key,
|
||||||
needs_change = True
|
needs_change = True
|
||||||
|
|
||||||
if needs_change:
|
if needs_change:
|
||||||
balancer.add_ssl_termination(**ssl_attrs)
|
try:
|
||||||
|
balancer.add_ssl_termination(**ssl_attrs)
|
||||||
|
except pyrax.exceptions.PyraxException, e:
|
||||||
|
module.fail_json(msg='%s' % e.message)
|
||||||
changed = True
|
changed = True
|
||||||
elif state == 'absent':
|
elif state == 'absent':
|
||||||
# Remove SSL termination if it's already configured.
|
# Remove SSL termination if it's already configured.
|
||||||
if existing_ssl:
|
if existing_ssl:
|
||||||
balancer.delete_ssl_termination()
|
try:
|
||||||
|
balancer.delete_ssl_termination()
|
||||||
|
except pyrax.exceptions.PyraxException, e:
|
||||||
|
module.fail_json(msg='%s' % e.message)
|
||||||
changed = True
|
changed = True
|
||||||
|
|
||||||
if https_redirect is not None and balancer.httpsRedirect != https_redirect:
|
if https_redirect is not None and balancer.httpsRedirect != https_redirect:
|
||||||
|
@ -168,7 +174,10 @@ def cloud_load_balancer_ssl(module, loadbalancer, state, enabled, private_key,
|
||||||
# while the SSL termination changes above are being applied.
|
# while the SSL termination changes above are being applied.
|
||||||
pyrax.utils.wait_for_build(balancer, interval=5, attempts=attempts)
|
pyrax.utils.wait_for_build(balancer, interval=5, attempts=attempts)
|
||||||
|
|
||||||
balancer.update(httpsRedirect=https_redirect)
|
try:
|
||||||
|
balancer.update(httpsRedirect=https_redirect)
|
||||||
|
except pyrax.exceptions.PyraxException, e:
|
||||||
|
module.fail_json(msg='%s' % e.message)
|
||||||
changed = True
|
changed = True
|
||||||
|
|
||||||
if changed and wait:
|
if changed and wait:
|
||||||
|
|
Loading…
Reference in a new issue