]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Remove confusing comment and code for LBaaS
authorArata Notsu <notsu@virtualtech.jp>
Mon, 11 Nov 2013 11:33:43 +0000 (20:33 +0900)
committerArata Notsu <notsu@virtualtech.jp>
Mon, 11 Nov 2013 11:33:43 +0000 (20:33 +0900)
The comment "the driver is optional" is inconsistent with what actually
done. And "vif_driver = None" after raising an exception makes no sense.
So this patch removes these lines.

Change-Id: I7f3d5b6df61037307e952a478f47c59485601847

neutron/services/loadbalancer/drivers/haproxy/agent_manager.py

index 6d9cb1be5d85ba25bbf9f95c771c5946ec076afe..e5074cf92b0a20c7bac3d622a4e12a8db5ba45c9 100644 (file)
@@ -124,10 +124,8 @@ class LbaasAgentManager(periodic_task.PeriodicTasks):
         try:
             vif_driver = importutils.import_object(conf.interface_driver, conf)
         except ImportError:
-            # the driver is optional
             msg = _('Error importing interface driver: %s')
             raise SystemExit(msg % conf.interface_driver)
-            vif_driver = None
 
         try:
             self.driver = importutils.import_object(