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

Missing var sshd_config_path for RedHat-8 #29

Closed
FrankGiesecke opened this issue Jan 20, 2022 · 4 comments · Fixed by skriptfabrik/ansible-role-sshd#2
Closed

Missing var sshd_config_path for RedHat-8 #29

FrankGiesecke opened this issue Jan 20, 2022 · 4 comments · Fixed by skriptfabrik/ansible-role-sshd#2

Comments

@FrankGiesecke
Copy link

FrankGiesecke commented Jan 20, 2022

If the target system relies on RedHat 8, the role throws an error.

TASK [arillso.sshd : set hostkeys according to openssh-version] ****************
fatal: [****]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'sshd_config_path' is undefined\n\nThe error appears to be in '/builds/mw/infrastructure/ansible-management/projects/internal-server-landscape/.ansible/roles/arillso.sshd/tasks/distribution/subtasks/crypto.yml': line 4, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: set hostkeys according to openssh-version\n  ^ here\n"}

The reason for this is, that the file vars/RedHat-8.yml doesn't contain the definition of sshd_config_path.

The variable is also missing in:

Closes #22

@felixb
Copy link
Contributor

felixb commented Jan 31, 2022

Mind adding a Rocky-8.yml to support rocky linux with this PR?

@FrankGiesecke
Copy link
Author

@felixb It might be better to create a single Issue for rocky linux support with all necessary information and fix this in a separate pull request.
Feel free to mention me in the issue and I will create a fix for this.

@felixb
Copy link
Contributor

felixb commented Jan 31, 2022

created a quick PR in #31

@FrankGiesecke
Copy link
Author

So, let us hope that the PRs will be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants