-
Notifications
You must be signed in to change notification settings - Fork 2
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
testing flask gnmi timeout #32
Conversation
added a try exept on the function "sub_telemetry" and "update_port_status"
add except grpc.FutureTimeoutError to the gNMIclient to avoind timeout errors
fixed ident
write the Flask errors and output to the docker logs
fixed bug with the pygnmi timeout connection
missed a timeout and the sleep
The flask app have been changed to fix the issue with unavailable connections, it will try to connect every 10 seconds (the amount of seconds can be changed on the flask app under the function '__init__' of the class 'Telemetry'). while the nodes are not reachable you will see some messages like those bellow:
while the node is available and an GNMI connection can be established but the GNMI path is not accessible yet, can be exepted the see some exception errors link the one bellow:
after that the all will run as it is supposed to be. |
error on upgrading pip. added the '.' before '/venv/bin/python3' to force the pip to be upgraded under the virtual environment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to uncomment nodes from yml file. Those were commented only for tests.
@tiago-amado I don't think we need to proceed with this now As far as the future state of the automation panel is concerned, @siva19susi is almost done with the refactored versiont that we will use later in favor of the current python version. |
superseded by #38 |
if you run
in this branch you will see the error