Add CPU graph for Smart Edge 600 #5917

Closed
kyob opened this Issue Feb 16, 2017 · 15 comments

Comments

Projects
None yet
3 participants
@kyob

kyob commented Feb 16, 2017

DO NOT DELETE THIS INFORMATION.

Please read this information carefully.

GitHub issues is for bugs, please do not post issues asking for help or how to do X, Y or Z.
You can use our irc channel ##librenms on freenode to ask questions or our community site.
If you have a feature request please post this on our community site.

Please confirm each of the sections below by putting an x in the box like [x].

  • Is your install up to date? Updating your install
    Please do not submit an issue if your install is not up to date within the last 24 hours or on a stable monthly release.
  • Please include all of the information between the ==================================== section of ./validate.php which you can run from the cli.
  • Please provide ALL info asked for here.
  • Please provide as much detail as possible.
  • Please do NOT post more than 10 lines of debug information here, use a pastebin service or GitHub Gists.
Component Version
LibreNMS d06aaaa
DB Schema 164
PHP 7.0.15
MySQL 5.5.52-MariaDB
RRDTool 1.4.8
SNMP NET-SNMP 5.7.2

CPU OID:

  1. rbnCpuMeterFiveSecondAvg 1.3.6.1.4.1.2352.2.6.1.1
  2. rbnCpuMeterOneMinuteAvg 1.3.6.1.4.1.2352.2.6.1.2
  3. rbnCpuMeterFiveMinuteAvg 1.3.6.1.4.1.2352.2.6.1.3

@kyob kyob changed the title from Add CPU graph for Smart Edge 6000 to Add CPU graph for Smart Edge 600 Feb 16, 2017

@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Feb 20, 2017

Contributor

Your snmpwalk doesn't include the full tree. It probably timed out when retrieving it.

Can you try walking .1.3.6.1.4.1.2352?

Contributor

geordish commented Feb 20, 2017

Your snmpwalk doesn't include the full tree. It probably timed out when retrieving it.

Can you try walking .1.3.6.1.4.1.2352?

@kyob

This comment has been minimized.

Show comment
Hide comment
@kyob

kyob Feb 20, 2017

@geordish .1.3.6.1.4.1.2352 but it's not all because I got timed out

kyob commented Feb 20, 2017

@geordish .1.3.6.1.4.1.2352 but it's not all because I got timed out

@geordish geordish referenced this issue Feb 20, 2017

Merged

Add SEOS CPU discovery #5953

2 of 2 tasks complete
@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Feb 20, 2017

Contributor

Created PR #5953. Mind testing?

Contributor

geordish commented Feb 20, 2017

Created PR #5953. Mind testing?

laf added a commit that referenced this issue Feb 20, 2017

@laf laf closed this Feb 20, 2017

@kyob

This comment has been minimized.

Show comment
Hide comment
@kyob

kyob Feb 21, 2017

After #5953 has been merge I did daily.sh then validate.php (no errors).

So I remove my device Smart Edge 600 and added again which doesn't help still no graph for CPU.

How can I verify it works?

kyob commented Feb 21, 2017

After #5953 has been merge I did daily.sh then validate.php (no errors).

So I remove my device Smart Edge 600 and added again which doesn't help still no graph for CPU.

How can I verify it works?

@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Feb 21, 2017

Contributor

Provide the output of

./discovery.php -h HOSTNAME -d -m sensors
Contributor

geordish commented Feb 21, 2017

Provide the output of

./discovery.php -h HOSTNAME -d -m sensors

@geordish geordish reopened this Feb 21, 2017

@kyob

This comment has been minimized.

Show comment
Hide comment

kyob commented Feb 21, 2017

./discovery.php -h HOSTNAME -d -m sensors
https://gist.github.com/kyob/847a9aaf48dfa706570cae700d23c552

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 21, 2017

Member

./discovery.php -h HOSTNAME -d -m processors was meant

Member

laf commented Feb 21, 2017

./discovery.php -h HOSTNAME -d -m processors was meant

@kyob

This comment has been minimized.

Show comment
Hide comment

kyob commented Feb 21, 2017

./discovery.php -h HOSTNAME -d -m processors
https://gist.github.com/kyob/7704ec639b65a90e7a4b4d06019e5b19

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 21, 2017

Member

This shows your device is not responding to the OID .1.3.6.1.4.1.2352.2.6.1.1 which is the oid you gave in your original request as being the CPU

Member

laf commented Feb 21, 2017

This shows your device is not responding to the OID .1.3.6.1.4.1.2352.2.6.1.1 which is the oid you gave in your original request as being the CPU

@kyob

This comment has been minimized.

Show comment
Hide comment
@kyob

kyob Feb 21, 2017

But I get this from librenms server:

snmpwalk -On -v2c -c COMMUNITY HOST .1.3.6.1.4.1.2352.2.6.1.1
.1.3.6.1.4.1.2352.2.6.1.1.0 = INTEGER: 2

kyob commented Feb 21, 2017

But I get this from librenms server:

snmpwalk -On -v2c -c COMMUNITY HOST .1.3.6.1.4.1.2352.2.6.1.1
.1.3.6.1.4.1.2352.2.6.1.1.0 = INTEGER: 2

@geordish geordish referenced this issue Feb 21, 2017

Closed

SEOS CPU OID correction #5966

0 of 2 tasks complete
@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Feb 21, 2017

Contributor

I've just created a PR to correct this. Want to try it?

Contributor

geordish commented Feb 21, 2017

I've just created a PR to correct this. Want to try it?

@kyob

This comment has been minimized.

Show comment
Hide comment
@kyob

kyob Feb 21, 2017

Yes I did:

./scripts/github-apply 5966
Checking patch includes/discovery/processors/seos.inc.php...
error: includes/discovery/processors/seos.inc.php: already exists in working directory
Checking patch includes/discovery/processors/seos.inc.php...

I remove device and added again.

Nothing has change still graph are duplicated.

kyob commented Feb 21, 2017

Yes I did:

./scripts/github-apply 5966
Checking patch includes/discovery/processors/seos.inc.php...
error: includes/discovery/processors/seos.inc.php: already exists in working directory
Checking patch includes/discovery/processors/seos.inc.php...

I remove device and added again.

Nothing has change still graph are duplicated.

@geordish geordish referenced this issue Feb 21, 2017

Merged

Update SEOS processor OID #5967

2 of 2 tasks complete
@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Feb 21, 2017

Contributor

Sorry, my mistake.

#5967

Contributor

geordish commented Feb 21, 2017

Sorry, my mistake.

#5967

@kyob

This comment has been minimized.

Show comment
Hide comment
@kyob

kyob Feb 21, 2017

./scripts/github-apply 5967
Checking patch includes/discovery/processors/seos.inc.php...
Applied patch includes/discovery/processors/seos.inc.php cleanly.

...waiting for results.

Results below:

se600-temperature2

Graphs are duplicated.

kyob commented Feb 21, 2017

./scripts/github-apply 5967
Checking patch includes/discovery/processors/seos.inc.php...
Applied patch includes/discovery/processors/seos.inc.php cleanly.

...waiting for results.

Results below:

se600-temperature2

Graphs are duplicated.

@laf laf closed this Feb 21, 2017

@lock

This comment has been minimized.

Show comment
Hide comment
@lock

lock bot May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

lock bot commented May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

@librenms librenms locked as resolved and limited conversation to collaborators May 17, 2018

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