Icinga / icingaweb2-module-businessprocess

Create top-level views of your applications in a graphical editor. Rules express dependencies between existing hosts and services and let you alert on application level. Business processes are displayed in a tree or list overview.

Home Page:https://icinga.com/docs/businessprocess/latest

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

IcingaDB - Services in BPs are not shown anymore for filtered roles (missing nodes)

MarcusCaepio opened this issue · comments

Describe the bug

Hi all,
I have two different views between the monitoring and the icingadbweb module right now.
Monitoring: When a role has access to a BP, the user can see the whole BP, no matter if the user has access to the several service details or not. When they don't have access to the service details, they get a permission denied info. But they can see the whole status of the BP itself

IcingaDB: The services, which are locked for the role disappear in the business process, too. The user gets the red bar with "X nodes are missing". So the BP is not usable anymore. I have to unlock every service for the roles, what was not necessary in the monitoring module.

To Reproduce

  1. Create a BP
  2. Filter a service for a specific role
  3. The service is visible in the BP for the filtered role when using the monitoring module, service details are locked with "no permission"
  4. The service is not visible in the BP for the filtered role, when using the Icingadbweb module, user gets "nodes missing"

Expected behavior

Same view in both monitoring and icingadbweb modules

Screenshots

image
image

Your Environment

Ubuntu 20.04.4 lts
Icinga Web 2 Version 2.11.1
PHP Version 7.4.3
icinga/icinga-php-thirdparty 0.11.0
icinga/icinga-php-library 0.9.1
businessprocess 2.4.0 Configure

Additional context

Module is currently not usable with IcingaDB :(

Good morning all,
the PR is open since 19 days. When will it be merged? We really would like to use the module again :)

It's included in version 2.4.1 released just now.