genieparser icon indicating copy to clipboard operation
genieparser copied to clipboard

Added support for 'show image version' in ArubaOS

Open ykoehler opened this issue 3 years ago • 3 comments

Description

Added parsing for 'show image version' within ArubaOS

Motivation and Context

New CLI command support

Impact (If any)

Screenshots:

2022-03-11 16:59:33,012: %UNICON-INFO: +++ thomas-mm with via 'cli': executing command 'show image version' +++
show image version

----------------------------------
Partition               : 0:0 (/mnt/disk1)
Software Version        : ArubaOS 8.6.0.16 (Digitally Signed SHA1/SHA256 - Production Build)
Build number            : 82590
Label                   : 82590
Built on                : Thu Dec 16 13:28:34 UTC 2021
----------------------------------
Partition               : 0:1 (/mnt/disk2) **Default boot**
Software Version        : ArubaOS 8.10.0.0 LSR (Digitally Signed SHA1/SHA256 - Developer/Internal Build)
Build number            : 0000
Label                   : [email protected]
Built on                : Wed Mar 9 09:41:5
(ArubaMM-VA_AF_70_12) *[mynode] #
{'thomas-mm': {'partition': {0: {'disk': '/mnt/disk1', 'firmware_version': 'ArubaOS 8.6.0.16', 'signature_type': 'SHA1/SHA256', 'build_type': 'Production', 'build_number': 82590, 'label': '82590', 'built_on': 'Thu Dec 16 13:28:34 UTC 2021'}, 1: {'disk': '/mnt/disk2', 'firmware_version': 'ArubaOS 8.10.0.0 LSR', 'signature_type': 'SHA1/SHA256', 'build_type': 'Developer/Internal', 'build_number': 0, 'label': '[email protected]', 'built_on': 'Wed Mar 9 09:41:5'}, 'default_partition': 1}}}
(pyats) root@c2942142d6df:/pyats# 

Checklist:

  • [x] I have updated the changelog.
  • [ ] I have updated the documentation (If applicable).
  • [x] I have added tests to cover my changes (If applicable).
  • [x] All new and existing tests passed.
  • [x] All new code passed compilation.

ykoehler avatar Mar 11 '22 17:03 ykoehler

Hi, I was unclear if the unittest need to be in /tests or in /src/.../arubaos/tests/

also, didn't figure out how I could run them localy.

ykoehler avatar Mar 11 '22 18:03 ykoehler

ok, sorry for all those delay, took a while to get back on this since I was making inquiry within my company to see if I was allowed to submit such code etc...

Apparently I am ok to submit such code, but neeed to do so on my work email, so I will cancel this PR and re-submit with my work email [email protected]

ykoehler avatar Jun 01 '22 14:06 ykoehler

Hi Yannick, If you have opened a new PR with a different email, could you please comment the link to the new PR?

GerriorL avatar Aug 09 '22 19:08 GerriorL