cmdbsyncer 4.1.0.dev6__tar.gz → 4.1.0.dev8__tar.gz

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cmdbsyncer
-Version: 4.1.0.dev6
+Version: 4.1.0.dev8
 Summary: Rule-based modular synchronizer for Checkmk, Netbox, Ansible, I-DOIT, Jira CMDB and 15+ other CMDB/monitoring systems
 Author-email: Kuhn & Ruess GmbH <info@kuhn-ruess.de>
 Maintainer-email: Bastian Kuhn <mail@bastian-kuhn.de>

cmdbsyncer-4.1.0.dev8/application/buildinfo.txt

@@ -0,0 +1 @@
+BUILD_DATE=2026-05-11T10:58:19Z

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/application/changelog/v4.1.md

@@ -53,6 +53,9 @@
 - FEAT: Quick-search on Rule Management matches name and ruleset
 - FIX: Saving a rule with outcomes that target different rulesets is rejected with a flash
 - FEAT: New "Manage Notification Rules" view exports notification rules to Checkmk 2.4 / 2.5 with `cmdbsyncer checkmk export_notifications`; admin edits are detected and corrected on the next run
+- FIX: Checkmk HW/SW inventorize multiprocessing pool initialises a fresh MongoDB connection per worker so writes no longer fail with an instant "Timeout error"
+- FIX: Checkmk HW/SW inventory worker errors show the failing hostname, exception type and full traceback instead of a misleading "Timeout error"
+- FIX: Checkmk API errors with a non-JSON body (HTML error page, auth redirect, disabled endpoint) report status code, URL and body snippet instead of the useless message "None NoneNone"
 
 ### API
 - FEAT: `/api/v1/objects/<hostname>/relations` reads, adds and removes typed Host relations (CMDB_MODE only)
@@ -83,6 +86,7 @@
 - FEAT: Set Theme, Set 2FA and Change Password pages now keep the admin navigation
 - FIX: Settings → License menu entry is hidden on Community Edition installs (no enterprise package present), so admins are not pointed at a page with no upload form and an empty feature table
 - FIX: PyPI installs ship the bundled theme CSS files (Gruvbox Dark / Light, Nord, Dracula) so the theme picker actually shows themes other than Default
+- FIX: Log details and traceback boxes pick up theme colours so they no longer render as a half-white card on Gruvbox Dark / Nord / Dracula
 
 ### Plugin runtime
 - FIX: HTTP sessions and CA-cert temp files are released even when a plugin's init fails

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/application/modules/log/views.py

@@ -17,7 +17,8 @@ def format_log(_view, _ctx, model, _path):
     rows = ''.join(
         '<tr>'
         '<th style="width:1%;vertical-align:top;padding:2px 8px 2px 0;'
-        f'white-space:nowrap;color:#666;font-weight:600;">{escape(entry.level)}</th>'
+        'white-space:nowrap;color:var(--surface-muted,#666);'
+        f'font-weight:600;">{escape(entry.level)}</th>'
         '<td style="vertical-align:top;word-break:break-word;'
         f'overflow-wrap:anywhere;white-space:pre-wrap;">{escape(entry.message)}</td>'
         '</tr>'
@@ -34,19 +35,23 @@ def format_traceback(_view, _ctx, model, _path):
     """
     Tracebacks are routinely 30-80 lines and would otherwise blow up the
     detail card. Render them inside a collapsed `<details>` so the row
-    stays scannable and the operator can pop it open on click.
+    stays scannable and the operator can pop it open on click. Colours
+    come from theme tokens so the `<pre>` block doesn't show up as a
+    white card on Gruvbox/Nord/Dracula rows.
     """
     trace = (model.traceback or '').strip()
     if not trace or trace == 'NoneType: None':
         return ''
     return Markup(
         '<details style="margin:0;">'
-        '<summary style="cursor:pointer;color:#2c5d99;'
+        '<summary style="cursor:pointer;color:var(--surface-muted,#2c5d99);'
         'font-family:ui-monospace,monospace;font-size:0.88rem;">'
         'Show traceback</summary>'
         '<pre style="margin-top:6px;padding:8px 10px;'
-        'background:#f6f8fa;border:1px solid #e2e6ea;border-radius:6px;'
-        'font-size:0.82rem;white-space:pre-wrap;'
+        'background:var(--surface-subtle,#f6f8fa);'
+        'border:1px solid var(--surface-border,#e2e6ea);'
+        'color:var(--surface-text,inherit);'
+        'border-radius:6px;font-size:0.82rem;white-space:pre-wrap;'
         f'overflow-x:auto;">{escape(trace)}</pre>'
         '</details>'
     )

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/application/plugins/checkmk/cmk2.py

@@ -91,7 +91,7 @@ class CMK2(Plugin):
             ) from exc
 
 
