Fixed Synology System Temerature being overwritten #1649

Merged
merged 2 commits into from Aug 9, 2015

Projects

None yet

4 participants

@arnoldthebat
Contributor

Issue #1648. Hardcoded to 99 since its unlikely to have a disk array that large!

@arnoldthebat arnoldthebat Fixed Synology System Temerature being overwritten
81ed33e
@laf
Member
laf commented Aug 9, 2015

Thanks @arnoldthebat.

What's overwriting the data?

We've updated our contributors agreement, would you mind resigning and pushing another commit (just add a space after your name to force the file to change).

@arnoldthebat arnoldthebat Resigning
ae6e01e
@arnoldthebat
Contributor

Hi @laf

The index used to persist the System temperature was hardcoded to '2'. If the system in question has > 2 disks the array value of the disk overwrites the system temperature..

@f0o f0o merged commit 249e89f into librenms:master Aug 9, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment