Skip to content

make sure audit_log_dir exists, fix problem with tmout.sh file #24

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

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

rronneburger
Copy link

  • make sure audit_log_dir exists before writing logs (until now only audit_conf_dir is handled, leading to errors if they are not the same)
  • fix problem with tmout.sh file on 24.04, when additional content is added and the previous content is still there (leading to error-messages upon login).

Overall Review of Changes:
Two small fixes to the playbook that I had running on several 24.04 machines.

How has this been tested?:
Ran the playbook on several machines setup from scratch with 24.04.

… contain readonly twice (leading to error messages upon login)
@uk-bolly
Copy link
Member

uk-bolly commented Mar 3, 2025

hi @rronneburger

Thank you for taking the time to raise this PR, unfortunately it is failing the workflow tests for 5.4.3.2. If files doesn't exist, i can see its also missing sign off and gpg signing in the commit.
Happy to take to the PR if we can get these issues resolved.

Many thanks again

uk-bolly

@rronneburger rronneburger marked this pull request as draft March 3, 2025 12:46
@rronneburger
Copy link
Author

@uk-bolly They way I understand it to sign this commit I have to reveal my email address to anyone, attracting more spam? In this case I'd rather close the PR, if you want you can still use or adopt the code as you see fit.

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 this pull request may close these issues.

2 participants