fix: skips the migration on proxy tab

splunk_add_on_ucc_framework/global_config_update.py

@@ -248,6 +248,10 @@ def _dump_with_migrated_tabs(global_config: GlobalConfig, path: str) -> None:
     for i, tab in enumerate(
         global_config.content.get("pages", {}).get("configuration", {}).get("tabs", [])
     ):
+        if tab.get("type") == "proxyTab":
+            # Collapsing the tab is not required for ProxyTab because we can't be certain
+            # that a particular tab is a Proxy tab, as we can with a Logging tab.
+            continue
         global_config.content["pages"]["configuration"]["tabs"][i] = _collapse_tab(tab)
 
     _dump(global_config.content, path, global_config._is_global_config_yaml)

splunk_add_on_ucc_framework/tabs/proxy_tab.py

@@ -114,14 +114,8 @@ def tab_type(self) -> Optional[str]:
     @classmethod
     def from_definition(cls, definition: Dict[str, Any]) -> Optional["Tab"]:
         """
-        This function checks if the definition either has type==ProxyTab, or if it is a normal tab (which later will
-        be converted), that satisfies the following conditions:
-        1. This dictionary has only 3 keys: name, title and entity (e.g. no keys like warnings)
-        2. It has only one entity
-        3. The entity is singleSelect and has the predefined log levels.
-
-        Note: Although it is possible to set custom levels, this function will omit other levels, as it would be harder
-        to determine whether the tab is indeed a logging tab.
+        This function checks if the definition has type == ProxyTab; if it does, it gets converted;
+        otherwise, it returns None.
         """
         if definition.get("type") == "proxyTab":
             entity = []