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

feat: Add molecule verify playbook #491

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

dadimah
Copy link
Contributor

@dadimah dadimah commented Jan 28, 2025

  • Add molecule verify playbook

@dadimah dadimah force-pushed the feature/add_molecule-verify branch 5 times, most recently from 7c56aef to 6b6ea60 Compare January 28, 2025 16:45
@dadimah
Copy link
Contributor Author

dadimah commented Jan 28, 2025

@geerlingguy Can u review this PR? Simple PR to verify the host is capable of running the docker containers.

@geerlingguy
Copy link
Owner

This looks great! Any particular reason for commenting out the 'become' line?

@dadimah
Copy link
Contributor Author

dadimah commented Jan 29, 2025

@geerlingguy Thank you for the prompt reivew. I had disabled the become true else during the molecule coverage gather facts is failing with below error.

Ref: https://github.com/geerlingguy/ansible-role-docker/actions/runs/13014753313

TASK [Gathering Facts] ********************************************************* fatal: [instance]: FAILED! => {"ansible_facts": {}, "changed": false, "failed_modules": {"ansible.legacy.setup": {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python3.9"}, "failed": true, "module_stderr": "sudo: PAM account management error: Authentication service cannot retrieve authentication info\nsudo: a password is required\n", "module_stdout": "", "msg": "MODULE FAILURE: No start of json char found\nSee stdout/stderr for the exact error", "rc": 1}}, "msg": "The following modules failed to execute: ansible.legacy.setup\n"}

@geerlingguy
Copy link
Owner

Ah, related to geerlingguy/docker-rockylinux9-ansible#6

Well then, I guess if it works, it ships!

@geerlingguy geerlingguy merged commit 1800d2e into geerlingguy:master Jan 29, 2025
7 checks passed
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