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

pgbackrest: do not perform the role of ssh_keys in check mode #466

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

vitabaks
Copy link
Owner

@vitabaks vitabaks commented Sep 13, 2023

Add the ability to perform role "pgbackrest" in check mode.

ansible-playbook --check (don't make any changes; instead, try to predict some of the changes that may occur)

Fixed:

TASK [pgbackrest : ssh_keys | Ensure ssh key are created for "postgres" user on pgbackrest server] ***
ok: [10.128.150.35]
TASK [pgbackrest : ssh_keys | Ensure ssh key are created for "postgres" user on database servers] ***
ok: [10.128.150.247]
TASK [pgbackrest : ssh_keys | Get public ssh key from pgbackrest server] *******
ok: [10.128.150.35]
TASK [pgbackrest : ssh_keys | Get public ssh key from database servers] ********
ok: [10.128.150.247]
TASK [pgbackrest : ssh_keys | Add pgbackrest ssh key in "~postgres/.ssh/authorized_keys" on database servers] ***
ok: [10.128.150.247] => (item=10.128.150.35)
TASK [pgbackrest : ssh_keys | Add database ssh keys in "~postgres/.ssh/authorized_keys" on pgbackrest server] ***
ok: [10.128.150.35] => (item=10.128.150.247)
TASK [pgbackrest : known_hosts | add ssh public keys in "~postgres/.ssh/known_hosts" on database servers] ***
failed: [10.128.150.247] (item=None) => {"censored": "the output has been hidden due to the fact that 'no_log: true' was specified for this result", "changed": false}
failed: [10.128.150.247] (item=None) => {"censored": "the output has been hidden due to the fact that 'no_log: true' was specified for this result", "changed": false}
fatal: [10.128.150.247]: FAILED! => {"censored": "the output has been hidden due to the fact that 'no_log: true' was specified for this result", "changed": false}

Related PR: #459

@vitabaks vitabaks self-assigned this Sep 13, 2023
@vitabaks vitabaks merged commit 22eb132 into master Sep 13, 2023
18 checks passed
@vitabaks vitabaks deleted the pgbackrest-check-mode branch September 13, 2023 10:21
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.

1 participant