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

sshd restart required after crypto policy update #302

Closed
meta-metaferia opened this issue Oct 30, 2024 · 1 comment · Fixed by #303
Closed

sshd restart required after crypto policy update #302

meta-metaferia opened this issue Oct 30, 2024 · 1 comment · Fixed by #303

Comments

@meta-metaferia
Copy link

On RHEL8, when these variables are set to true

sshd_sysconfig: true
sshd_sysconfig_override_crypto_policy: true

It goes and update this file /etc/sysconfig/sshd . However, at the end of the role execution it only does an sshd reload which is not enough for the change made inside /etc/sysconfig/sshd to take effect, a restart of the sshd service is required.

Jakuje added a commit to Jakuje/ansible-sshd that referenced this issue Dec 16, 2024
Jakuje added a commit to Jakuje/ansible-sshd that referenced this issue Dec 16, 2024
Some changes, such as sysconfig change, requires the service to be
restarted.

Fixes: willshersystems#302

Signed-off-by: Jakub Jelen <[email protected]>
@Jakuje
Copy link
Collaborator

Jakuje commented Dec 16, 2024

Good catch! Sorry for a delay though. I filled #303. Can you check it will solve your issue?

Jakuje added a commit to Jakuje/ansible-sshd that referenced this issue Dec 17, 2024
Some changes, such as sysconfig change, requires the service to be
restarted.

Fixes: willshersystems#302

Signed-off-by: Jakub Jelen <[email protected]>
Jakuje added a commit that referenced this issue Dec 19, 2024
Some changes, such as sysconfig change, requires the service to be
restarted.

Fixes: #302

Signed-off-by: Jakub Jelen <[email protected]>
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.

2 participants