Skip to content
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

Implemented check #436

Merged
merged 3 commits into from
Sep 6, 2016
Merged

Implemented check #436

merged 3 commits into from
Sep 6, 2016

Conversation

retr0h
Copy link
Contributor

@retr0h retr0h commented Sep 3, 2016

The ability to call molecule check to perform a "Dry Run". This is
currently not incorporated into the default test sequence.

Fixes: #128

@retr0h
Copy link
Contributor Author

retr0h commented Sep 3, 2016

Please provide feedback.

The ability to call `molecule check` to perform a "Dry Run".  This is
currently not incorporated into the default test sequence.

Fixes: ansible#128
@alop
Copy link
Contributor

alop commented Sep 6, 2016

this basically does ansible with the 'check' argument, right?

@alop
Copy link
Contributor

alop commented Sep 6, 2016

👍

@retr0h
Copy link
Contributor Author

retr0h commented Sep 6, 2016

this basically does ansible with the 'check' argument, right?

Yeah, and added to the workflow.

@retr0h
Copy link
Contributor Author

retr0h commented Sep 6, 2016

  syntax: commands succeeded
  py27-ansible19-unit: commands succeeded
  py27-ansible19-functional: commands succeeded
  py27-ansible20-unit: commands succeeded
  py27-ansible20-functional: commands succeeded
  py27-ansible21-unit: commands succeeded
  py27-ansible21-functional: commands succeeded
  doc: commands succeeded
  congratulations :)

@retr0h retr0h merged commit 2048180 into ansible:master Sep 6, 2016
@retr0h retr0h deleted the enhancement-128 branch September 6, 2016 19:52
@geerlingguy
Copy link
Contributor

Thanks for this, and to anyone finding this in the future, you can add it to your sequence in molecule.yml with:

scenario:
  name: default
  test_sequence:
    - lint
    - destroy
    - dependency
    - syntax
    - create
    - prepare
    - converge
    - idempotence
    # Added --check run test.
    - check
    - side_effect
    - verify
    - destroy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants