Addressing review comments

Author: kumar-shivam-ranjan

ads/aqua/app.py

@@ -433,7 +433,7 @@ def get_container_image(self, container_type: str = None) -> str:
         )
         return container_image
 
-    @cached(cache=TTLCache(maxsize=20, ttl=timedelta(hours=5), timer=datetime.now))
+    @cached(cache=TTLCache(maxsize=20, ttl=timedelta(minutes=30), timer=datetime.now))
     def list_service_containers(self) -> List[ContainerSummary]:
         """
         List containers from containers.conf in OCI Datascience control plane

ads/aqua/extension/common_handler.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python
-# Copyright (c) 2024 Oracle and/or its affiliates.
+# Copyright (c) 2025 Oracle and/or its affiliates.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/
 
 
@@ -12,14 +12,12 @@
 from tornado.web import HTTPError
 
 from ads.aqua.common.decorator import handle_exceptions
-from ads.aqua.common.errors import AquaResourceAccessError, AquaRuntimeError
+from ads.aqua.common.errors import AquaRuntimeError
 from ads.aqua.common.utils import (
     get_huggingface_login_timeout,
-    known_realm,
 )
 from ads.aqua.extension.base_handler import AquaAPIhandler
 from ads.aqua.extension.errors import Errors
-from ads.aqua.extension.utils import ui_compatability_check
 
 
 class ADSVersionHandler(AquaAPIhandler):
@@ -50,14 +48,7 @@ def get(self):
             AquaResourceAccessError: raised when aqua is not accessible in the given session/region.
 
         """
-        if ui_compatability_check():
-            return self.finish({"status": "ok"})
-        elif known_realm():
-            return self.finish({"status": "compatible"})
-        else:
-            raise AquaResourceAccessError(
-                "The AI Quick actions extension is not compatible in the given region."
-            )
+        return self.finish({"status": "ok"})
 
 
 class NetworkStatusHandler(AquaAPIhandler):

tests/unitary/with_extras/aqua/test_common_handler.py

@@ -1,7 +1,7 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*--
 
-# Copyright (c) 2024 Oracle and/or its affiliates.
+# Copyright (c) 2025 Oracle and/or its affiliates.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/
 
 import os
@@ -56,60 +56,3 @@ def test_get_ok(self):
                     self.common_handler.request.path = "aqua/hello"
                     result = self.common_handler.get()
                     assert result["status"] == "ok"
-
-    def test_get_compatible_status(self):
-        """Test to check if compatible is returned when ODSC_MODEL_COMPARTMENT_OCID is not set
-        but CONDA_BUCKET_NS is one of the namespaces from the GA list."""
-        with patch.dict(
-            os.environ,
-            {"ODSC_MODEL_COMPARTMENT_OCID": "", "CONDA_BUCKET_NS": AQUA_GA_LIST[0]},
-        ):
-            reload(ads.config)
-            reload(ads.aqua)
-            reload(ads.aqua.extension.utils)
-            reload(ads.aqua.extension.common_handler)
-            with patch(
-                "ads.aqua.extension.base_handler.AquaAPIhandler.finish"
-            ) as mock_finish:
-                with patch(
-                    "ads.aqua.extension.utils.fetch_service_compartment"
-                ) as mock_fetch_service_compartment:
-                    mock_fetch_service_compartment.return_value = None
-                    mock_finish.side_effect = lambda x: x
-                    self.common_handler.request.path = "aqua/hello"
-                    result = self.common_handler.get()
-                    assert result["status"] == "compatible"
-
-    def test_raise_not_compatible_error(self):
-        """Test to check if error is returned when ODSC_MODEL_COMPARTMENT_OCID is not set
-        and CONDA_BUCKET_NS is not one of the namespaces from the GA list."""
-        with patch.dict(
-            os.environ,
-            {"ODSC_MODEL_COMPARTMENT_OCID": "", "CONDA_BUCKET_NS": "test-namespace"},
-        ):
-            reload(ads.config)
-            reload(ads.aqua)
-            reload(ads.aqua.extension.utils)
-            reload(ads.aqua.extension.common_handler)
-            with patch(
-                "ads.aqua.extension.base_handler.AquaAPIhandler.finish"
-            ) as mock_finish:
-                with patch(
-                    "ads.aqua.extension.utils.fetch_service_compartment"
-                ) as mock_fetch_service_compartment:
-                    mock_fetch_service_compartment.return_value = None
-                    mock_finish.side_effect = lambda x: x
-                    self.common_handler.write_error = MagicMock()
-                    self.common_handler.request.path = "aqua/hello"
-                    self.common_handler.get()
-
-                    assert self.common_handler.write_error.call_args[1].get(
-                        "reason"
-                    ) == (
-                        "The AI Quick actions extension is not "
-                        "compatible in the given region."
-                    ), "Incorrect error message."
-                    assert (
-                        self.common_handler.write_error.call_args[1].get("status_code")
-                        == 404
-                    ), "Incorrect status code."