fix: do not require calls of _reload endpoint (#1547)

**Issue number:** ADDON-77916

### PR Type

**What kind of change does this PR introduce?**
* [ ] Feature
* [x] Bug Fix
* [ ] Refactoring (no functional or API changes)
* [ ] Documentation Update
* [ ] Maintenance (dependency updates, CI, etc.)

## Summary

### Changes

This change uses the exposed interface to set `need_reload` from
`splunktaucclib` to False by default in order to prevent reloads of
selected endpoints.

### User experience

Improves the performance of the add-ons built using UCC and those that
are using `splunktaucclib` to not require reloads during API calls to
the endpoints.

## Checklist

If an item doesn't apply to your changes, leave it unchecked.

* [x] I have performed a self-review of this change according to the
[development
guidelines](https://splunk.github.io/addonfactory-ucc-generator/contributing/#development-guidelines)
* [x] Tests have been added/modified to cover the changes [(testing
doc)](https://splunk.github.io/addonfactory-ucc-generator/contributing/#build-and-test)
* [ ] Changes are documented
* [x] PR title and description follows the [contributing
principles](https://splunk.github.io/addonfactory-ucc-generator/contributing/#pull-requests)

---------

Co-authored-by: kkedziak <kkedziak@splunk.com>

Author: artemrys

splunk_add_on_ucc_framework/commands/rest_builder/endpoint/base.py

@@ -123,7 +123,7 @@ def generate_rh(self) -> str:
 
 
 class RestEndpointBuilder:
-    def __init__(self, name: Optional[str], namespace: str, **kwargs: str):
+    def __init__(self, name: Optional[str], namespace: str, **kwargs: Any):
         self._name = name
         self._namespace = namespace
         self._entities: List[RestEntityBuilder] = []
@@ -144,6 +144,7 @@ def __init__(self, name: Optional[str], namespace: str, **kwargs: str):
             self._rest_handler_name = f"{self._namespace}_rh_{self._name}"
         self._rest_handler_module = kwargs.get("rest_handler_module")
         self._rest_handler_class = kwargs.get("rest_handler_class")
+        self._need_reload = kwargs.get("need_reload", False)
 
     @property
     def name(self) -> str:
@@ -173,6 +174,10 @@ def rh_class(self) -> Optional[str]:
     def entities(self) -> List[RestEntityBuilder]:
         return self._entities
 
+    @property
+    def need_reload(self) -> bool:
+        return self._need_reload
+
     def add_entity(self, entity: RestEntityBuilder) -> None:
         self._entities.append(entity)
 

splunk_add_on_ucc_framework/commands/rest_builder/endpoint/multiple_model.py

@@ -57,6 +57,7 @@ class MultipleModelEndpointBuilder(RestEndpointBuilder):
     models=[
 {models}
     ],
+    need_reload={need_reload},
 )
 
 
@@ -81,4 +82,5 @@ def generate_rh(self) -> str:
             entities="\n".join(entities),
             models=indent(models_lines, 2),
             conf_name=self.conf_name,
+            need_reload=self.need_reload,
         )

splunk_add_on_ucc_framework/commands/rest_builder/endpoint/single_model.py

@@ -62,7 +62,8 @@ class SingleModelEndpointBuilder(RestEndpointBuilder):
 endpoint = SingleModel(
     '{conf_name}',
     model,
-    config_name='{config_name}'
+    config_name='{config_name}',
+    need_reload={need_reload},
 )
 
 
@@ -85,4 +86,5 @@ def generate_rh(self) -> str:
             entity=entity.generate_rh(),
             conf_name=self.conf_name,
             config_name=self._name,
+            need_reload=self.need_reload,
         )

splunk_add_on_ucc_framework/commands/rest_builder/global_config_builder_schema.py

@@ -84,6 +84,10 @@ def oauth_conf_file_names(self) -> List[str]:
     def endpoints(self) -> List[RestEndpointBuilder]:
         return list(self._endpoints.values())
 
+    @property
+    def need_reload(self) -> bool:
+        return False
+
     def _parse_builder_schema(self) -> None:
         self._builder_configs()
         self._builder_settings()
@@ -102,6 +106,7 @@ def _builder_configs(self) -> None:
                 rest_handler_class=config.get(
                     "restHandlerClass", REST_HANDLER_DEFAULT_CLASS
                 ),
+                need_reload=self.need_reload,
             )
             self._endpoints[name] = endpoint
             content = self._get_oauth_enitities(config["entity"])
@@ -140,6 +145,7 @@ def _builder_settings(self) -> None:
             namespace=self.global_config.namespace,
             rest_handler_module=REST_HANDLER_DEFAULT_MODULE,
             rest_handler_class=REST_HANDLER_DEFAULT_CLASS,
+            need_reload=self.need_reload,
         )
         self._endpoints["settings"] = endpoint
         for setting in self.global_config.settings:
@@ -173,6 +179,7 @@ def _builder_inputs(self) -> None:
                     rest_handler_name=rest_handler_name,
                     rest_handler_module=rest_handler_module,
                     rest_handler_class=rest_handler_class,
+                    need_reload=self.need_reload,
                 )
                 self._endpoints[name] = single_model_endpoint
                 content = self._get_oauth_enitities(input_item["entity"])

splunk_add_on_ucc_framework/install_python_libraries.py

@@ -28,7 +28,7 @@
 logger = logging.getLogger("ucc_gen")
 
 
-LIBS_REQUIRED_FOR_UI = {"splunktaucclib": "6.4.0"}
+LIBS_REQUIRED_FOR_UI = {"splunktaucclib": "6.6.0"}
 LIBS_REQUIRED_FOR_OAUTH = {"solnlib": "5.5.0"}
 
 

tests/testdata/expected_addons/expected_output_global_config_configuration/Splunk_TA_UCCExample/bin/splunk_ta_uccexample_rh_account.py

@@ -152,7 +152,8 @@
 endpoint = SingleModel(
     'splunk_ta_uccexample_account',
     model,
-    config_name='account'
+    config_name='account',
+    need_reload=False,
 )
 
 

tests/testdata/expected_addons/expected_output_global_config_configuration/Splunk_TA_UCCExample/bin/splunk_ta_uccexample_rh_settings.py

@@ -172,6 +172,7 @@
         model_logging, 
         model_custom_abc
     ],
+    need_reload=False,
 )
 
 

tests/testdata/expected_addons/expected_output_global_config_everything/Splunk_TA_UCCExample/bin/splunk_ta_uccexample_rh_account.py

@@ -239,7 +239,8 @@
 endpoint = SingleModel(
     'splunk_ta_uccexample_account',
     model,
-    config_name='account'
+    config_name='account',
+    need_reload=False,
 )
 
 

tests/testdata/expected_addons/expected_output_global_config_everything/Splunk_TA_UCCExample/bin/splunk_ta_uccexample_rh_settings.py

@@ -177,6 +177,7 @@
         model_logging, 
         model_custom_abc
     ],
+    need_reload=False,
 )
 
 

tests/testdata/test_addons/package_files_conflict_test/globalConfig.json

@@ -199,7 +199,7 @@
     "meta": {
         "name": "test_addon",
         "restRoot": "test_addon",
-        "version": "5.56.0+12f4cd8cd",
+        "version": "5.57.0+f11804ebf",
         "displayName": "This is my add-on",
         "schemaVersion": "0.0.9"
     }