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

Junos vrf support in get_arp() #567

Merged
merged 1 commit into from Dec 18, 2017

Conversation

ckishimo
Copy link
Contributor

Support for VRF in get_arp() following #512 in Junos

ckishimo@ex9208> show arp vpn VRF | display xml rpc    
<rpc-reply xmlns:junos="http://xml.juniper.net/junos/16.1R5/junos">
    <rpc>
        <get-arp-table-information>
                <vpn>VRF</vpn>
        </get-arp-table-information>
    </rpc>
    <cli>
        <banner>[edit]</banner>
    </cli>
</rpc-reply>
ckishimo@ex9208> show arp vpn VRF                      
MAC Address       Address         Name                      Interface               Flags
2c:21:31:a5:14:84 192.168.60.2    192.168.60.2              ge-3/1/3.0              none
[
    {
        "age": 882.0, 
        "interface": "ge-3/1/3.0", 
        "ip": "192.168.60.2", 
        "mac": "2C:21:31:A5:14:84"
    }
]

@mirceaulinic
Copy link
Member

@ckishimo can you please rebase your branch?

Add missing "vrf" token to IOS arp command

implement vrf in get_arp()
@ckishimo
Copy link
Contributor Author

@mirceaulinic oops! I just did... pls let me know if everything is in order now. Thanks

@coveralls
Copy link

coveralls commented Dec 18, 2017

Coverage Status

Coverage decreased (-0.03%) to 78.598% when pulling 260ecdf on ckishimo:arp-vrf into 86617bc on napalm-automation:arp-vrf.

@mirceaulinic mirceaulinic merged commit f34c23d into napalm-automation:arp-vrf Dec 18, 2017
@ckishimo ckishimo deleted the arp-vrf branch December 18, 2017 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants