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

RuntimeWarning: coroutine 'AMIParser.asl_cmd' was never awaited #191

Closed
tsawyer opened this issue Dec 20, 2023 · 8 comments
Closed

RuntimeWarning: coroutine 'AMIParser.asl_cmd' was never awaited #191

tsawyer opened this issue Dec 20, 2023 · 8 comments

Comments

@tsawyer
Copy link
Member

tsawyer commented Dec 20, 2023

From journalctl -u allmon3.service -f running the latest Allmon3. Is this indicating one of the servers can't be login into?

Dec 20 08:19:52 XE2LRS-ASL3 allmon3[900753]: /usr/lib/python3/dist-packages/asl_allmon/allmon3_ws_voter/__init__.py:82: RuntimeWarning: coroutine 'AMIParser.asl_cmd' was never awaited
Dec 20 08:19:52 XE2LRS-ASL3 allmon3[900753]:   parser.asl_cmd("core show version")
Dec 20 08:19:52 XE2LRS-ASL3 allmon3[900753]: RuntimeWarning: Enable tracemalloc to get the object allocation traceback
@jxmx
Copy link
Member

jxmx commented Dec 20, 2023

Can't tell from that, please provide the rest of the log?

@tsawyer
Copy link
Member Author

tsawyer commented Dec 21, 2023

Here's everything from today.
dec20.log

@jxmx jxmx closed this as completed in 660d9e1 Dec 21, 2023
jxmx added a commit that referenced this issue Dec 21, 2023
Add missing await for the keepalive (Fix #191)
@jxmx
Copy link
Member

jxmx commented Dec 21, 2023

If you copy that file to /usr/lib/python3/dist-packages/asl_allmon/allmon3_ws_voter/ and re-test, it should fix your problem.
@tsawyer

@tsawyer
Copy link
Member Author

tsawyer commented Dec 22, 2023

Having trouble reproducing this. It seems restarting allmon3, wait for (or make) some RTCM activity. Then after 10 or 15 seconds of no activity the message will appear once.

@jxmx
Copy link
Member

jxmx commented Dec 22, 2023

Yes, it was a bug in the keep alive, idle code. Copy the fixed file above onto your system and let me know if it resolves it. It should.

@jxmx jxmx reopened this Dec 22, 2023
@tsawyer
Copy link
Member Author

tsawyer commented Dec 22, 2023

There is no link to any file above except the diff view. Where is the file?

@tsawyer
Copy link
Member Author

tsawyer commented Dec 22, 2023

I edited the source, added the await. So far it's ok.

@tsawyer
Copy link
Member Author

tsawyer commented Dec 22, 2023

It's been running for an hour now. No errors. Good to go.

@jxmx jxmx closed this as completed Dec 22, 2023
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

No branches or pull requests

2 participants