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

option to re_use old volume #182

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

guskovd
Copy link

@guskovd guskovd commented Mar 10, 2018

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.02%) to 98.276% when pulling f4aa2f2 on guskovd:master into c1398fb on test-kitchen:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.02%) to 98.276% when pulling f4aa2f2 on guskovd:master into c1398fb on test-kitchen:master.

@coveralls
Copy link

coveralls commented Mar 10, 2018

Coverage Status

Coverage decreased (-18.05%) to 81.25% when pulling 6fd2ac8 on guskovd:master into c1398fb on test-kitchen:master.

@jjasghar
Copy link
Contributor

Hi! Thanks for the contribution @guskovd.

Can you explain why this is needed? Also, can you deal with the failing checks?

@guskovd
Copy link
Author

guskovd commented May 23, 2018

#181

I have already tried to explain why this is necessary. My idea is to be able to store constant data between tests. (For the Docker users the idea seems obvious).
For 2 months I used a kitchen-openstack from my repository. Works as it should

@jjasghar
Copy link
Contributor

jjasghar commented Jun 1, 2018

Can you add some documentation for this? I'll like to merge this in and release this gem.

@jjasghar
Copy link
Contributor

jjasghar commented Jun 6, 2018

Having some documentation around reuse_volume would be helpful in the README.

@guskovd
Copy link
Author

guskovd commented Jun 6, 2018

OK. I will try to do it in the near future

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.

3 participants