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

Upgrade: Add PATH for patronictl; Override become_method in Molecule tests #832

Merged
merged 51 commits into from
Dec 11, 2024

Conversation

vitabaks
Copy link
Owner

@vitabaks vitabaks commented Dec 7, 2024

Fixed:

  TASK [upgrade : Remove 'standby_cluster' parameter from DCS (if exists)] *******
 fatal: [10.172.2.20]: FAILED! => {"changed": false, "cmd": "patronictl -c /etc/patroni/patroni.yml edit-config -s standby_cluster=null --force", "msg": "[Errno 2] No such file or directory: b'patronictl': b'patronictl'", "rc": 2, "stderr": "", "stderr_lines": [], "stdout": "", "stdout_lines": []}

The problem was reproduced on Oracle Linux 8 - https://github.com/vitabaks/autobase/actions/runs/12208193286/job/34060985583

Additionally

Override become_method to su in molecule tests, as tests are executed under root and sudo is unnecessary.

Fixed:

  fatal: [10.172.0.20]: FAILED! => {"changed": false, "module_stderr": "sudo: PAM account management error: Authentication service cannot retrieve authentication info\nsudo: a password is required\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

@vitabaks vitabaks changed the title Upgrade: Add PATH environment for patronictl command Upgrade: Add PATH for patronictl; Enable SSH key auth in Molecule tests Dec 10, 2024
@vitabaks vitabaks changed the title Upgrade: Add PATH for patronictl; Enable SSH key auth in Molecule tests Upgrade: Add PATH for patronictl; Override become_method in Molecule tests Dec 10, 2024
@vitabaks vitabaks merged commit 2fe25cd into master Dec 11, 2024
15 checks passed
@vitabaks vitabaks deleted the patronictl-env branch December 11, 2024 07:48
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