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

Updated Aruba AOSCX show_vsf_detail with additional values #1942

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ritkit
Copy link

@ritkit ritkit commented Dec 19, 2024

I needed the Topology type, hostname, and serial numbers from members in the stacks. I also fixed the status value as it would only get the first word and there are statuses with multiple works. Such as "No Link".

Co-authored-by: Michael Bear <38406045+mjbear@users.noreply.github.com>
@mjbear
Copy link
Contributor

mjbear commented Dec 20, 2024

I also fixed the status value as it would only get the first word and there are statuses with multiple works. Such as "No Link".

@ritkit
If you have different output, you could also add a snippet to the test data.

While I don't see "No Such" link, there is "No Split" as a multi-word status already.

@ritkit
Copy link
Author

ritkit commented Dec 23, 2024

@mjbear I pulled the changes we discussed and added the second raw test with second yml output. Take a look and see if it looks correct.

@mjbear
Copy link
Contributor

mjbear commented Dec 24, 2024

@mjbear I pulled the changes we discussed and added the second raw test with second yml output. Take a look and see if it looks correct.

@ritkit
Yep, the new test data looks good.
Congrats! 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants