-
Notifications
You must be signed in to change notification settings - Fork 185
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
Upgrading molecule to version 3 #457
Conversation
Hmm I seem to have messed up the CI even more :) Will work off this PR to figure out what went wrong |
Looked into the errors on the
There seems to be some unhandled typing within how molecule is parsing things. Specifically, the values are unicode while molecule expects a string. I tried running this using python3 and I was able to get it to run, which leads me to suspect that the molecule project has probably abandoned python2 due to its deprecation. The molecule page on the PyPi repository https://pypi.org/project/molecule/ also suggests that it now requires python3:
|
Nevermind - turns out it was the The 1.13.0 package was published 12h ago and that's what broke. |
Also for posterity: I followed this checklist to upgrade from molecule 2.x to 3.x ansible/molecule#2560 |
Molecule 3.x changing how the
lint
directive is used. I'm removing it entirely because we're already usingansible-lint
as a separate step/stage of the CircleCI pipeline.I'm not entirely sure yamllint and flake8 linting are necessary either, but we can add those back in at a later time.