gluster / gluster-health-report

Gluster Health Report Tool
GNU Lesser General Public License v3.0
23 stars 20 forks source link

glusterd-report: Fix incorrect reporting of glusterd uptime #18

Closed pdvian closed 7 years ago

pdvian commented 7 years ago

We seems to have been comparing uptime in seconds to str "86400" seconds. It should be compared with int value of glusterd uptime to 86400 seconds (24 hr).

Signed-off-by: Prashant D pdhange@redhat.com

pdvian commented 7 years ago

Even if glusterd's uptime is more than 24 hours, we are still getting below warning:

[WARNING] Glusterd uptime is less than 24 hours uptime_sec=278646

Here 278646 sec converts to 3 days 22 hours

ps -C glusterd --no-header -o etimes

278649

aravindavk commented 7 years ago

Good catch. Thanks for the patch.