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.

Sign up for GitHub

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

Jump to bottom

Do we do update-rc.d for ssh_tunnel #106

Open
jchonig opened this issue Aug 21, 2024 · 2 comments
Open

Do we do update-rc.d for ssh_tunnel #106

jchonig opened this issue Aug 21, 2024 · 2 comments
Assignees
@jchonig
Labels
bug

Comments

@jchonig
Copy link
Contributor

jchonig commented Aug 21, 2024

I don't see it in the code.

Is it done because @terrillmoore 's setup scripts do it?

Add it to the handler?
@jchonig jchonig added the bug label Aug 21, 2024
@jchonig jchonig self-assigned this Aug 21, 2024
@terrillmoore
Copy link
Contributor

I don't know. It seems to be working, but ... I certainly don't remember doing anything on this in the new python scripts. I might have it buried someplace in the older shell scripts.

@jchonig
Copy link
Contributor Author

jchonig commented Aug 24, 2024

We don't do it. M/Monit is probably starting ssh_tunnel. We need to figure out the right priorities and start update-rc.d

jchonig added a commit that referenced this issue Oct 1, 2024
@jchonig
Ensure we set ssh_tunnel to restart on reboot
e9db7da 
Apparently we forgot this and monit was restarting it for us.

Also, use ansible.builtin.reboot to avoid failing the playbook when we
restart the tunnel when configuration changes.

#106
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jchonig jchonig

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jchonig @terrillmoore