Skip to content
This repository has been archived by the owner on Jan 16, 2019. It is now read-only.

Implements new getter: get_optics #134

Merged
merged 1 commit into from
Apr 4, 2017
Merged

Implements new getter: get_optics #134

merged 1 commit into from
Apr 4, 2017

Conversation

kaage
Copy link
Contributor

@kaage kaage commented Mar 31, 2017

This PR simply implements get_optics getter for napalm-ios. I follows same output format as the ones for napalm-eos and napalm-junos. As not all output values are availble they are set to zero (or 0.0.) by default.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 67.205% when pulling 12711c1 on kaage:develop into f3bb7cd on napalm-automation:develop.

1 similar comment
@coveralls
Copy link

coveralls commented Mar 31, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 67.205% when pulling 12711c1 on kaage:develop into f3bb7cd on napalm-automation:develop.

@kaage kaage mentioned this pull request Mar 31, 2017
Closed
@ktbyers
Copy link
Contributor

ktbyers commented Apr 4, 2017

Looks good.

@ktbyers ktbyers merged commit 1069480 into napalm-automation:develop Apr 4, 2017
@ktbyers ktbyers added this to the 0.6.2 milestone May 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6.2
Development

Successfully merging this pull request may close these issues.
3 participants
@kaage @coveralls @ktbyers