Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix get_environnements available memory #605

Merged
merged 1 commit into from
Jan 2, 2018

Conversation

targuan
Copy link
Contributor

@targuan targuan commented Dec 30, 2017

Report total memory instead of free memory for ios and eos drivers

@coveralls
Copy link

coveralls commented Dec 30, 2017

Coverage Status

Coverage remained the same at 78.708% when pulling 60e1b31 on targuan:bugfix/total_memory into d7ce657 on napalm-automation:develop.

@dbarrosop dbarrosop self-requested a review January 2, 2018 08:29
@dbarrosop dbarrosop merged commit d53615d into napalm-automation:develop Jan 2, 2018
bewing added a commit that referenced this pull request Jan 8, 2018
* develop: (128 commits)
  bump version (#613)
  Add napalm logo (previously removed probably during reunification)
  Interface is always enabled, unless explicitly specified in config
  Fix get_environnements available memory (#605)
  [docs] pynxos instead of pycsco for NXOS
  QFX10008 apparently still uses the same old RPC (as EX9208)
  Fix for napalm-automation/napalm-iosxr#104. IOSXRv returns empty list for oper.PlatformInventory.RackTable.Rack.Attributes.BasicInfo, causing a fatal IndexError for the variable 'platform_attr_tree' in the 'get_facts' function. In fact, IOSXRv walkdata oper.PlatformInventory returns no data at all. As a result, only an interface_list will be returned and the remaining default facts values from the function will be used. This condition does NOT occur on physical IOSXR hosts. Therefore, original indexing needs to be preserved while the error condition is handled for IOSXRv hosts.
  Biatch
  Screw you too
  Who said rebasing is not fun
  fix missing file in power supply mocked data
  Readd the state / status key. The rebase hell continues
  Lost this commit somehow in this cherry-pick hell
  Pass password when using ssh key
  suppress FutureWarning (#218)
  fix removal of cluster key (#219)
  fix pep8 errors
  implement find operation in cli
  fix multihop ttl to its default value
  remove cluster from the neighbor output
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants