insightconnect-plugin-runtime 5.4.1__tar.gz → 5.4.2__tar.gz

{insightconnect-plugin-runtime-5.4.1/insightconnect_plugin_runtime.egg-info → insightconnect-plugin-runtime-5.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: insightconnect-plugin-runtime
-Version: 5.4.1
+Version: 5.4.2
 Summary: InsightConnect Plugin Runtime
 Home-page: https://github.com/rapid7/komand-plugin-sdk-python
 Author: Rapid7 Integrations Alliance
@@ -26,7 +26,6 @@ Requires-Dist: apispec-webframeworks==1.0.0
 Requires-Dist: blinker==1.7.0
 Requires-Dist: structlog==24.1.0
 Requires-Dist: python-json-logger==2.0.7
-Requires-Dist: APScheduler==3.10.4
 
 
 # InsightConnect Python Plugin Runtime ![Build Status](https://github.com/rapid7/komand-plugin-sdk-python/workflows/Continuous%20Integration/badge.svg)
@@ -186,7 +185,7 @@ Running a specific test file:
 |                   | Plugin  | Slim Plugin |
 |:------------------|:-------:|:-----------:|
 | Python Version    | 3.9.18  |   3.9.18    |
-| OS                | Alpine  |  Bookworm   |
+| OS                | Alpine  |  Bullseye   |
 | Package installer |   apk   |     apt     |
 | Shell             | /bin/sh |  /bin/bash  |
 | Image Size        | ~350MB  |   ~180MB    |
@@ -213,6 +212,7 @@ after cloning this repository.
 
 ## Changelog
 
+* 5.4.2 - Remove background scheduler to simplify when the server gets custom_config values | Revert to use `bullseye` for slim image.
 * 5.4.1 - Retry logic added to get values from external API for custom_config values.
 * 5.4.0 - Implementation of custom_config parameter for plugin tasks | Alpine image updated OpenSSL and expat | Use `bookworm` for slim image.
 * 5.3.2 - Updated OpenSSL in alpine image and core packages to latest versions.

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2}/README.md

@@ -156,7 +156,7 @@ Running a specific test file:
 |                   | Plugin  | Slim Plugin |
 |:------------------|:-------:|:-----------:|
 | Python Version    | 3.9.18  |   3.9.18    |
-| OS                | Alpine  |  Bookworm   |
+| OS                | Alpine  |  Bullseye   |
 | Package installer |   apk   |     apt     |
 | Shell             | /bin/sh |  /bin/bash  |
 | Image Size        | ~350MB  |   ~180MB    |
@@ -183,6 +183,7 @@ after cloning this repository.
 
 ## Changelog
 
+* 5.4.2 - Remove background scheduler to simplify when the server gets custom_config values | Revert to use `bullseye` for slim image.
 * 5.4.1 - Retry logic added to get values from external API for custom_config values.
 * 5.4.0 - Implementation of custom_config parameter for plugin tasks | Alpine image updated OpenSSL and expat | Use `bookworm` for slim image.
 * 5.3.2 - Updated OpenSSL in alpine image and core packages to latest versions.

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2}/insightconnect_plugin_runtime/server.py

@@ -6,8 +6,6 @@ import logging
 from apispec import APISpec
 from apispec.ext.marshmallow import MarshmallowPlugin
 from apispec_webframeworks.flask import FlaskPlugin
-from apscheduler.schedulers.background import BackgroundScheduler
-
 
 from flask import Flask, request_started, request
 import gunicorn.app.base
@@ -108,7 +106,7 @@ class PluginServer(gunicorn.app.base.BaseApplication):
         self.workers = workers
         self.threads = threads
         # initialise before reaching out to CPS for configured values
-        self.config_options, self.schedule_interval = {}, None
+        self.config_options = {}
         self.get_plugin_properties_from_cps()
         self.app, self.blueprints = self.create_flask_app()
 
@@ -195,7 +193,6 @@ class PluginServer(gunicorn.app.base.BaseApplication):
                     plugin_config = resp_json.get("plugins", {}).get(plugin, {})
 
                     self.config_options = plugin_config
-                    self.schedule_interval = resp_json.get("config", {}).get("interval", DEFAULT_SCHEDULE_INTERVAL_MINUTES)
                     self.logger.info("Plugin configuration successfully retrieved...")
                     break
                 except MissingSchema as missing_schema:
@@ -260,21 +257,6 @@ class PluginServer(gunicorn.app.base.BaseApplication):
                 blueprint, url_prefix=VERSION_MAPPING[blueprint.name]
             )
 
-    def register_scheduled_tasks(self):
-        # Once Flask server is up and running also start the get_plugin_configs
-        try:
-            if self.plugin.tasks and self.schedule_interval:
-                self.logger.info(f"Starting scheduled task(s) to run at interval of {self.schedule_interval} minutes...")
-                scheduler = BackgroundScheduler(daemon=True)
-                scheduler.add_job(self.get_plugin_properties_from_cps, 'interval', minutes=self.schedule_interval)
-                scheduler.start()
-            else:
-                reason = "No tasks found found within plugin," if self.schedule_interval else "No schedule defined,"
-                self.logger.info(f"{reason} not starting scheduled tasks...")
-        except Exception as exception:
-            self.logger.error("Unable to start up scheduler, plugin will not be refreshing these configuration values."
-                              f"Error={exception}", exc_info=True)
-
     @staticmethod
     def bind_request_details(sender: Flask, **extras) -> None:
         """
@@ -306,8 +288,6 @@ class PluginServer(gunicorn.app.base.BaseApplication):
             try:
                 self.register_blueprint()
                 self.register_api_spec()
-                if is_running_in_cloud():
-                    self.register_scheduled_tasks()
                 self.arbiter.run()
             except RuntimeError as e:
                 sys.stderr.write("\nError: %s\n" % e)

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2/insightconnect_plugin_runtime.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: insightconnect-plugin-runtime
-Version: 5.4.1
+Version: 5.4.2
 Summary: InsightConnect Plugin Runtime
 Home-page: https://github.com/rapid7/komand-plugin-sdk-python
 Author: Rapid7 Integrations Alliance
@@ -26,7 +26,6 @@ Requires-Dist: apispec-webframeworks==1.0.0
 Requires-Dist: blinker==1.7.0
 Requires-Dist: structlog==24.1.0
 Requires-Dist: python-json-logger==2.0.7
-Requires-Dist: APScheduler==3.10.4
 
 
 # InsightConnect Python Plugin Runtime ![Build Status](https://github.com/rapid7/komand-plugin-sdk-python/workflows/Continuous%20Integration/badge.svg)
@@ -186,7 +185,7 @@ Running a specific test file:
 |                   | Plugin  | Slim Plugin |
 |:------------------|:-------:|:-----------:|
 | Python Version    | 3.9.18  |   3.9.18    |
-| OS                | Alpine  |  Bookworm   |
+| OS                | Alpine  |  Bullseye   |
 | Package installer |   apk   |     apt     |
 | Shell             | /bin/sh |  /bin/bash  |
 | Image Size        | ~350MB  |   ~180MB    |
@@ -213,6 +212,7 @@ after cloning this repository.
 
 ## Changelog
 
+* 5.4.2 - Remove background scheduler to simplify when the server gets custom_config values | Revert to use `bullseye` for slim image.
 * 5.4.1 - Retry logic added to get values from external API for custom_config values.
 * 5.4.0 - Implementation of custom_config parameter for plugin tasks | Alpine image updated OpenSSL and expat | Use `bookworm` for slim image.
 * 5.3.2 - Updated OpenSSL in alpine image and core packages to latest versions.

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2}/insightconnect_plugin_runtime.egg-info/requires.txt

@@ -12,4 +12,3 @@ apispec-webframeworks==1.0.0
 blinker==1.7.0
 structlog==24.1.0
 python-json-logger==2.0.7
-APScheduler==3.10.4

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2}/setup.py

@@ -5,7 +5,7 @@ with open("README.md", "r") as fh:
 
 setup(
     name="insightconnect-plugin-runtime",
-    version="5.4.1",
+    version="5.4.2",
     description="InsightConnect Plugin Runtime",
     long_description=long_description,
     long_description_content_type="text/markdown",
@@ -27,8 +27,7 @@ setup(
         "apispec-webframeworks==1.0.0",
         "blinker==1.7.0",
         "structlog==24.1.0",
-        "python-json-logger==2.0.7",
-        "APScheduler==3.10.4"
+        "python-json-logger==2.0.7"
     ],
     tests_require=[
         "pytest",

{insightconnect-plugin-runtime-5.4.1 → insightconnect-plugin-runtime-5.4.2}/tests/unit/test_server_cloud_plugins.py

@@ -1,9 +1,9 @@
 from requests.exceptions import HTTPError, Timeout, TooManyRedirects
 from parameterized import parameterized
-from unittest import TestCase
+from unittest import TestCase, skip
 from unittest.mock import patch, MagicMock
 
-from insightconnect_plugin_runtime.server import PluginServer, DEFAULT_SCHEDULE_INTERVAL_MINUTES
+from insightconnect_plugin_runtime.server import PluginServer
 from tests.plugin.hello_world import KomandHelloWorld
 from .utils import MockResponse, Logger
 
@@ -18,26 +18,20 @@ class TestServerCloudPlugins(TestCase):
         self.plugin_name = self.plugin.name.lower().replace(" ", "_")
 
     @parameterized.expand([["Set cloud to false", False], ["Set cloud to true", True]])
-    @patch("insightconnect_plugin_runtime.server.PluginServer.register_scheduled_tasks")
     @patch("insightconnect_plugin_runtime.server.request_get")
-    def test_cloud_plugin_no_tasks_ignore_cps(self, _test_name, cloud, mocked_req, mocked_scheduler, mock_cloud, _run):
+    def test_cloud_plugin_no_tasks_ignore_cps(self, _test_name, cloud, mocked_req, mock_cloud, _run):
         mock_cloud.return_value = cloud  # Mock plugin running in cloud vs not
         self.plugin.tasks = None  # ensure still no tasks as other tests edit this and could fail before reverting
         plugin_server = PluginServer(self.plugin)  # this plugin has no tasks by default
 
         plugin_server.start()
         self.assertEqual(plugin_server.config_options, {})
-        self.assertEqual(plugin_server.schedule_interval, None)
-
-        # Depending on if we're running cloud or not this could be called but we only reach to CPS if we have tasks
-        self.assertEquals(mocked_scheduler.called, cloud)
 
         # Plugin server never calls out to CPS as either we are not running in cloud mode or have no tasks.
         self.assertFalse(mocked_req.called)
 
-    @patch("insightconnect_plugin_runtime.server.PluginServer.register_scheduled_tasks")
     @patch("insightconnect_plugin_runtime.server.request_get")
-    def test_cloud_plugin_calls_cps(self, mocked_req, mocked_scheduler, _mock_cloud, _run):
+    def test_cloud_plugin_calls_cps(self, mocked_req, _mock_cloud, _run):
         mocked_req.return_value = MockResponse({"plugins":{self.plugin_name: PLUGIN_VALUE_1, 'plugin': PLUGIN_VALUE_2},
                                                 "config": {"interval": 25}})
         self.plugin.tasks = 'fake tasks'  # this plugin by default has no tasks so force it to have some
@@ -49,19 +43,15 @@ class TestServerCloudPlugins(TestCase):
 
         # We only save the plugin config for the current config and ignore `other_plugin`
         self.assertDictEqual(plugin_server.config_options, PLUGIN_VALUE_1)
-        self.assertEqual(plugin_server.schedule_interval, SCHEDULE_INTERVAL)
 
-        # We should now schedule this to run
-        self.assertEquals(mocked_scheduler.called, True)
         self.plugin.tasks = None  # reset tasks value
 
     @parameterized.expand([["error", HTTPError], ["error", Timeout], ["unexpected", TooManyRedirects]])
-    @patch("insightconnect_plugin_runtime.server.PluginServer.register_scheduled_tasks")
     @patch("insightconnect_plugin_runtime.server.request_get")
     @patch("structlog.get_logger")
     @patch("insightconnect_plugin_runtime.server.CPS_RETRY", new=2)  # reduce retries in unit tests
     @patch("insightconnect_plugin_runtime.server.RETRY_SLEEP", new=1)  # reduce sleep in unit tests
-    def test_cps_schedule_raises_an_error(self, test_cond, exception, log, mocked_req, _mocked_scheduler, _mock_cloud, _run):
+    def test_cps_raises_an_error(self, test_cond, exception, log, mocked_req, _mock_cloud, _run):
         log.return_value = Logger()
         # If we have successfully got config and scheduler options, and later this call fails we should keep values
         mocked_req.return_value = MockResponse({"plugins": {self.plugin_name: PLUGIN_VALUE_1, 'plugin': PLUGIN_VALUE_2},
@@ -72,9 +62,8 @@ class TestServerCloudPlugins(TestCase):
         plugin_server.start()
 
         self.assertDictEqual(plugin_server.config_options, PLUGIN_VALUE_2)
-        self.assertEqual(plugin_server.schedule_interval, DEFAULT_SCHEDULE_INTERVAL_MINUTES)  # no resp uses default
 
-        # First call has happened and now successful - force scheduler to hit specific handled and unexpected errors.
+        # First call has happened and now successful - force to hit specific handled and unexpected errors.
         mocked_req.side_effect = exception("Warning HTTP error returned...")
         plugin_server.get_plugin_properties_from_cps()
         # we log error in all and `unexpected` in TooManyRedirects as there is no direct catch for this
@@ -82,16 +71,13 @@ class TestServerCloudPlugins(TestCase):
 
         # Values should not have changed
         self.assertDictEqual(plugin_server.config_options, PLUGIN_VALUE_2)
-        self.assertEqual(plugin_server.schedule_interval, DEFAULT_SCHEDULE_INTERVAL_MINUTES)
 
-        # Next schedule returns updated values, new schedule and no configurations for plugins
-        new_schedule = 12345678
-        mocked_req.return_value = MockResponse({"config": {"interval": new_schedule}})
+        # Next schedule returns no configurations for plugins
+        mocked_req.return_value = MockResponse({})
         mocked_req.side_effect = None
         plugin_server.get_plugin_properties_from_cps()
 
         # And this new values are now updated for the plugin server
         self.assertDictEqual(plugin_server.config_options, {})
-        self.assertEqual(plugin_server.schedule_interval, new_schedule)
 
         self.plugin.tasks = None  # reset tasks value