napalm-automation-community / napalm-vyos

NAPALM Driver for the VyOS
Apache License 2.0
38 stars 27 forks source link

Add vrf arg to ping #12

Closed mirceaulinic closed 7 years ago

mirceaulinic commented 7 years ago

Looks like vyos doesn't know about VRF so these changes should be reflected only in the method header

mirceaulinic commented 7 years ago

Solves https://github.com/napalm-automation/napalm-vyos/issues/11

ppieprzycki commented 7 years ago

@mirceaulinic That's right at this moment vyos doesn't support vrf. It's future request. Change seems ok. If I modify constant file in napalm_base this method seems to work

Go ahead