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

Dashboard Migration-Tool - some parameter are not converted correctly #953

Closed
stevie-sy opened this issue Jan 4, 2024 · 2 comments · Fixed by #995
Closed

Dashboard Migration-Tool - some parameter are not converted correctly #953

stevie-sy opened this issue Jan 4, 2024 · 2 comments · Fixed by #995
Assignees
Labels
bug Something isn't working
Milestone

Comments

@stevie-sy
Copy link

We tested the new migration tool for dashboard from the monitoring modul to icingadb-web - wich cames with the latest version of icingaweb/icingadb-web. Basically it worked very well. But there were two parameter which wouldn't converted correctly:

e.g.

  1. If there was in the filter service_state_type=Soft, that would be converted to service.state.state_type=(soft|hard). But that throws following error:
    image
    With a string comparision - only "sof"t or " hard" - the dashboard works

  2. If somebody has the param "modifyFilter" in his filter, the dasboard doesn't work. So it looks like that the migration don't handle this (e.g. delete the param)

Your Environment

Include as many relevant details about the environment you experienced the problem in.

  • Icinga DB Web version (System - About): 1.1.1
  • Icinga Web 2 version (System - About): 2.12.1
  • Web browser: Firefox, Edge
  • Icinga 2 version (icinga2 --version): 2.14.1
  • Icinga DB version (icingadb --version): 1.1.1
  • PHP version used (php --version): 8.0.3
  • Server operating system and version: rhel 9
@carraroj
Copy link

carraroj commented Jan 4, 2024

ref/NC/804872

@nilmerg nilmerg added this to the 1.1.2 milestone Jan 8, 2024
@nilmerg
Copy link
Member

nilmerg commented Jan 8, 2024

  1. (soft|hard) is array syntax and completely valid. The class ipl\Web\SearchBar\Terms doesn't seem to handle this correctly. There may be more cases of invalid handling, (by the search bar) so further errors should be expected. Though, javascript might not fully support this syntax yet. To fix this, having it rendered should suffice for the moment.
  2. The class UrlMigrator of Icinga DB Web is responsible for parameter handling. It needs to drop modifyFilter as suggested.

@nilmerg nilmerg added bug Something isn't working difficulty/easy Anyone can do this labels Jan 8, 2024
@nilmerg nilmerg self-assigned this Mar 21, 2024
@nilmerg nilmerg removed the difficulty/easy Anyone can do this label Mar 21, 2024
nilmerg added a commit that referenced this issue Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants