Skip to content
This repository has been archived by the owner on Mar 7, 2018. It is now read-only.

Faulty data in tests and output #84

Closed
tehhobbit opened this issue Nov 8, 2016 · 0 comments · Fixed by #79
Closed

Faulty data in tests and output #84

tehhobbit opened this issue Nov 8, 2016 · 0 comments · Fixed by #79
Labels
Milestone

Comments

@tehhobbit
Copy link
Contributor

Description of Issue/Question

When polling environment on an arista the following dict is returned

{
'fans': []
'cpu': []
'temperature: []
'power': []
'available_ram': []
'used_ram': []
}

According to https://github.com/napalm-automation/napalm-base/blob/develop/napalm_base/base.py#L325 the dict should look like

{
'fans': []
'cpu': []
'temperature': []
'power': []
}

Setup

napalm-eos version

(Paste verbatim output from pip freeze | grep napalm-eos between quotes below)


eOS version

(Paste verbatim output from show version | json between quotes below)


Steps to Reproduce the Issue

Error Traceback

(Paste the complete traceback of the exception between quotes below)


dbarrosop pushed a commit that referenced this issue Nov 8, 2016
* Clean up and bug fixes

* Fixes a bug where onboard sensors where not displayed
* Fixes a bug where the regex used for ram would cause a stacktrace
* Fixes a bug where total ram was returned instead of available
* Fixes a bug where the regex for cpu usage would cause a stacktrace
* Fixes a bug where only cpu used by userspace was reported
* Clean up of overly nested loops

* Tests for newer versions of EOS
@dbarrosop dbarrosop added the bug label Nov 8, 2016
@dbarrosop dbarrosop added this to the 0.4.4 milestone Nov 8, 2016
bewing pushed a commit to bewing/napalm-eos that referenced this issue Dec 6, 2016
* Clean up and bug fixes

* Fixes a bug where onboard sensors where not displayed
* Fixes a bug where the regex used for ram would cause a stacktrace
* Fixes a bug where total ram was returned instead of available
* Fixes a bug where the regex for cpu usage would cause a stacktrace
* Fixes a bug where only cpu used by userspace was reported
* Clean up of overly nested loops

* Tests for newer versions of EOS
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants