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

Add support for ROLESPEC_NO_REINSTALL #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add support for ROLESPEC_NO_REINSTALL #12

wants to merge 2 commits into from

Conversation

htgoebel
Copy link
Contributor

Setting this variable makes rolespec not reinstall existing software. Currently this only effects ansible.

@nickjj
Copy link
Owner

nickjj commented Jan 25, 2015

Have you taken a look at this?
https://github.com/nickjj/rolespec/#test-modes

Would any of those options work for you?

@htgoebel
Copy link
Contributor Author

Yes, I did. But -t does not install ansible at all, even if it is missing. So the developer always has to think, if ansible is already installed in his very setup at this very moment. But this is a computers job, not a human's one.

@nickjj
Copy link
Owner

nickjj commented Jan 25, 2015

It would be expected that you run rolespec normally at least once before using -t or -p.

@htgoebel
Copy link
Contributor Author

My use-case is different: I'm running the test in a VM while developing the roles. And it is simply a wast of time to reinstall esp. Ansible every few minutes. Using -t or -p is error-frown, because one has to think about running rolespec without once first. So I simply do not use -t or -p.

This may not meet you use-case, but is important for me - and it does not hurt anyone.

@nickjj
Copy link
Owner

nickjj commented Jan 26, 2015

Hmm, when you say developing the roles do you mean you have the roles sitting in the VM and you are running tests against the VM?

If so, that's exactly how I develop roles in the "I'm developing the role" stage. What's wrong with doing a run once to get things installed then just hammer away at your role using -p or -t depending on what you want to test?

@htgoebel
Copy link
Contributor Author

Are you going to integrate this pullrequest somewhen? It's pending for more than half a year now.

@htgoebel
Copy link
Contributor Author

htgoebel commented Jan 6, 2018

@nickjj Please decide whether you want to merge or close this pull-request. I want to get rid of my fork. Thanks.

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.

2 participants