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

autopilot: include only servers from the same region #15290

Merged
merged 1 commit into from
Nov 17, 2022

Commits on Nov 17, 2022

  1. autopilot: include only servers from the same region

    When we migrated to the updated autopilot library in Nomad 1.4.0, the interface
    for finding servers changed. Previously autopilot would get the serf members and
    call `IsServer` on each of them, leaving it up to the implementor to filter out
    clients (and in Nomad's case, other regions). But in the "new" autopilot
    library, the equivalent interface is `KnownServers` for which we did not filter
    by region. This causes spurious attempts for the cross-region stats fetching,
    which results in TLS errors and a lot of log noise.
    
    Filter the member set by region to fix the regression.
    tgross committed Nov 17, 2022
    Configuration menu
    Copy the full SHA
    662f6db View commit details
    Browse the repository at this point in the history