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

Include code description in the output of failed controls #1096

Merged
merged 1 commit into from
Sep 20, 2016

Conversation

alexpop
Copy link
Contributor

@alexpop alexpop commented Sep 19, 2016

Fixes #1093

Using the example profile in issue #1093, this is the output based on inspec master:

$ be inspec exec /tmp/t.rb -t ssh://root@ap-cc6.opschef.tv --password 'xxx'

Target:  ssh://root@ap-cc6.opschef.tv:22

    test-kernel_parameter-999: verify certain settings in sysctl (1 failed)
       expected: >= 18419238472
          got:    128
       Kernel Parameter net.core.somaxconn value should be > 100

Profile Summary: 0 successful, 1 failures, 0 skipped

and this is the output after the code change in this PR:

$ be inspec exec /tmp/t.rb -t ssh://root@ap-cc6.opschef.tv --password 'xxx'

Target:  ssh://root@ap-cc6.opschef.tv:22

    test-kernel_parameter-999: verify certain settings in sysctl (1 failed)
       Kernel Parameter net.core.somaxconn value should be >= 18419238472
     expected: >= 18419238472
          got:    128
       Kernel Parameter net.core.somaxconn value should be > 100


Profile Summary: 0 successful, 1 failures, 0 skipped

@alexpop
Copy link
Contributor Author

alexpop commented Sep 20, 2016

Rebased and passing all tests!

@chris-rock
Copy link
Contributor

Awesome, thanks @alexpop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Feature not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants