Broken dependency from pip's sh module

Hey guys,

there seem to be a fresh version of https://pypi.org/project/sh/ released 15h ago (at the time of writing this), which is a minor bump (despite being ~3 years in develop, but let’s shelve that for now). The thing is it breaks molecule converge (and possibly other tasks as well). Or at least in the instances/config I’m using it in.

The case in question is trying to run DO provisioner using 2.x branch of molecule. More details as per the ticket’s template.

When force-installed the previous version: pip install sh==1.12.14 it worked just fine. Not sure if that’s an issue of of molecule or that said package, but reporting here, since so far it seem like we’re maybe trying to pass a variable in a format that is not well formatted, given the error message. But that’s just a hunch.

Issue Type

• Bug report

Molecule and Ansible details

molecule==2.22
ansible==2.9.0

Molecule installation method: pip

Ansible installation method (one of): pip

Desired Behavior

molecule provisioning ran OK

Actual Behaviour

  'env': value u'/[a project]/provisioning/.env.yml' of env key 'MOLECULE_ENV_FILE' must be a str

please note I do not have the .env.yml file anywhere added explicitly in my provisioning folder (or subfolders for that matter).

[markgoddard]

A Python 2 fix would be nice.

[apatard]

fwiw, I’ve noticed that molecule’s molecule/test/conftest.py:run_command is broken with this new version of sh.py, breaking molecule’s testsuite. The error message is different but forcing sh == 1.12.14 makes it work again.