Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #9750] Classic UI is showing inaccurate value to retry_interval #1556

Closed
icinga-migration opened this issue Jul 28, 2015 · 3 comments

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/9750

Created by aklimov on 2015-07-28 07:53:53 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2015-08-04 17:50:05 +00:00)
Target Version: 1.14
Last Update: 2015-08-04 17:50:05 +00:00 (in Redmine)

Icinga Version: 2.3.8
OS Version: any

Host config:

object Host "localhost" {
  import "generic-host"

  address = "127.0.0.1"
  address6 = "::1"

  check_interval = 10s
  retry_interval = 5s

  vars.os = "Linux"
  vars.sla = "24x7"
}

Classic UI output: see attached screenshot

Attachments

Changesets

2015-08-04 17:44:49 +00:00 by mfriedrich 506a281

config.cgi: Fix double values in time format strings

fixes #9750
@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-07-30 08:56:12 +00:00

  • Project changed from Icinga 2 to Core, Classic UI, IDOUtils
  • Subject changed from Classic UI is lying about the check retry interval to Classic UI is showing inaccurate value to retry_interval
  • Category set to Classic UI
  • OS Version set to any

The problem is that the Icinga 1.x Classic UI is rounding down the floating point value.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-08-04 17:41:23 +00:00

  • File added Auswahl_059.png
  • Status changed from New to Assigned
  • Assigned to set to mfriedrich
  • Target Version set to 1.14

The problem is the calculation inside get_interval_time_string using integers and modulo for calculating the correct time formatting. Furthermore the time_string char buffer is statically set to 16 bytes, while 32 weould suffice more.

Auswahl_059.png

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-08-04 17:50:05 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 506a281.

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

No branches or pull requests

1 participant