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

Fix random Arbor ArbOS ghost command bug #169

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

Conversation

kedare
Copy link
Contributor

@kedare kedare commented Dec 3, 2021

On many of our production device, Arbor ArbOS would randomly insert the command in the command result, this is a fix for it.

It may be interesting to insert the fix also in the commands run in compliances and others, would making this new function global in the .js and calling is in runCommand works ?

@SCadilhac
Copy link
Contributor

I didn't see the issue in some testing. Are you sure it's really the device sending back the command? Can you take a debug snapshot (https://github.com/netfishers-onl/Netshot/wiki/Snapshot-troubleshooting) to confirm?
By the way, don't you want to add parsing of the hardware block to extract the part number and serial number?

@kedare
Copy link
Contributor Author

kedare commented Dec 13, 2021

All our instances are running in VM so I cannot have the proper output to add support for modules on my side (but I already get the "device/vm" serial number properly).

I confirm you that this bug is appearing when doing snapshot on all our (VM) instances quite randomly, almost half the time when running the 1.0 driver, and never happens wiht 1.1

I tried the snapshot with debug mode as said in the doc you pointed but the snapshots never ends (keep consuming 100% cpu on 2 cores, had to restart netshot after 5 minutes, the last log I get is Ssh$JschLogger: Authentication succeeded (password).)

@kedare
Copy link
Contributor Author

kedare commented Dec 28, 2021

Do you have the same issue with debug mode ? I cannot test more because of this.

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.

None yet

2 participants