-
Notifications
You must be signed in to change notification settings - Fork 656
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
Add podman functional tests (#2159) #2323
Conversation
6432900
to
454700a
Compare
Hey @sshnaidm, thanks for this. I mentioned a comment in the IRC and also on #2317 (comment) regarding our plans for the drivers. I suppose the best course of action is to get this PR passing soon and merge it OR close the PR and follow the path of removing the driver code (see #2303) and then create a repostiory in a similar fashion to https://github.com/pycontribs/molecule-inspec. This is all a bit experimental right now but that's the current state. Let me know what you think. |
@sshnaidm please fix this, we need better podman coverage now more than ever, especially as we do not have docker on centos8 and rhel8. |
The problem is that podman driver will work only with patch ansible/ansible#57741 in ansible and this will be only in 2.9. I've created backport in ansible/ansible#62915 but not sure if it will be merged. |
@sshnaidm I can try to add a fixture that enables tests only on specific ansible versions. Would this help you? |
This should help you #2337 |
454700a
to
eb2d62f
Compare
Seems like failed tests are hetznercloud tests, not related to podman. And all podman tests are skipped, because it can't be installed on ubuntu. Need to test it on centos. |
6c09c66
to
a29847a
Compare
Restarted the build 👍 |
As I see podman tests are skipped, although podman is installed in last version and ansible version is 2.9rc01. I don't understand why it's still skipped: https://travis-ci.com/ansible/molecule/jobs/240039472 |
a29847a
to
9b178a1
Compare
OK, I think I found what is the problem. Current ansible
I've changed minimum version to |
OK, now podman runs finally, and fails. Will check what is the reason. |
b0ce903
to
54d3d3d
Compare
a893653
to
e9e94dd
Compare
2837dc5
to
7e8ae66
Compare
d14c124
to
8be68bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor bits, also please mention in the PR message how to test locally podman, to make it easier for others to test it locally.
I am looking forward to get this merged today!
....role_name}}/{{cookiecutter.molecule_directory}}/{{cookiecutter.scenario_name}}/molecule.yml
Outdated
Show resolved
Hide resolved
@sshnaidm I can help with the requested changes if you're running low on free cycles |
Run podman tests for molecule podman driver Closes ansible#2159 Signed-off-by: Sagi Shnaidman <sshnaidm@redhat.com>
Signed-off-by: Sagi Shnaidman sshnaidm@redhat.com
Add functional tests for podman driver
PR Type