Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency of Module does not match existing Icingaweb version #42

Open
laugmanuel opened this issue Aug 10, 2021 · 4 comments
Open

Dependency of Module does not match existing Icingaweb version #42

laugmanuel opened this issue Aug 10, 2021 · 4 comments

Comments

@laugmanuel
Copy link

Icingaweb2 reports unmet dependencies:

image

However, the affected Icinga installation uses monitoring in 2.9.2
image

Expected Behavior

The module should find the present version of monitoring and not report any unmet dependencies.

Current Behavior

The plugin reports unmet dependencies.

Possible Solution

Steps to Reproduce (for bugs)

Install the plugin an instance with Icingaweb2 2.9.2.

Context

The plugin works fine if it is activated, but a user can not activate it using the current icingaweb2 version (due to unmet dependencies).

Your Environment

  • Module version (System - About): Pnp 1.1.0 (Commit 13a50ef)
  • Icinga Web 2 version and modules (System - About): Icingaweb2 2.9.2 (Commit 07e25a923bd80001e9ab52bbbbc1008aa8befb06)
  • Icinga 2 version (icinga2 --version): 2.13.0-1
  • Operating System and version: RHEL 7.9
  • Webserver, PHP versions: HTTPD 2.4.6-97 & PHP 7.3.20
@laugmanuel laugmanuel changed the title Dependency of Module does not match Dependency of Module does not match existing Icingaweb version Aug 10, 2021
@ryanrudak
Copy link

ryanrudak commented Oct 5, 2021

I think the syntax convention in the 'module.info' file has changed.

from old:
Depends: monitoring>=2.1.0

to new:
Depends: monitoring (>= 2.1.0)

@dpenezic
Copy link

dpenezic commented Nov 7, 2021

Just to confirm @ryanrudak bug fix solution , when I done that change , dependency is correctly validated.

@withanhdammit
Copy link

Confirmed, I had the same issue, and the same fix.

danopt added a commit to danopt/icingaweb2-module-pnp that referenced this issue Aug 16, 2022
@jamespo
Copy link

jamespo commented Nov 14, 2023

With Icinga DB Web isn't the monitoring module outdated now & not required at all? Ideally the requirements would be updated to require one or other of monitoring or icingadb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants