Skip to content

Conversation

@sukhwinder33445
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 commented Jan 7, 2026

fixes #491

The controllers for icingadb and the monitoring backend have been separated to remove the unwanted dependency.
Minor adjustments made to the view scripts so that it can be used by both backends.

@sukhwinder33445 sukhwinder33445 self-assigned this Jan 7, 2026
@sukhwinder33445 sukhwinder33445 added the bug Something isn't working label Jan 7, 2026
@cla-bot cla-bot bot added the cla/signed label Jan 7, 2026
@sukhwinder33445 sukhwinder33445 force-pushed the fix/remove-monitoring-module-dependency branch 2 times, most recently from ff4e5db to ac0635c Compare January 7, 2026 14:10
@sukhwinder33445 sukhwinder33445 marked this pull request as draft January 7, 2026 15:54
By separating `monitoring` module controllers.

- Adjust view-script paths accordingly
…essage

- Host|ServiceController: The extended class `CompatController` has its own tabs. Therefore, set the view script tabs to null
  to avoid creating tabs twice.
- View-scripts: Add a condition to create tabs only when the `monitoring` backend (ido[Host|Service]Controller) is used.
@sukhwinder33445 sukhwinder33445 force-pushed the fix/remove-monitoring-module-dependency branch from ac0635c to d17c450 Compare January 13, 2026 07:52
@sukhwinder33445 sukhwinder33445 marked this pull request as ready for review January 13, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cla/signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Businessprocess module dependency of monitoring module

2 participants