LBaaS scheduler should not use hardcoded tenant name

This fix removes the hardcoded 'admin' tenant name from lbaas scheduler and reads the tenant from config. It also fixes the default admin_tenant_name in svc_monitor.py to 'admin'. Closes-Bug: #1543296 Change-Id: If5e558e27ef6fd81d81cf56a317f39dbac2854ce
Juniper · Feb 10, 2016 · b2b73ba · b2b73ba
1 parent 8e2589e
commit b2b73ba
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/config/svc-monitor/svc_monitor/scheduler/vrouter_scheduler.py b/src/config/svc-monitor/svc_monitor/scheduler/vrouter_scheduler.py
@@ -57,7 +57,8 @@ def _get_az_vrouter_list(self):
         if not self._args.netns_availability_zone:
             return None
         az_list = self._nc.oper('availability_zones', 'list',
-            'admin', detailed=True)
+                                self._args.admin_tenant_name,
+                                detailed=True)
         az_vr_list = []
         for az in az_list:
             if self._args.netns_availability_zone not in str(az):

diff --git a/src/config/svc-monitor/svc_monitor/svc_monitor.py b/src/config/svc-monitor/svc_monitor/svc_monitor.py
@@ -911,7 +911,7 @@ def parse_args(args_str):
         'auth_insecure': True,
         'admin_user': 'user1',
         'admin_password': 'password1',
-        'admin_tenant_name': 'default-domain'
+        'admin_tenant_name': 'admin'
     }
     schedops = {
         'si_netns_scheduler_driver':