-
Notifications
You must be signed in to change notification settings - Fork 2k
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
make version checks specific to region (1.2.x) #14910
make version checks specific to region (1.2.x) #14910
Conversation
a6feda2
to
cd1c80c
Compare
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.
LGTM; maybe fold these two functions together?
* One-time tokens are not replicated between regions, so we don't want to enforce that the version check across all of serf, just members in the same region. * Cleans up a bunch of legacy checks. This changeset is specific to 1.2.x and the changes for other versions of Nomad will be manually backported in separate PRs
cd1c80c
to
77e9a4e
Compare
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.
LGTM! just the test case suggestion
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
that the version check across all of serf, just members in the same region.
This changeset is specific to 1.2.x and the changes for other versions of
Nomad will be manually backported in separate PRs.
(And yes the 1.1 in the branch name is wrong 😁 )