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

enable acceptance tests on travis #416

Merged
merged 14 commits into from
Jun 22, 2018

Conversation

bastelfreak
Copy link
Member

No description provided.

@bastelfreak bastelfreak force-pushed the acceptance branch 3 times, most recently from 9f5310c to 9f45cdb Compare May 9, 2018 17:34
@bastelfreak
Copy link
Member Author

so I had the feeling that there is a bug on centos-7. The tests prove that. I will work a fix later if I find the root cause.

@bastelfreak bastelfreak changed the title [WIP] enable acceptance tests on travis enable acceptance tests on travis Jun 22, 2018
@bastelfreak
Copy link
Member Author

@solarkennedy can you please review and merge this? I pulled the centos tests out into another PR, so we can proceed here.

@solarkennedy solarkennedy merged commit c579a04 into voxpupuli:master Jun 22, 2018
@solarkennedy
Copy link
Contributor

Thanks! These are super tricky to get right.

@bastelfreak bastelfreak deleted the acceptance branch June 22, 2018 23:57
spuder pushed a commit to spuder/puppet-consul that referenced this pull request Feb 25, 2020
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