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

New scheduler configuration does not work well with non-standard server config #8752

Open
bartv opened this issue Feb 14, 2025 · 1 comment · May be fixed by #8805
Open

New scheduler configuration does not work well with non-standard server config #8752

bartv opened this issue Feb 14, 2025 · 1 comment · May be fixed by #8805
Assignees

Comments

@bartv
Copy link
Contributor

bartv commented Feb 14, 2025

The scheduler uses the server.server-address to connect to the orchestrator server. This is documented as: The public ip address of the server. This is required for example to inject the inmanta agent in virtual machines at boot time. This setting is actually no longer useful because we no longer start agents remotely

However, in setup where you do not set bind-address to the public ip or you use the external ip of the loadbalancer the scheduler will not be able to connect and it does not clearly say this in log messages.

The setup that caused this was the following:

  • cloudflared on the same host
  • bind-address set to 127.0.0.1
  • server-address to the public hostname orchestrator.example.com

The only hint in the logs was the agent and the server logs complaining about connection timeout. This was solved by setting server-address to 127.0.0.1

proposal

  1. connect to the bind-adress, except if it is 0.0.0.0, then we connect to 127.0.0.1
  2. make sure the error reporting is clear when the scheduler is not starting (e.g. refer to the correct log file from the server log)
@wouterdb wouterdb added discussion There is a discussion happening on this issue and removed discussion There is a discussion happening on this issue labels Feb 25, 2025
@wouterdb
Copy link
Contributor

@arnaudsjs arnaudsjs self-assigned this Mar 3, 2025
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 a pull request may close this issue.

3 participants