-
-
Notifications
You must be signed in to change notification settings - Fork 548
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
Serverspec to InSpec migration #460
Conversation
ae45d33
to
624c7cf
Compare
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>
Anything I can do to get this merged? |
I need to test this on all platforms and verify it truly tests for
success. I hopefully will get to that in the next week or so.
Can you confirm this does not break fixtures for chefspec? I have not had a
chance to check.
…On Jan 5, 2017 2:41 AM, "Christoph Hartmann" ***@***.***> wrote:
Anything I can do to get this merged?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#460 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAA3Ep03upyS1ynhPZYWXzqCu7aPXH0Kks5rPMjKgaJpZM4Lam9g>
.
|
ChefSpec is executed in travis: https://travis-ci.org/sous-chefs/apache2/builds/188876022#L222 I fixed the location there as well. |
@svanzoest Any updates on that? |
The only difference (apart from the verifier) is the new error from the According to inspec apache2 isn't running. But:
|
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.
Please fix the apache2 test on default-debian-86
We have a failure on Debian 8.6, but that's an Inspec issue and there's not much we can do to work around that at the moment. There's an issue for it upstream. |
Related to inspec/inspec#931 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
This PR migrates the existing integration tests to InSpec. It does this with minimal changes. Also it uses the same tests for
default
andprefork
suite.Check List