Updated pr.

Author: lu-ohai

ads/common/auth.py

@@ -517,17 +517,12 @@ def create_signer(self) -> Dict:
         """
         if self.oci_config:
             configuration = ads.telemetry.update_oci_client_config(self.oci_config)
+        elif self.signer_kwargs:
+            configuration = ads.telemetry.update_oci_client_config(self.signer_kwargs)
         else:
-            try:
-                configuration = ads.telemetry.update_oci_client_config(
-                    oci.config.from_file(self.oci_config_location, self.oci_key_profile)
-                )
-            except:
-                if not os.path.exists(os.path.expanduser(self.oci_config_location)):
-                    logger.info(f"Failed to get config from folder {self.oci_config_location}. Using 'signer_kwargs' instead.")
-                    configuration = ads.telemetry.update_oci_client_config(self.signer_kwargs)
-                else:
-                    raise
+            configuration = ads.telemetry.update_oci_client_config(
+                oci.config.from_file(self.oci_config_location, self.oci_key_profile)
+            )
 
         logger.info(f"Using 'api_key' authentication.")
         return {

tests/unitary/default_setup/auth/test_auth.py

@@ -114,6 +114,7 @@ def test_set_auth_with_kwargs(self, mock_load_private_key):
         assert signer["config"]["key_content"] == "test_key_content"
         assert "additional_user_agent" in signer["config"]
         assert signer["signer"] != None
+        set_auth()
 
 class TestOCIMixin(TestCase):
     def tearDown(self) -> None: