-
-
Notifications
You must be signed in to change notification settings - Fork 86
API Endpoints to submit arpnip and macsuck results #942
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
Conversation
* missing Result/Device.pm column added * pae_is... columns instead of pae_capabilities * moved most code to Util/PortAccessEntity.pm so the update can be done in discover and macsuck
- device.pae_control (text) is now device.pae_is_enabled (bool) - also store pae_authconfig_port_control (port mode auto/force(un)Auth)
- ... again because of botched merge - at least perlgolfed away a set of curly braces
Hi @ollyg, I gave this a quick spin and it works great for me! I actually made a quick prototype with Ansible + TextFSM for a CLI based macsuck: https://github.com/rc9000/ntcsuck. The one thing that puzzles me: it seems to only work once, then I have to restart the netdisco-web process (I was testing with the classic |
Thanks @rc9000 ! |
@rc9000 you could also try ?enqueue=true and see if it makes a difference (I wonder if running the backend job inline in the web process is breaking something - it was never intended to work that way!) ...which reminds me, that when using enqueue=true the implementation sets timestamp to when the job is run, not when you submit the data (which would be better), so I'll patch for that. |
Oh nevermind it just doesn't work in netdisco-web-fg, the requests > 1 fail with job log:
Probably because of the single process environment. When launching with |
@rc9000 but that is strange because that's how I'm developing and testing ... in the foreground. I will have a closer look... You can also use the command line, btw: |
@rc9000 aha! I can reproduce, thank you |
pushed updates to the PR which remove the enqueue option for web submit, and also fix a couple bugs |
resolves #893