Merge "Fixes error message formatting in Load Balancers panel"

This commit is contained in:
Jenkins
2013-03-13 00:04:08 +00:00
committed by Gerrit Code Review

View File

@@ -108,11 +108,10 @@ class AddPool(workflows.Workflow):
def handle(self, request, context): def handle(self, request, context):
try: try:
pool = api.lbaas.pool_create(request, **context) pool = api.lbaas.pool_create(request, **context)
context['name']
return True return True
except: except:
exceptions.handle(request, msg = self.format_status_message(self.failure_message)
self.failure_message) exceptions.handle(request, msg)
return False return False
@@ -241,8 +240,8 @@ class AddVip(workflows.Workflow):
api.lbaas.vip_create(request, **context) api.lbaas.vip_create(request, **context)
return True return True
except: except:
exceptions.handle(request, msg = self.format_status_message(self.failure_message)
self.failure_message) exceptions.handle(request, msg)
return False return False
@@ -330,10 +329,6 @@ class AddMember(workflows.Workflow):
success_url = "horizon:project:loadbalancers:index" success_url = "horizon:project:loadbalancers:index"
default_steps = (AddMemberStep,) default_steps = (AddMemberStep,)
def format_status_message(self, message):
member_id = self.context.get('member_id')
return message % member_id
def handle(self, request, context): def handle(self, request, context):
if context['members'] == []: if context['members'] == []:
self.failure_message = _('No instances available.%s') self.failure_message = _('No instances available.%s')
@@ -355,8 +350,7 @@ class AddMember(workflows.Workflow):
context['member_id'] = api.lbaas.member_create( context['member_id'] = api.lbaas.member_create(
request, **context).id request, **context).id
except: except:
exceptions.handle(request, exceptions.handle(request, _("Unable to add member."))
self.failure_message)
return False return False
return True return True
@@ -434,16 +428,11 @@ class AddMonitor(workflows.Workflow):
success_url = "horizon:project:loadbalancers:index" success_url = "horizon:project:loadbalancers:index"
default_steps = (AddMonitorStep,) default_steps = (AddMonitorStep,)
def format_status_message(self, message):
monitor_id = self.context.get('monitor_id')
return message % monitor_id
def handle(self, request, context): def handle(self, request, context):
try: try:
context['monitor_id'] = api.lbaas.pool_health_monitor_create( context['monitor_id'] = api.lbaas.pool_health_monitor_create(
request, **context).get('id') request, **context).get('id')
return True return True
except: except:
exceptions.handle(request, exceptions.handle(request, _("Unable to add monitor."))
self.failure_message)
return False return False