Project

General

Profile

Actions

Bug #27151

closed

The Host configuration chart shows 100% even if few hosts are not in sync or reporting.

Added by Marek Hulán almost 5 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Dashboard
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1666324

Description of problem:

"Host configuration chart for Puppet" on Dashboard shows 100% ok, even though there are eg. 2 out of 1000 hosts in "out of sync".
The problem here is: When it says 100% ok, it should mean all - not almost all.

This problem is probably due to a roundup function that makes everything above and equal to 99.5% into 100% in the graph.
While the calculation is mathematically correct, it is pretty misleading, since you can not have fractions of a host.

Examples:
  • When 99.75% out of 1000 hosts are OK, it should show 99% and not 100%.

Version-Release number of selected component (if applicable):

Red Hat Satellite 6.3.4

How reproducible:

100%

Actual results:

When it says 100% ok, it should mean all whereas it is not the case here.

Expected results:

It should not show 100% if some host is still not proper this could mislead.

Actions #1

Updated by Marek Hulán almost 5 years ago

  • Subject changed from The Host configuration chart shows 100% even if few hosts are not in sync or reporting. to The Host configuration chart shows 100% even if few hosts are not in sync or reporting.

This is still reproducible with current nightly. Just mock the report data that hosts ok are 100 000, you'll see 100% is ok.

Actions #2

Updated by The Foreman Bot about 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Amir Fefer
  • Pull request https://github.com/theforeman/foreman/pull/7542 added
Actions #3

Updated by The Foreman Bot about 4 years ago

  • Fixed in Releases 2.1.0 added
Actions #4

Updated by Amir Fefer about 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by The Foreman Bot about 4 years ago

  • Pull request https://github.com/theforeman/foreman/pull/7548 added
Actions #6

Updated by Tomer Brisker over 3 years ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/7548)
Actions

Also available in: Atom PDF