dainok / netdoc

Automatic Network Documentation plugin for NetBox
GNU General Public License v3.0
84 stars 14 forks source link

Test allied telesis #120

Closed tatumi0726 closed 5 months ago

tatumi0726 commented 5 months ago

Hello!

We have created each file for Allied-Telesis and have even completed testing.

We would appreciate your confirmation that it is merge-worthy. Please let us know if any corrections are necessary. Thank you in advance for your time.

dainok commented 5 months ago

@tatumi0726 question: I see no commands about VRF. Is VRF not supported or just not included in your scripts?

dainok commented 5 months ago

@tatumi0726 second question: for each command I need the minimum alias, if exists. For example:

tatumi0726 commented 5 months ago

Hello, @dainok

Sorry for the late reply.

I have written VRF in Ingestor to collect from the logs obtained by "show ip route". Should I re-create it?

I think it is possible to get it by "show ip route vrf ".

I will answer with the alias of each command. show arp -> sh ar show etherchannel summary -> sh et s show interface -> sh in show interface switchport -> sh in s show ip interface -> sh ip in show_lldp_neighbors_detail -> sh ll n d show_mac_address_table -> sh mac a show_static_channel_group -> sh stat show system -> sh sy show_vlan_all -> sh vl al

Please contact us if there is any other information you need to provide. Thank you in advance.

2024年2月12日(月) 2:01 Andrea Dainese @.***>:

@tatumi0726 https://github.com/tatumi0726 second question: for each command I need the minimum alias, if exists. For example:

  • show arp -> sh ar
  • show system -> show sys
  • ... Thank you

— Reply to this email directly, view it on GitHub https://github.com/dainok/netdoc/pull/120#issuecomment-1937809337, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM4WL5DB6KUMFGIZ6I5EMVLYTD2PDAVCNFSM6AAAAABDDO64PSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZXHAYDSMZTG4 . You are receiving this because you were mentioned.Message ID: @.***>

dainok commented 5 months ago

Hi @tatumi0726

VRF: ok sorry I didn't realize that. Aliases: I need also the alias for "show ip route"

Next steps:

  1. I need to complete the PR for templates. I'm adding some requirements from NTS like error management and field normalization.
  2. Once the PR is completed, I'll ask you to check it on your production systems. That's because NTC it's not maintained by me and I prefer to commit only when I'm sure that everything is working
  3. Once the templates are released I'll check netdoc code.
tatumi0726 commented 5 months ago

Hello, @dainok

My apologies...I will contact you as follows

show ip route -> sh ip ro

All of this was confirmed using the actual device.

I understand about the next steps. I will implement what I can and look forward to hearing from you!

Thank you very much in advance.

2024年2月12日(月) 17:48 Andrea Dainese @.***>:

Hi @tatumi0726 https://github.com/tatumi0726

VRF: ok sorry I didn't realize that. Aliases: I need also the alias for "show ip route"

Next steps:

  1. I need to complete the PR for templates. I'm adding some requirements from NTS like error management and field normalization.
  2. Once the PR is completed, I'll ask you to check it on your production systems. That's because NTC it's not maintained by me and I prefer to commit only when I'm sure that everything is working
  3. Once the templates are released I'll check netdoc code.

— Reply to this email directly, view it on GitHub https://github.com/dainok/netdoc/pull/120#issuecomment-1938250565, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM4WL5HR6AY4UF3R4T3DX63YTHJMVAVCNFSM6AAAAABDDO64PSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZYGI2TANJWGU . You are receiving this because you were mentioned.Message ID: @.***>

dainok commented 5 months ago

@tatumi0726 any way to get VRF info in "show ip interface"? "show ip interface" includes IP from any VRF?

dainok commented 5 months ago

@tatumi0726 I see in show static-channel-group that 2 different outputs are possible. I need an example from OldSA

tatumi0726 commented 5 months ago

Good evening, @dainok.

The "show ip interface" did not output VRF information. I believe the command that would output the information you wanted would be 'show ip vrf interface'

Example output: awplus> show ip vrf interface ↓ Interface IP-Address Status Protocol Vrf lo1 unassigned admin up running red lo2 unassigned admin up running green vlan1 192.168.10.1/24 admin up running red vlan2 192.168.20.1/24 admin up running green

Regarding the example output of show static-channel-group. During the creation of this file, I forgot to separate it once because there were differences in the command output on slightly older firmware. The following is an example output.

Example output: awplus> show static-channel-group ↓ % Static Aggregator: sa11 % Member: sa1 port1.1.1 port1.1.2 port1.1.3 port1.1.4

Thank you for your time...

dainok commented 5 months ago

Hi @tatumi0726 ❗️ I almost completed templates. VRF/routing is missing, but I need more time to think about. In the meanwhile please double check the template I adjusted. Templates will be the fundamental part to parse data. You are required to check templates on your production environment:

Here the templates: https://github.com/dainok/ntc-templates/tree/add_allied_telesis/ntc_templates/templates Here I test I developed: https://github.com/dainok/ntc-templates/tree/add_allied_telesis/tests/allied_telesis_awplus

Notify my each exception and attach an output file so I can improve templates and tests.

tatumi0726 commented 5 months ago

Hi, @dainok!

Thanks for your reply. I am very sorry for your busy schedule and for the maintenance of the template.

I understand about the template file confirmation.

I will check it this Friday or Saturday and report back to you.

2024年2月13日(火) 23:30 Andrea Dainese @.***>:

Hi @tatumi0726 https://github.com/tatumi0726 ❗️ I almost completed templates. VRF/routing is missing, but I need more time to think about. In the meanwhile please double check the template I adjusted. Templates will be the fundamental part to parse data. You are required to check templates on your production environment:

Here the templates: https://github.com/dainok/ntc-templates/tree/add_allied_telesis/ntc_templates/templates Here I test I developed: https://github.com/dainok/ntc-templates/tree/add_allied_telesis/tests/allied_telesis_awplus

Notify my each exception and attach an output file so I can improve templates and tests.

— Reply to this email directly, view it on GitHub https://github.com/dainok/netdoc/pull/120#issuecomment-1941641288, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM4WL5AUBOXBB227RQ5DRM3YTN2G5AVCNFSM6AAAAABDDO64PSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBRGY2DCMRYHA . You are receiving this because you were mentioned.Message ID: @.***>

dainok commented 5 months ago

Moved to https://github.com/dainok/netdoc/pull/122