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

NAS-129867 / 24.10 / setup HA middleware connection on service restart #13971

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

yocalebo
Copy link
Contributor

@yocalebo yocalebo commented Jul 3, 2024

Security requirements dictate that we should stop binding our internal HA connection on 0.0.0.0 so we did just that here: #13830. However, often times our developers (and our CI/CD) will systemctl restart middlewared routinely. When this happens, we don't re-bind the HA connection. This fixes that scenario so that we always re-bind on service restart.

NOTE: When this happens on a running HA machine, the local and remote sessions from BEFORE the service restart go into a TIME_WAIT status and will eventually be reaped by the kernel (as expected).

@yocalebo yocalebo requested a review from a team July 3, 2024 14:38
@bugclerk
Copy link
Contributor

bugclerk commented Jul 3, 2024

@bugclerk bugclerk changed the title setup HA middleware connection on service restart NAS-129867 / 24.10 / setup HA middleware connection on service restart Jul 3, 2024
@yocalebo yocalebo merged commit 7687364 into master Jul 3, 2024
2 of 3 checks passed
@yocalebo yocalebo deleted the NAS-129867 branch July 3, 2024 14:44
@bugclerk
Copy link
Contributor

bugclerk commented Jul 3, 2024

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Jul 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants