Config admin clients as default

Change-Id: Ib4dc349ab7ed4193189ed6bc2bae699d51064998

8 files changed, 16 insertions, 15 deletions

diff --git a/devstack/plugin.sh b/devstack/plugin.sh
index 7529a057..12b63304 100644
--- a/devstack/plugin.sh
+++ b/devstack/plugin.sh
@@ -237,11 +237,6 @@ function configure_trove {
     iniset $TROVE_CONF DEFAULT taskmanager_manager trove.taskmanager.manager.Manager
     iniset $TROVE_CONF DEFAULT default_datastore $TROVE_DATASTORE_TYPE
 
-    iniset $TROVE_CONF DEFAULT remote_nova_client trove.common.clients_admin.nova_client_trove_admin
-    iniset $TROVE_CONF DEFAULT remote_cinder_client trove.common.clients_admin.cinder_client_trove_admin
-    iniset $TROVE_CONF DEFAULT remote_neutron_client trove.common.clients_admin.neutron_client_trove_admin
-    iniset $TROVE_CONF DEFAULT remote_glance_client trove.common.clients_admin.glance_client_trove_admin
-
     iniset $TROVE_CONF cassandra tcp_ports 7000,7001,7199,9042,9160
     iniset $TROVE_CONF couchbase tcp_ports 8091,8092,4369,11209-11211,21100-21199
     iniset $TROVE_CONF couchdb tcp_ports 5984
@@ -275,11 +270,6 @@ function configure_trove {
     iniset $TROVE_GUESTAGENT_CONF service_credentials region_name $REGION_NAME
     iniset $TROVE_GUESTAGENT_CONF service_credentials auth_url $TROVE_AUTH_ENDPOINT
 
-    iniset $TROVE_GUESTAGENT_CONF DEFAULT remote_nova_client trove.common.clients_admin.nova_client_trove_admin
-    iniset $TROVE_GUESTAGENT_CONF DEFAULT remote_cinder_client trove.common.clients_admin.cinder_client_trove_admin
-    iniset $TROVE_GUESTAGENT_CONF DEFAULT remote_neutron_client trove.common.clients_admin.neutron_client_trove_admin
-    iniset $TROVE_GUESTAGENT_CONF DEFAULT remote_glance_client trove.common.clients_admin.glance_client_trove_admin
-
     # 1. To avoid 'Connection timed out' error of sudo command inside the guest agent
     # 2. Config the controller IP address used by guest-agent to download Trove code during initialization (only valid for dev_mode=true).
     common_cloudinit=/etc/trove/cloudinit/common.cloudinit
diff --git a/releasenotes/notes/ussuri-03-add-service-status-updated.yaml b/releasenotes/notes/ussuri-add-service-status-updated.yaml
index 7cb00d71..7cb00d71 100644
--- a/releasenotes/notes/ussuri-03-add-service-status-updated.yaml
+++ b/releasenotes/notes/ussuri-add-service-status-updated.yaml
diff --git a/releasenotes/notes/ussuri-admin-clients-a14514a835ae11ea.yaml b/releasenotes/notes/ussuri-admin-clients-a14514a835ae11ea.yaml
new file mode 100644
index 00000000..5d746df8
--- /dev/null
+++ b/releasenotes/notes/ussuri-admin-clients-a14514a835ae11ea.yaml
@@ -0,0 +1,11 @@
+---
+upgrade:
+  - |
+    Trove is now using admin clients by default to communicate with Nova,
+    Cinder, Neutron and Glance. Deployers want to stick to the old clients need
+    to explicitly config the following options:
+
+    * remote_nova_client
+    * remote_cinder_client
+    * remote_neutron_client
+    * remote_glance_client
diff --git a/releasenotes/notes/ussuri-02-database-instance-healthy.yaml b/releasenotes/notes/ussuri-database-instance-healthy.yaml
index 8ffedef5..8ffedef5 100644
--- a/releasenotes/notes/ussuri-02-database-instance-healthy.yaml
+++ b/releasenotes/notes/ussuri-database-instance-healthy.yaml
diff --git a/releasenotes/notes/ussuri-04-delete-datastoredad784e2345711ea.yaml b/releasenotes/notes/ussuri-delete-datastoredad784e2345711ea.yaml
index 4fe62519..4fe62519 100644
--- a/releasenotes/notes/ussuri-04-delete-datastoredad784e2345711ea.yaml
+++ b/releasenotes/notes/ussuri-delete-datastoredad784e2345711ea.yaml
diff --git a/releasenotes/notes/ussuri-01-service-credential-config.yaml b/releasenotes/notes/ussuri-service-credential-config.yaml
index d10a1aff..d10a1aff 100644
--- a/releasenotes/notes/ussuri-01-service-credential-config.yaml
+++ b/releasenotes/notes/ussuri-service-credential-config.yaml
diff --git a/trove/common/cfg.py b/trove/common/cfg.py
index 3a627cbe..37027bd9 100644
--- a/trove/common/cfg.py
+++ b/trove/common/cfg.py
@@ -334,13 +334,13 @@ common_opts = [
                default='trove.common.clients.guest_client',
                help='Client to send Guest Agent calls to.'),
     cfg.StrOpt('remote_nova_client',
-               default='trove.common.clients.nova_client',
+               default='trove.common.clients_admin.nova_client_trove_admin',
                help='Client to send Nova calls to.'),
     cfg.StrOpt('remote_neutron_client',
-               default='trove.common.clients.neutron_client',
+               default='trove.common.clients_admin.neutron_client_trove_admin',
                help='Client to send Neutron calls to.'),
     cfg.StrOpt('remote_cinder_client',
-               default='trove.common.clients.cinder_client',
+               default='trove.common.clients_admin.cinder_client_trove_admin',
                help='Client to send Cinder calls to.'),
     cfg.StrOpt('remote_swift_client',
                default='trove.common.clients.swift_client',
@@ -349,7 +349,7 @@ common_opts = [
                default='trove.common.trove_remote.trove_client',
                help='Client to send Trove calls to.'),
     cfg.StrOpt('remote_glance_client',
-               default='trove.common.clients.glance_client',
+               default='trove.common.clients_admin.glance_client_trove_admin',
                help='Client to send Glance calls to.'),
     cfg.StrOpt('exists_notification_transformer',
                help='Transformer for exists notifications.'),
diff --git a/trove/tests/unittests/taskmanager/test_models.py b/trove/tests/unittests/taskmanager/test_models.py
index 907eeab6..8d7b1915 100644
--- a/trove/tests/unittests/taskmanager/test_models.py
+++ b/trove/tests/unittests/taskmanager/test_models.py
@@ -395,7 +395,7 @@ class FreshInstanceTasksTest(BaseFreshInstanceTasksTest):
     @patch.object(taskmanager_models.FreshInstanceTasks, '_build_volume_info')
     @patch.object(taskmanager_models.FreshInstanceTasks, '_guest_prepare')
     @patch.object(template, 'SingleInstanceConfigTemplate')
-    @patch('trove.common.clients.neutron_client')
+    @patch('trove.common.clients_admin.neutron_client_trove_admin')
     def test_create_instance_with_mgmt_port(self,
                                             mock_neutron_client,
                                             mock_single_instance_template,