-
-
Notifications
You must be signed in to change notification settings - Fork 380
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 support for Fedora 30 #86
Conversation
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.
Please see review comments—additionally, if you could modify .travis.yml and add fedora30
to the test suite, that would be helpful in making sure the changes work and continue to work on Fedora 30 :)
Also properly list support for Fedora 30, and improve formatting in a when condition. This addresses comments in PR86: geerlingguy#86
I am not adding Fedora 30 because there is no image for it, only: docker.io/geerlingguy/docker-fedora29-ansible
Done the best I could. It looks like you only have an image for Fedora 29. |
`podman search` wasn't seeing the image, but the image exists: https://hub.docker.com/r/geerlingguy/docker-fedora30-ansible
I added this after all. Sorry for the earlier confusion. |
4679098
to
d0273ec
Compare
"python-psycopg2" is replaced by python2-psycopg2, and the verification task will fail. EL8 will need this also
by forcing the use of the C.UTF-8 locale, which is all they have installed by default
By not causing the /usr/bin/python symlink to be installed Otherwise, Ansible molecule converge would use python2, which lacks python2-dnf.
@geerlingguy I worked on this about a month ago and my memory is rusty, but I believe I implemented all of your requested changes completely. |
All requested changes implemented and build is green , let's merge 👍 |
Requires #53 first.