-
Notifications
You must be signed in to change notification settings - Fork 12
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
Molecule v2 #5
Molecule v2 #5
Conversation
4e7825a
to
8666848
Compare
See https://travis-ci.org/snoopycrimecop/ansible-roles/builds/428194277 and more specifically https://travis-ci.org/snoopycrimecop/ansible-roles/jobs/428194280 showing this works together with ome/ome-ansible-molecule#10. The number of failures gives us a feeling of how much we need to fix to be forward compatible with Ansible 2.4/Molecule 2. Here's a proposed roadmap for the next steps:
Thoughts? |
8666848
to
78f95df
Compare
Is it worth skipping straight to Ansible 2.5 (I think I've already tried this version with IDR/deployment but would need to double-check) or 2.6? |
A few immmediate thoughts about #5 (comment) although this probably will need to be sorted over a voice discussion:
|
Nothing stands out as immediately useful, though the additional support for cloud provisoining might be useful in future. If we're going through the process of testing all roles (and based on your PR description changes to roles will be required) I think we should go for 2.6 if there are no obvious problems, 2.5 if not. |
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.
One optional comment
molecule/default/INSTALL.rst
Outdated
|
||
.. code-block:: bash | ||
|
||
$ sudo pip install docker |
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.
I know it's auto-generated, but I this it'd be better to remove INSTALL.rst since we have our own set of dependencies.
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.
See ba6baa7
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.
Thanks 👍
Now released as 1.0.1 and deployed to https://galaxy.ansible.com/openmicroscopy/basedeps |
This PR makes the necessary changes to test this role with Ansible 2.4 and Molecule 2.
molecule init scenario -r $(basename $PWD) -s default -d docker
molecule test
and upgrades tests to be compliant with Molecule 2/testinfra 1.14With ome/ome-ansible-molecule#10, this PR should be tested by Travis against Ansible 2.4 in the daily https://travis-ci.org/snoopycrimecop/ansible-roles build.
Proposed tag:
1.0.1
See also #4