-
Notifications
You must be signed in to change notification settings - Fork 201
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
RQD assumes it's hostname is reachable by Cuebot #1510
Comments
Have you tried using the |
Thanks @lithorus, this worked. Cuebot can now identify the proper host running RQD using its IP address. Maybe it could be tested only if there is no deliberate overrides in rqd.conf, like The full story is that I stumbled on this issue while working on a tray icon for OpenCue and I need to make sure I can reach the machine properly through the server, so I had to implement such logic and thought it could be a useful addition to the RQD codebase. |
I think using the When implementing this, please keep in mind the possibility of a future design where rqd will not directly interact with cuebot using grpc, but use a queueing mechanism (eg. Kafka), and Cuebot will continue to interact directly with Rqd using grpc. |
Are there any plans on doing single direction connection instead of bi-directional communication between cuebot and rqd? That would also solve it (and also make a network admin happy).. |
Describe the bug
Hi all !
When launched, RQD sends a host report to Cuebot stating its hostname.
But Cuebot cannot reach the RQD machine back (and doesn't know it).
In most scenarios it works, but on my setup, machines are able to communicate only via a host nickname, they cannot see the other machines via their local hostname.
(I'm using NordVPN meshnet feature to emulate a local network, I guess it should behave the same on other VPN solutions.)
To Reproduce
(with all machines on nordvpn meshnet, or any vpn I guess)
Under the hood
Expected behavior
Send a hostname that Cuebot can reach.
Suggestion: do a handshake
Version Number
Dev
The text was updated successfully, but these errors were encountered: