Skip to content

feat(rabe_backup): create local user and configure authorized_keys and sudo rules - #106

Open
spameier wants to merge 3 commits into
mainfrom
feat/rabe_backup-automation
Open

feat(rabe_backup): create local user and configure authorized_keys and sudo rules#106
spameier wants to merge 3 commits into
mainfrom
feat/rabe_backup-automation

Conversation

@spameier

Copy link
Copy Markdown
Member

No description provided.

@spameier
spameier requested a review from smirta April 21, 2026 18:16
@spameier

Copy link
Copy Markdown
Member Author

@smirta please review :)

I am not really sure why currently the known_hosts file is modified by this role. As I understand it, the role is supposed to configure a system to be able to be backed up by our central system..

Comment thread roles/rabe_backup/README.md Outdated
@smirta

smirta commented Apr 22, 2026

Copy link
Copy Markdown
Member

Ugh, that's a long time ago. Looking at it, it indeed doesn't make a lot of sense. Did you test your change? How do we use it when we're using a FreeIPA-provisioned backup user? We skip the user creation and authorized_Key part?

Co-authored-by: Simon N. <smirta@users.noreply.github.com>
@spameier

Copy link
Copy Markdown
Member Author

Did you test your change?

Not yet :)

How do we use it when we're using a FreeIPA-provisioned backup user? We skip the user creation and authorized_Key part?

Yes, if the system is IPA-joined we don't specify a user and public key as these settings are configured in IPA.

@spameier

Copy link
Copy Markdown
Member Author

@hairmare how can I make ansible-lint happy here?

@hairmare

Copy link
Copy Markdown
Member

it seems to complain about this one:

syntax-check[unknown-module]: couldn't resolve module/action 'community.general.sudoers'. This often indicates a misspelling, missing collection, or incorrect module path.

afaict we need to add community.general.sudoers to galaxy.yml

rabe_foreman has some as example: https://github.com/radiorabe/ansible-collection-rabe_foreman/blob/main/galaxy.yml#L14

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.

3 participants