-    def request(self, url, method='GET', data=None,  # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-locals
+    def request(self, url, method='GET', data=None,  # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-locals,too-many-branches
                 params=None, additional_header=None, api_version="api/1.0/"):
         """
         Handle Request to CMK
@@ -140,9 +140,22 @@ class CMK2(Plugin):
                 return {}, {"error": "Object not found"}
             if response.status_code != 200:
                 if  response_json.get('title') not in error_whitelist:
-                    raise CmkException(f"{response_json.get('title')} "\
-                                       f"{response_json.get('detail')}"\
-                                       f"{response_json.get('fields')}")
+                    title = response_json.get('title')
+                    detail = response_json.get('detail')
+                    fields = response_json.get('fields')
+                    if title is None and detail is None and fields is None:
+                        # Non-JSON error body (HTML error page, auth
+                        # redirect, legacy endpoint disabled, …) — surface
+                        # status + URL + a body snippet so the failure is
+                        # debuggable instead of a useless "None NoneNone".
+                        body = (response.text or '').strip().replace('\n', ' ')
+                        if len(body) > 300:
+                            body = body[:300] + '…'
+                        raise CmkException(
+                            f"HTTP {response.status_code} {method.upper()} "
+                            f"{url}: {body or '<empty response body>'}"
+                        )
+                    raise CmkException(f"{title} {detail}{fields}")
                 return {}, {'status_code': response.status_code}
             resp_header['status_code'] = response.status_code
 

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/application/plugins/checkmk/inventorize.py

@@ -6,9 +6,11 @@ import datetime
 import hashlib
 import json
 import multiprocessing
+import traceback
 
 from rich.progress import Progress, SpinnerColumn, TimeElapsedColumn, MofNCompleteColumn
 
+from application import init_db
 from application.models.host import Host, app
 from application.models.host_inventory_tree import (
     HostInventoryTree,
@@ -140,23 +142,47 @@ class InventorizeHosts(CMK2):  # pylint: disable=too-many-instance-attributes
                       *Progress.get_default_columns(),
                       TimeElapsedColumn()) as progress:
             task1 = progress.add_task("Requesting HW/SW Inventory Data from Checkmk", total=total)
-            with multiprocessing.Pool() as pool:
+            # init_db gives each worker a fresh MongoEngine connection;
+            # without it the workers inherit the parent's connection across
+            # fork() and the first _save_inventory_tree() write raises
+            # immediately (matching the "instant timeout" symptom).
+            with multiprocessing.Pool(initializer=init_db) as pool:
+                # Pair every async result with its hostname so error output
+                # can name the failing host. Previously the hostname was
+                # only recoverable from the worker's return tuple, which we
+                # never see when the worker raised or timed out.
                 tasks = []
                 for host_resp in response:
                     hostname = host_resp['extensions']['host_name']
                     self.add_host(hostname)
-                    task = pool.apply_async(self.get_hw_sw_inventory_data, args=(hostname,))
-                    tasks.append(task)
+                    async_result = pool.apply_async(
+                        self.get_hw_sw_inventory_data, args=(hostname,)
+                    )
+                    tasks.append((hostname, async_result))
 
-                for task in tasks:
+                process_timeout = app.config['PROCESS_TIMEOUT']
+                for hostname, task in tasks:
                     try:
-                        hostname, data = task.get(timeout=app.config['PROCESS_TIMEOUT'])
+                        _host, data = task.get(timeout=process_timeout)
                     except multiprocessing.TimeoutError:
-                        progress.console.print("- ERROR: Timeout for a object")
+                        progress.console.print(
+                            f"- {ColorCodes.FAIL}TIMEOUT{ColorCodes.ENDC} "
+                            f"after {process_timeout}s for {hostname}"
+                        )
                     except Exception as error:  # pylint: disable=broad-exception-caught
+                        # apply_async preserves the worker traceback on
+                        # .get(); print it so the operator can see what
+                        # really blew up instead of a misleading
+                        # "Timeout error" label. --debug re-raises to abort
+                        # the run for interactive inspection.
                         if self.debug:
                             raise
-                        progress.console.print(f"- ERROR: Timeout error for object ({error})")
+                        progress.console.print(
+                            f"- {ColorCodes.FAIL}ERROR{ColorCodes.ENDC} "
+                            f"in worker for {hostname}: "
+                            f"{type(error).__name__}: {error}"
+                        )
+                        progress.console.print(traceback.format_exc())
                     else:
                         progress.advance(task1)
                         if data is not None:

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/cmdbsyncer.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cmdbsyncer
-Version: 4.1.0.dev6
+Version: 4.1.0.dev8
 Summary: Rule-based modular synchronizer for Checkmk, Netbox, Ansible, I-DOIT, Jira CMDB and 15+ other CMDB/monitoring systems
 Author-email: Kuhn & Ruess GmbH <info@kuhn-ruess.de>
 Maintainer-email: Bastian Kuhn <mail@bastian-kuhn.de>

{cmdbsyncer-4.1.0.dev6 → cmdbsyncer-4.1.0.dev8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "cmdbsyncer"
-version = "4.1.0.dev6"
+version = "4.1.0.dev8"
 description = "Rule-based modular synchronizer for Checkmk, Netbox, Ansible, I-DOIT, Jira CMDB and 15+ other CMDB/monitoring systems"
 readme = "README.md"
 license = { text = "MIT" }

cmdbsyncer-4.1.0.dev6/application/buildinfo.txt

@@ -1 +0,0 @@
-BUILD_DATE=2026-05-08T14:36:22Z